Skip to content

Commit 1f97563

Browse files
committed
message test [nfc]: Tighten message-deletion test slightly
The `length` check isn't needed because it's covered by the `deepEquals`. (It was needed in a previous draft of this test). Then we can also inline `expected`, and reformat slightly.
1 parent c4ca1ab commit 1f97563

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

test/model/message_list_test.dart

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -321,12 +321,11 @@ void main() {
321321
check(model).messages.length.equals(30);
322322
await store.handleEvent(eg.deleteMessageEvent(messagesToDelete));
323323
checkNotifiedOnce();
324-
final expected = [
324+
check(model.messages.map((message) => message.id)).deepEquals([
325325
...messages.sublist(0, 2),
326326
...messages.sublist(5, 10),
327-
...messages.sublist(15)].map((message) => message.id);
328-
check(model).messages.length.equals(expected.length);
329-
check(model.messages.map((message) => message.id)).deepEquals(expected);
327+
...messages.sublist(15),
328+
].map((message) => message.id));
330329
});
331330
});
332331

0 commit comments

Comments
 (0)