migrate to nnbd

Signed-off-by: Sahil Kumar <[email protected]>

packages/stream_chat_persistence/test/src/converter/list_coverter_test.dart

@@ -32,7 +32,7 @@ void main() {
     test('should return list of String if json data list is provided', () {
       final data = ['data1', 'data2', 'data3'];
       final res = listConverter.mapToDart(jsonEncode(data));
-      expect(res.length, data.length);
+      expect(res!.length, data.length);
     });
   });
 

packages/stream_chat_persistence/test/src/dao/channel_dao_test.dart

@@ -4,8 +4,8 @@ import 'package:stream_chat_persistence/src/db/moor_chat_database.dart';
 import 'package:test/test.dart';
 
 void main() {
-  ChannelDao channelDao;
-  MoorChatDatabase database;
+  late ChannelDao channelDao;
+  late MoorChatDatabase database;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');
@@ -32,7 +32,8 @@ void main() {
 
     // Should match the dummy channel
     final updatedChannel = await channelDao.getChannelByCid(cid);
-    expect(updatedChannel.id, id);
+    expect(updatedChannel, isNotNull);
+    expect(updatedChannel!.id, id);
     expect(updatedChannel.cid, cid);
     expect(updatedChannel.type, type);
   });
@@ -53,7 +54,8 @@ void main() {
 
     // Should match the dummy channel
     final updatedChannel = await channelDao.getChannelByCid(cid);
-    expect(updatedChannel.id, id);
+    expect(updatedChannel, isNotNull);
+    expect(updatedChannel!.id, id);
     expect(updatedChannel.cid, cid);
     expect(updatedChannel.type, type);
 
@@ -108,7 +110,8 @@ void main() {
 
     // Should match the dummy channel
     final updatedChannel = await channelDao.getChannelByCid(cid);
-    expect(updatedChannel.id, id);
+    expect(updatedChannel, isNotNull);
+    expect(updatedChannel!.id, id);
     expect(updatedChannel.cid, cid);
     expect(updatedChannel.type, type);
 
@@ -119,7 +122,8 @@ void main() {
 
     // Should match the new channel
     final newUpdatedChannel = await channelDao.getChannelByCid(cid);
-    expect(newUpdatedChannel.id, id);
+    expect(newUpdatedChannel, isNotNull);
+    expect(newUpdatedChannel!.id, id);
     expect(newUpdatedChannel.cid, cid);
     expect(newUpdatedChannel.type, newType);
   });

packages/stream_chat_persistence/test/src/dao/channel_query_dao_test.dart

@@ -8,8 +8,8 @@ import 'package:test/test.dart';
 import '../utils/date_matcher.dart';
 
 void main() {
-  MoorChatDatabase database;
-  ChannelQueryDao channelQueryDao;
+  late MoorChatDatabase database;
+  late ChannelQueryDao channelQueryDao;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');
@@ -147,7 +147,7 @@ void main() {
         // Should match lastMessageAt date
         expect(
           updatedChannel.lastMessageAt,
-          isSameDateAs(insertedChannel.lastMessageAt),
+          isSameDateAs(insertedChannel.lastMessageAt!),
         );
       }
     });
@@ -160,10 +160,7 @@ void main() {
         const pagination = PaginationParams(offset: offset, limit: limit);
 
         // Inserting test data for get channels
-        final insertedChannels = await _insertTestDataForGetChannel(
-          filter,
-          count: 30,
-        );
+        await _insertTestDataForGetChannel(filter, count: 30);
 
         // Should match with the inserted channels
         final updatedChannels = await channelQueryDao.getChannels(
@@ -210,7 +207,7 @@ void main() {
         // Should match lastMessageAt date
         expect(
           updatedChannel.lastMessageAt,
-          isSameDateAs(insertedChannel.lastMessageAt),
+          isSameDateAs(insertedChannel.lastMessageAt!),
         );
       }
     });
@@ -226,8 +223,8 @@ void main() {
 
     test('should return sorted channels using custom field', () async {
       int sortComparator(ChannelModel a, ChannelModel b) {
-        final aData = a.extraData['test_custom_field'] as int;
-        final bData = b.extraData['test_custom_field'] as int;
+        final aData = a.extraData!['test_custom_field'] as int;
+        final bData = b.extraData!['test_custom_field'] as int;
         return bData.compareTo(aData);
       }
 
@@ -261,7 +258,7 @@ void main() {
         // Should match lastMessageAt date
         expect(
           updatedChannel.lastMessageAt,
-          isSameDateAs(insertedChannel.lastMessageAt),
+          isSameDateAs(insertedChannel.lastMessageAt!),
         );
       }
     });

packages/stream_chat_persistence/test/src/dao/connection_event_dao_test.dart

@@ -6,8 +6,8 @@ import 'package:test/test.dart';
 import '../utils/date_matcher.dart';
 
 void main() {
-  ConnectionEventDao eventDao;
-  MoorChatDatabase database;
+  late ConnectionEventDao eventDao;
+  late MoorChatDatabase database;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');
@@ -30,7 +30,8 @@ void main() {
 
     // Should match the added event
     final updatedEvent = await eventDao.connectionEvent;
-    expect(updatedEvent.me.id, newEvent.me.id);
+    expect(updatedEvent, isNotNull);
+    expect(updatedEvent!.me!.id, newEvent.me!.id);
     expect(updatedEvent.totalUnreadCount, newEvent.totalUnreadCount);
     expect(updatedEvent.unreadChannels, newEvent.unreadChannels);
   });
@@ -70,7 +71,8 @@ void main() {
 
     // Should match the previously added event
     final fetchedEvent = await eventDao.connectionEvent;
-    expect(fetchedEvent.me.id, event.me.id);
+    expect(fetchedEvent, isNotNull);
+    expect(fetchedEvent!.me!.id, event.me!.id);
     expect(fetchedEvent.totalUnreadCount, event.totalUnreadCount);
     expect(fetchedEvent.unreadChannels, event.unreadChannels);
 
@@ -80,7 +82,8 @@ void main() {
 
     // Should match the updated event
     final fetchedNewEvent = await eventDao.connectionEvent;
-    expect(fetchedNewEvent.me.id, event.me.id);
+    expect(fetchedNewEvent, isNotNull);
+    expect(fetchedNewEvent!.me!.id, event.me!.id);
     expect(fetchedNewEvent.totalUnreadCount, event.totalUnreadCount);
     expect(fetchedNewEvent.unreadChannels, newEvent.unreadChannels);
   });

packages/stream_chat_persistence/test/src/dao/member_dao_test.dart

@@ -8,8 +8,8 @@ import 'package:test/test.dart';
 import '../utils/date_matcher.dart';
 
 void main() {
-  MemberDao memberDao;
-  MoorChatDatabase database;
+  late MemberDao memberDao;
+  late MoorChatDatabase database;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');
@@ -53,7 +53,7 @@ void main() {
     for (var i = 0; i < fetchedMembers.length; i++) {
       final member = memberList[i];
       final fetchedMember = fetchedMembers[i];
-      expect(fetchedMember.user.id, member.user.id);
+      expect(fetchedMember.user!.id, member.user!.id);
       expect(fetchedMember.banned, member.banned);
       expect(fetchedMember.shadowBanned, member.shadowBanned);
       expect(fetchedMember.createdAt, isSameDateAs(member.createdAt));
@@ -63,7 +63,7 @@ void main() {
       expect(fetchedMember.updatedAt, isSameDateAs(member.updatedAt));
       expect(
         fetchedMember.inviteAcceptedAt,
-        isSameDateAs(member.inviteAcceptedAt),
+        isSameDateAs(member.inviteAcceptedAt!),
       );
     }
   });
@@ -80,7 +80,7 @@ void main() {
     for (var i = 0; i < fetchedMembers.length; i++) {
       final member = memberList[i];
       final fetchedMember = fetchedMembers[i];
-      expect(fetchedMember.user.id, member.user.id);
+      expect(fetchedMember.user!.id, member.user!.id);
       expect(fetchedMember.banned, member.banned);
       expect(fetchedMember.shadowBanned, member.shadowBanned);
       expect(fetchedMember.createdAt, isSameDateAs(member.createdAt));
@@ -90,7 +90,7 @@ void main() {
       expect(fetchedMember.updatedAt, isSameDateAs(member.updatedAt));
       expect(
         fetchedMember.inviteAcceptedAt,
-        isSameDateAs(member.inviteAcceptedAt),
+        isSameDateAs(member.inviteAcceptedAt!),
       );
     }
 
@@ -118,13 +118,13 @@ void main() {
     expect(newFetchedMembers.length, fetchedMembers.length + 1);
     expect(
       newFetchedMembers
-          .firstWhere((it) => it.user.id == copyMember.user.id)
+          .firstWhere((it) => it.user!.id == copyMember.user!.id)
           .banned,
       true,
     );
     expect(
       newFetchedMembers
-          .where((it) => it.user.id == newMember.user.id)
+          .where((it) => it.user!.id == newMember.user!.id)
           .isNotEmpty,
       true,
     );

packages/stream_chat_persistence/test/src/dao/message_dao_test.dart

@@ -6,8 +6,8 @@ import 'package:stream_chat_persistence/src/db/moor_chat_database.dart';
 import 'package:test/test.dart';
 
 void main() {
-  MessageDao messageDao;
-  MoorChatDatabase database;
+  late MessageDao messageDao;
+  late MoorChatDatabase database;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');
@@ -32,7 +32,7 @@ void main() {
         shadowed: math.Random().nextBool(),
         replyCount: index,
         updatedAt: DateTime.now(),
-        extraData: {'extra_test_field': 'extraTestData'},
+        extraData: const {'extra_test_field': 'extraTestData'},
         text: 'Dummy text #$index',
         pinned: math.Random().nextBool(),
         pinnedAt: DateTime.now(),
@@ -49,7 +49,7 @@ void main() {
         shadowed: math.Random().nextBool(),
         replyCount: index,
         updatedAt: DateTime.now(),
-        extraData: {'extra_test_field': 'extraTestData'},
+        extraData: const {'extra_test_field': 'extraTestData'},
         text: 'Dummy text #$index',
         quotedMessageId: messages[index].id,
         pinned: math.Random().nextBool(),
@@ -69,7 +69,7 @@ void main() {
         shadowed: math.Random().nextBool(),
         replyCount: index,
         updatedAt: DateTime.now(),
-        extraData: {'extra_test_field': 'extraTestData'},
+        extraData: const {'extra_test_field': 'extraTestData'},
         text: 'Dummy text #$index',
         pinned: math.Random().nextBool(),
         pinnedAt: DateTime.now(),
@@ -168,7 +168,8 @@ void main() {
 
     // Fetched message id should match the inserted message id
     final fetchedMessage = await messageDao.getMessageById(id);
-    expect(fetchedMessage.id, insertedMessages.first.id);
+    expect(fetchedMessage, isNotNull);
+    expect(fetchedMessage!.id, insertedMessages.first.id);
   });
 
   test('getThreadMessages', () async {
@@ -332,7 +333,7 @@ void main() {
       showInChannel: math.Random().nextBool(),
       replyCount: 4,
       updatedAt: DateTime.now(),
-      extraData: {'extra_test_field': 'extraTestData'},
+      extraData: const {'extra_test_field': 'extraTestData'},
       text: 'Dummy text #4',
       pinned: math.Random().nextBool(),
       pinnedAt: DateTime.now(),

packages/stream_chat_persistence/test/src/dao/pinned_message_dao_test.dart

@@ -6,8 +6,8 @@ import 'package:stream_chat_persistence/src/db/moor_chat_database.dart';
 import 'package:test/test.dart';
 
 void main() {
-  PinnedMessageDao pinnedMessageDao;
-  MoorChatDatabase database;
+  late PinnedMessageDao pinnedMessageDao;
+  late MoorChatDatabase database;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');
@@ -32,7 +32,7 @@ void main() {
         shadowed: math.Random().nextBool(),
         replyCount: index,
         updatedAt: DateTime.now(),
-        extraData: {'extra_test_field': 'extraTestData'},
+        extraData: const {'extra_test_field': 'extraTestData'},
         text: 'Dummy text #$index',
         pinned: math.Random().nextBool(),
         pinnedAt: DateTime.now(),
@@ -49,7 +49,7 @@ void main() {
         shadowed: math.Random().nextBool(),
         replyCount: index,
         updatedAt: DateTime.now(),
-        extraData: {'extra_test_field': 'extraTestData'},
+        extraData: const {'extra_test_field': 'extraTestData'},
         text: 'Dummy text #$index',
         quotedMessageId: messages[index].id,
         pinned: math.Random().nextBool(),
@@ -69,7 +69,7 @@ void main() {
         shadowed: math.Random().nextBool(),
         replyCount: index,
         updatedAt: DateTime.now(),
-        extraData: {'extra_test_field': 'extraTestData'},
+        extraData: const {'extra_test_field': 'extraTestData'},
         text: 'Dummy text #$index',
         pinned: math.Random().nextBool(),
         pinnedAt: DateTime.now(),
@@ -168,7 +168,8 @@ void main() {
 
     // Fetched message id should match the inserted message id
     final fetchedMessage = await pinnedMessageDao.getMessageById(id);
-    expect(fetchedMessage.id, insertedMessages.first.id);
+    expect(fetchedMessage, isNotNull);
+    expect(fetchedMessage!.id, insertedMessages.first.id);
   });
 
   test('getThreadMessages', () async {
@@ -333,7 +334,7 @@ void main() {
       showInChannel: math.Random().nextBool(),
       replyCount: 4,
       updatedAt: DateTime.now(),
-      extraData: {'extra_test_field': 'extraTestData'},
+      extraData: const {'extra_test_field': 'extraTestData'},
       text: 'Dummy text #4',
       pinned: math.Random().nextBool(),
       pinnedAt: DateTime.now(),

packages/stream_chat_persistence/test/src/dao/reaction_dao_test.dart

@@ -6,8 +6,8 @@ import 'package:stream_chat_persistence/src/db/moor_chat_database.dart';
 import 'package:test/test.dart';
 
 void main() {
-  ReactionDao reactionDao;
-  MoorChatDatabase database;
+  late ReactionDao reactionDao;
+  late MoorChatDatabase database;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');
@@ -16,7 +16,7 @@ void main() {
 
   Future<List<Reaction>> _prepareReactionData(
     String messageId, {
-    String userId,
+    String? userId,
     int count = 3,
   }) async {
     final users = List.generate(count, (index) => User(id: 'testUserId$index'));
@@ -29,7 +29,7 @@ void main() {
       showInChannel: math.Random().nextBool(),
       replyCount: 3,
       updatedAt: DateTime.now(),
-      extraData: {'extra_test_field': 'extraTestData'},
+      extraData: const {'extra_test_field': 'extraTestData'},
       text: 'Dummy text',
       pinned: math.Random().nextBool(),
       pinnedAt: DateTime.now(),

packages/stream_chat_persistence/test/src/dao/read_dao_test.dart

@@ -6,8 +6,8 @@ import 'package:test/test.dart';
 import '../utils/date_matcher.dart';
 
 void main() {
-  ReadDao readDao;
-  MoorChatDatabase database;
+  late ReadDao readDao;
+  late MoorChatDatabase database;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');

packages/stream_chat_persistence/test/src/dao/user_dao_test.dart

@@ -6,8 +6,8 @@ import 'package:test/test.dart';
 import 'package:stream_chat/stream_chat.dart';
 
 void main() {
-  UserDao userDao;
-  MoorChatDatabase database;
+  late UserDao userDao;
+  late MoorChatDatabase database;
 
   setUp(() {
     database = MoorChatDatabase.testable('testUserId');

packages/stream_chat_persistence/test/src/mapper/channel_mapper_test.dart

@@ -34,15 +34,21 @@ void main() {
       expect(channelModel.updatedAt, isSameDateAs(entity.updatedAt));
       expect(channelModel.memberCount, entity.memberCount);
       expect(channelModel.cid, entity.cid);
-      expect(channelModel.lastMessageAt, isSameDateAs(entity.lastMessageAt));
-      expect(channelModel.deletedAt, isSameDateAs(entity.deletedAt));
+      expect(channelModel.lastMessageAt, isSameDateAs(entity.lastMessageAt!));
+      expect(channelModel.deletedAt, isSameDateAs(entity.deletedAt!));
       expect(channelModel.extraData, entity.extraData);
-      expect(channelModel.createdBy.id, entity.createdById);
+      expect(channelModel.createdBy!.id, entity.createdById);
     });
 
     test('toChannelState should map entity into ChannelState ', () {
       final members = List.generate(3, (index) => Member());
-      final reads = List.generate(3, (index) => Read());
+      final reads = List.generate(
+        3,
+        (index) => Read(
+          user: User(id: 'testUserId$index'),
+          lastRead: DateTime.now(),
+        ),
+      );
       final messages = List.generate(3, (index) => Message());
 
       final channelState = entity.toChannelState(
@@ -59,7 +65,7 @@ void main() {
       expect(channelState.messages.length, messages.length);
       expect(channelState.pinnedMessages.length, messages.length);
 
-      final channelModel = channelState.channel;
+      final channelModel = channelState.channel!;
       expect(channelModel.id, entity.id);
       expect(channelModel.config.toJson()['max_message_length'], 33);
       expect(channelModel.frozen, entity.frozen);
@@ -67,10 +73,10 @@ void main() {
       expect(channelModel.updatedAt, isSameDateAs(entity.updatedAt));
       expect(channelModel.memberCount, entity.memberCount);
       expect(channelModel.cid, entity.cid);
-      expect(channelModel.lastMessageAt, isSameDateAs(entity.lastMessageAt));
-      expect(channelModel.deletedAt, isSameDateAs(entity.deletedAt));
+      expect(channelModel.lastMessageAt, isSameDateAs(entity.lastMessageAt!));
+      expect(channelModel.deletedAt, isSameDateAs(entity.deletedAt!));
       expect(channelModel.extraData, entity.extraData);
-      expect(channelModel.createdBy.id, entity.createdById);
+      expect(channelModel.createdBy!.id, entity.createdById);
     });
   });
 
@@ -103,9 +109,9 @@ void main() {
     expect(channelEntity.updatedAt, isSameDateAs(model.updatedAt));
     expect(channelEntity.memberCount, model.memberCount);
     expect(channelEntity.cid, model.cid);
-    expect(channelEntity.lastMessageAt, isSameDateAs(model.lastMessageAt));
-    expect(channelEntity.deletedAt, isSameDateAs(model.deletedAt));
+    expect(channelEntity.lastMessageAt, isSameDateAs(model.lastMessageAt!));
+    expect(channelEntity.deletedAt, isSameDateAs(model.deletedAt!));
     expect(channelEntity.extraData, model.extraData);
-    expect(channelEntity.createdById, model.createdBy.id);
+    expect(channelEntity.createdById, model.createdBy!.id);
   });
 }

packages/stream_chat_persistence/test/src/mapper/event_mapper_test.dart

@@ -16,7 +16,7 @@ void main() {
     );
     final event = entity.toEvent();
     expect(event, isA<Event>());
-    expect(event.me.id, ownUser.id);
+    expect(event.me!.id, ownUser.id);
     expect(event.totalUnreadCount, entity.totalUnreadCount);
     expect(event.unreadChannels, entity.unreadChannels);
   });

packages/stream_chat_persistence/test/src/mapper/member_mapper_test.dart

@@ -25,12 +25,12 @@ void main() {
     );
     final member = entity.toMember(user: user);
     expect(member, isA<Member>());
-    expect(member.user.id, entity.userId);
+    expect(member.user!.id, entity.userId);
     expect(member.createdAt, isSameDateAs(entity.createdAt));
     expect(member.updatedAt, isSameDateAs(entity.updatedAt));
     expect(member.role, entity.role);
-    expect(member.inviteAcceptedAt, isSameDateAs(entity.inviteAcceptedAt));
-    expect(member.inviteRejectedAt, isSameDateAs(entity.inviteRejectedAt));
+    expect(member.inviteAcceptedAt, isSameDateAs(entity.inviteAcceptedAt!));
+    expect(member.inviteRejectedAt, isSameDateAs(entity.inviteRejectedAt!));
     expect(member.invited, entity.invited);
     expect(member.banned, entity.banned);
     expect(member.shadowBanned, entity.shadowBanned);
@@ -55,12 +55,12 @@ void main() {
     final entity = member.toEntity(cid: cid);
     expect(entity, isA<MemberEntity>());
     expect(entity.channelCid, cid);
-    expect(entity.userId, member.user.id);
+    expect(entity.userId, member.user!.id);
     expect(entity.createdAt, isSameDateAs(member.createdAt));
     expect(entity.updatedAt, isSameDateAs(member.updatedAt));
     expect(entity.role, member.role);
-    expect(entity.inviteAcceptedAt, isSameDateAs(member.inviteAcceptedAt));
-    expect(entity.inviteRejectedAt, isSameDateAs(member.inviteRejectedAt));
+    expect(entity.inviteAcceptedAt, isSameDateAs(member.inviteAcceptedAt!));
+    expect(entity.inviteRejectedAt, isSameDateAs(member.inviteRejectedAt!));
     expect(entity.invited, member.invited);
     expect(entity.banned, member.banned);
     expect(entity.shadowBanned, member.shadowBanned);

packages/stream_chat_persistence/test/src/mapper/message_mapper_test.dart

@@ -32,7 +32,7 @@ void main() {
     );
     final entity = MessageEntity(
       id: 'testMessageId',
-      attachments: attachments?.map((it) => jsonEncode(it.toData()))?.toList(),
+      attachments: attachments.map((it) => jsonEncode(it.toData())).toList(),
       channelCid: 'testCid',
       type: 'testType',
       parentId: 'testParentId',
@@ -46,8 +46,9 @@ void main() {
       reactionCounts: reactions.fold(
         {},
         (prev, curr) =>
-            prev..update(curr.type, (value) => value + 1, ifAbsent: () => 1),
+            prev?..update(curr.type, (value) => value + 1, ifAbsent: () => 1),
       ),
+      mentionedUsers: const [],
       status: MessageSendingStatus.sent,
       updatedAt: DateTime.now(),
       extraData: {'extra_test_data': 'extraData'},
@@ -82,13 +83,13 @@ void main() {
     expect(message.status, entity.status);
     expect(message.updatedAt, isSameDateAs(entity.updatedAt));
     expect(message.extraData, entity.extraData);
-    expect(message.user.id, entity.userId);
-    expect(message.deletedAt, isSameDateAs(entity.deletedAt));
+    expect(message.user!.id, entity.userId);
+    expect(message.deletedAt, isSameDateAs(entity.deletedAt!));
     expect(message.text, entity.messageText);
     expect(message.pinned, entity.pinned);
-    expect(message.pinExpires, isSameDateAs(entity.pinExpires));
-    expect(message.pinnedAt, isSameDateAs(entity.pinnedAt));
-    expect(message.pinnedBy.id, entity.pinnedByUserId);
+    expect(message.pinExpires, isSameDateAs(entity.pinExpires!));
+    expect(message.pinnedAt, isSameDateAs(entity.pinnedAt!));
+    expect(message.pinnedBy!.id, entity.pinnedByUserId);
     expect(message.reactionCounts, entity.reactionCounts);
     expect(message.reactionScores, entity.reactionScores);
     for (var i = 0; i < message.attachments.length; i++) {
@@ -138,11 +139,11 @@ void main() {
       reactionCounts: reactions.fold(
         {},
         (prev, curr) =>
-            prev..update(curr.type, (value) => value + 1, ifAbsent: () => 1),
+            prev?..update(curr.type, (value) => value + 1, ifAbsent: () => 1),
       ),
       status: MessageSendingStatus.sending,
       updatedAt: DateTime.now(),
-      extraData: {'extra_test_data': 'extraData'},
+      extraData: const {'extra_test_data': 'extraData'},
       user: user,
       deletedAt: DateTime.now(),
       text: 'dummy text',
@@ -167,18 +168,18 @@ void main() {
     expect(entity.status, message.status);
     expect(entity.updatedAt, isSameDateAs(message.updatedAt));
     expect(entity.extraData, message.extraData);
-    expect(entity.userId, message.user.id);
-    expect(entity.deletedAt, isSameDateAs(message.deletedAt));
+    expect(entity.userId, message.user!.id);
+    expect(entity.deletedAt, isSameDateAs(message.deletedAt!));
     expect(entity.messageText, message.text);
     expect(entity.pinned, message.pinned);
-    expect(entity.pinExpires, isSameDateAs(message.pinExpires));
-    expect(entity.pinnedAt, isSameDateAs(message.pinnedAt));
-    expect(entity.pinnedByUserId, message.pinnedBy.id);
+    expect(entity.pinExpires, isSameDateAs(message.pinExpires!));
+    expect(entity.pinnedAt, isSameDateAs(message.pinnedAt!));
+    expect(entity.pinnedByUserId, message.pinnedBy!.id);
     expect(entity.reactionCounts, message.reactionCounts);
     expect(entity.reactionScores, message.reactionScores);
     expect(
       entity.attachments,
-      message.attachments?.map((it) => jsonEncode(it.toData()))?.toList(),
+      message.attachments.map((it) => jsonEncode(it.toData())).toList(),
     );
   });
 }

packages/stream_chat_persistence/test/src/mapper/pinned_message_mapper_test.dart

@@ -32,7 +32,7 @@ void main() {
     );
     final entity = PinnedMessageEntity(
       id: 'testMessageId',
-      attachments: attachments?.map((it) => jsonEncode(it.toData()))?.toList(),
+      attachments: attachments.map((it) => jsonEncode(it.toData())).toList(),
       channelCid: 'testCid',
       type: 'testType',
       parentId: 'testParentId',
@@ -46,8 +46,9 @@ void main() {
       reactionCounts: reactions.fold(
         {},
         (prev, curr) =>
-            prev..update(curr.type, (value) => value + 1, ifAbsent: () => 1),
+            prev?..update(curr.type, (value) => value + 1, ifAbsent: () => 1),
       ),
+      mentionedUsers: [],
       status: MessageSendingStatus.sent,
       updatedAt: DateTime.now(),
       extraData: {'extra_test_data': 'extraData'},
@@ -82,13 +83,13 @@ void main() {
     expect(message.status, entity.status);
     expect(message.updatedAt, isSameDateAs(entity.updatedAt));
     expect(message.extraData, entity.extraData);
-    expect(message.user.id, entity.userId);
-    expect(message.deletedAt, isSameDateAs(entity.deletedAt));
+    expect(message.user!.id, entity.userId);
+    expect(message.deletedAt, isSameDateAs(entity.deletedAt!));
     expect(message.text, entity.messageText);
     expect(message.pinned, entity.pinned);
-    expect(message.pinExpires, isSameDateAs(entity.pinExpires));
-    expect(message.pinnedAt, isSameDateAs(entity.pinnedAt));
-    expect(message.pinnedBy.id, entity.pinnedByUserId);
+    expect(message.pinExpires, isSameDateAs(entity.pinExpires!));
+    expect(message.pinnedAt, isSameDateAs(entity.pinnedAt!));
+    expect(message.pinnedBy!.id, entity.pinnedByUserId);
     expect(message.reactionCounts, entity.reactionCounts);
     expect(message.reactionScores, entity.reactionScores);
     for (var i = 0; i < message.attachments.length; i++) {
@@ -138,11 +139,11 @@ void main() {
       reactionCounts: reactions.fold(
         {},
         (prev, curr) =>
-            prev..update(curr.type, (value) => value + 1, ifAbsent: () => 1),
+            prev?..update(curr.type, (value) => value + 1, ifAbsent: () => 1),
       ),
       status: MessageSendingStatus.sending,
       updatedAt: DateTime.now(),
-      extraData: {'extra_test_data': 'extraData'},
+      extraData: const {'extra_test_data': 'extraData'},
       user: user,
       deletedAt: DateTime.now(),
       text: 'dummy text',
@@ -167,18 +168,18 @@ void main() {
     expect(entity.status, message.status);
     expect(entity.updatedAt, isSameDateAs(message.updatedAt));
     expect(entity.extraData, message.extraData);
-    expect(entity.userId, message.user.id);
-    expect(entity.deletedAt, isSameDateAs(message.deletedAt));
+    expect(entity.userId, message.user!.id);
+    expect(entity.deletedAt, isSameDateAs(message.deletedAt!));
     expect(entity.messageText, message.text);
     expect(entity.pinned, message.pinned);
-    expect(entity.pinExpires, isSameDateAs(message.pinExpires));
-    expect(entity.pinnedAt, isSameDateAs(message.pinnedAt));
-    expect(entity.pinnedByUserId, message.pinnedBy.id);
+    expect(entity.pinExpires, isSameDateAs(message.pinExpires!));
+    expect(entity.pinnedAt, isSameDateAs(message.pinnedAt!));
+    expect(entity.pinnedByUserId, message.pinnedBy!.id);
     expect(entity.reactionCounts, message.reactionCounts);
     expect(entity.reactionScores, message.reactionScores);
     expect(
       entity.attachments,
-      message.attachments?.map((it) => jsonEncode(it.toData()))?.toList(),
+      message.attachments.map((it) => jsonEncode(it.toData())).toList(),
     );
   });
 }

packages/stream_chat_persistence/test/src/mapper/user_mapper_test.dart

@@ -24,7 +24,7 @@ void main() {
     expect(user.role, entity.role);
     expect(user.createdAt, isSameDateAs(entity.createdAt));
     expect(user.updatedAt, isSameDateAs(entity.updatedAt));
-    expect(user.lastActive, isSameDateAs(entity.lastActive));
+    expect(user.lastActive, isSameDateAs(entity.lastActive!));
     expect(user.online, entity.online);
     expect(user.banned, entity.banned);
     expect(user.extraData, entity.extraData);
@@ -47,7 +47,7 @@ void main() {
     expect(entity.role, user.role);
     expect(entity.createdAt, isSameDateAs(user.createdAt));
     expect(entity.updatedAt, isSameDateAs(user.updatedAt));
-    expect(entity.lastActive, isSameDateAs(user.lastActive));
+    expect(entity.lastActive, isSameDateAs(user.lastActive!));
     expect(entity.online, user.online);
     expect(entity.banned, user.banned);
     expect(entity.extraData, user.extraData);

packages/stream_chat_persistence/test/src/utils/date_matcher.dart

@@ -1,13 +1,10 @@
 import 'package:test/test.dart';
-import 'package:meta/meta.dart';
 
 Matcher isSameDateAs(DateTime targetDate) =>
     _IsSameDateAs(targetDate: targetDate);
 
 class _IsSameDateAs extends Matcher {
-  const _IsSameDateAs({
-    @required this.targetDate,
-  }) : assert(targetDate != null, '');
+  const _IsSameDateAs({required this.targetDate});
 
   final DateTime targetDate;