test(persistence): minor improvement in update function tests

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

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

@@ -94,15 +94,40 @@ void main() {
       );
     }
 
-    // Updating the last added member
+    // Modifying one of the member and also adding one new
-    final updatedMember = fetchedMembers.last.copyWith(banned: true);
+    final copyMember = fetchedMembers.first.copyWith(banned: true);
-    await memberDao.updateMembers(cid, [updatedMember]);
+    final newUser = User(id: 'testUserId3');
+    final newMember = Member(
+      user: newUser,
+      banned: math.Random().nextBool(),
+      shadowBanned: math.Random().nextBool(),
+      createdAt: DateTime.now(),
+      isModerator: math.Random().nextBool(),
+      invited: math.Random().nextBool(),
+      inviteAcceptedAt: DateTime.now(),
+      role: 'testRole',
+      updatedAt: DateTime.now(),
+    );
+    await database.userDao.updateUsers([newUser]);
+    await memberDao.updateMembers(cid, [copyMember, newMember]);
 
-    // Last member banned field should match the previous updated user
+    // Fetched member length should be one more than inserted members.
+    // copyMember `banned` modified field should be true.
+    // Fetched members should contain the newMember.
     final newFetchedMembers = await memberDao.getMembersByCid(cid);
-    expect(newFetchedMembers.length, fetchedMembers.length);
+    expect(newFetchedMembers.length, fetchedMembers.length + 1);
-    final lastMember = newFetchedMembers.last;
+    expect(
-    expect(lastMember.banned, true);
+      newFetchedMembers
+          .firstWhere((it) => it.user.id == copyMember.user.id)
+          .banned,
+      true,
+    );
+    expect(
+      newFetchedMembers
+          .where((it) => it.user.id == newMember.user.id)
+          .isNotEmpty,
+      true,
+    );
   });
 
   test('deleteMemberByCids', () async {

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

@@ -242,7 +242,7 @@ void main() {
       false,
     );
     expect(
-      fetchedMessages.map((it) => it.id).contains('testMessageId${cid}4'),
+      fetchedMessages.map((it) => it.id).contains(newMessage.id),
       true,
     );
   });

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

@@ -243,7 +243,7 @@ void main() {
       false,
     );
     expect(
-      fetchedMessages.map((it) => it.id).contains('testMessageId${cid}4'),
+      fetchedMessages.map((it) => it.id).contains(newMessage.id),
       true,
     );
   });

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

@@ -128,7 +128,8 @@ void main() {
     );
     expect(
       fetchedReactions
-          .where((it) => it.userId == 'testUserId3' && it.type == 'testType3')
+          .where((it) =>
+              it.userId == newReaction.userId && it.type == newReaction.type)
           .isNotEmpty,
       true,
     );