Skip to content

Commit 82d19e7

Browse files
committed
refactor(fixtures): enhance in-app notification data structure
- Add unique notificationId to each in-app notification - Include notificationType using PushNotificationSubscriptionDeliveryType - Rename headlineId key to be more descriptive - Update data structure to accommodate future extensions
1 parent 47b5a71 commit 82d19e7

File tree

1 file changed

+42
-6
lines changed

1 file changed

+42
-6
lines changed

lib/src/fixtures/in_app_notifications.dart

Lines changed: 42 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,13 @@ final List<InAppNotification> inAppNotificationsFixturesData = [
2222
title: _headlineById(kHeadlineId11).title,
2323
body: _headlineById(kHeadlineId11).excerpt,
2424
imageUrl: _headlineById(kHeadlineId11).imageUrl,
25-
data: const {'contentType': 'headline', 'id': kHeadlineId11},
25+
data: {
26+
'notificationId': kInAppNotificationId1,
27+
'notificationType':
28+
PushNotificationSubscriptionDeliveryType.breakingOnly.name,
29+
'contentType': 'headline',
30+
'headlineId': kHeadlineId11,
31+
},
2632
),
2733
createdAt: DateTime.now().subtract(const Duration(minutes: 15)),
2834
readAt: null,
@@ -35,7 +41,13 @@ final List<InAppNotification> inAppNotificationsFixturesData = [
3541
title: _headlineById(kHeadlineId24).title,
3642
body: _headlineById(kHeadlineId24).excerpt,
3743
imageUrl: _headlineById(kHeadlineId24).imageUrl,
38-
data: const {'contentType': 'headline', 'id': kHeadlineId24},
44+
data: {
45+
'notificationId': kInAppNotificationId2,
46+
'notificationType':
47+
PushNotificationSubscriptionDeliveryType.breakingOnly.name,
48+
'contentType': 'headline',
49+
'headlineId': kHeadlineId24,
50+
},
3951
),
4052
createdAt: DateTime.now().subtract(const Duration(days: 1)),
4153
readAt: DateTime.now().subtract(const Duration(hours: 12)),
@@ -48,7 +60,13 @@ final List<InAppNotification> inAppNotificationsFixturesData = [
4860
title: _headlineById(kHeadlineId37).title,
4961
body: _headlineById(kHeadlineId37).excerpt,
5062
imageUrl: _headlineById(kHeadlineId37).imageUrl,
51-
data: const {'contentType': 'headline', 'id': kHeadlineId37},
63+
data: {
64+
'notificationId': kInAppNotificationId3,
65+
'notificationType':
66+
PushNotificationSubscriptionDeliveryType.breakingOnly.name,
67+
'contentType': 'headline',
68+
'headlineId': kHeadlineId37,
69+
},
5270
),
5371
createdAt: DateTime.now().subtract(const Duration(hours: 2)),
5472
readAt: null,
@@ -63,7 +81,13 @@ final List<InAppNotification> inAppNotificationsFixturesData = [
6381
title: _headlineById(kHeadlineId1).title,
6482
body: _headlineById(kHeadlineId1).excerpt,
6583
imageUrl: _headlineById(kHeadlineId1).imageUrl,
66-
data: const {'contentType': 'headline', 'id': kHeadlineId1},
84+
data: {
85+
'notificationId': kInAppNotificationId4,
86+
'notificationType':
87+
PushNotificationSubscriptionDeliveryType.breakingOnly.name,
88+
'contentType': 'headline',
89+
'headlineId': kHeadlineId1,
90+
},
6791
),
6892
createdAt: DateTime.now().subtract(const Duration(minutes: 5)),
6993
readAt: null,
@@ -76,7 +100,13 @@ final List<InAppNotification> inAppNotificationsFixturesData = [
76100
title: _headlineById(kHeadlineId2).title,
77101
body: _headlineById(kHeadlineId2).excerpt,
78102
imageUrl: _headlineById(kHeadlineId2).imageUrl,
79-
data: const {'contentType': 'headline', 'id': kHeadlineId2},
103+
data: {
104+
'notificationId': kInAppNotificationId5,
105+
'notificationType':
106+
PushNotificationSubscriptionDeliveryType.breakingOnly.name,
107+
'contentType': 'headline',
108+
'headlineId': kHeadlineId2,
109+
},
80110
),
81111
createdAt: DateTime.now().subtract(const Duration(days: 2)),
82112
readAt: DateTime.now().subtract(const Duration(days: 1)),
@@ -89,7 +119,13 @@ final List<InAppNotification> inAppNotificationsFixturesData = [
89119
title: _headlineById(kHeadlineId7).title,
90120
body: _headlineById(kHeadlineId7).excerpt,
91121
imageUrl: _headlineById(kHeadlineId7).imageUrl,
92-
data: const {'contentType': 'headline', 'id': kHeadlineId7},
122+
data: {
123+
'notificationId': kInAppNotificationId6,
124+
'notificationType':
125+
PushNotificationSubscriptionDeliveryType.breakingOnly.name,
126+
'contentType': 'headline',
127+
'headlineId': kHeadlineId7,
128+
},
93129
),
94130
createdAt: DateTime.now().subtract(const Duration(hours: 4)),
95131
readAt: null,

0 commit comments

Comments
 (0)