Skip to content

Commit bd22cd2

Browse files
committed
msglist test [nfc]: Reorder to set up listener next to message list itself
1 parent f661e66 commit bd22cd2

File tree

1 file changed

+8
-12
lines changed

1 file changed

+8
-12
lines changed

test/model/message_list_test.dart

+8-12
Original file line numberDiff line numberDiff line change
@@ -64,11 +64,7 @@ void main() async {
6464
group('maybeUpdateMessage', () {
6565
test('update a message', () async {
6666
final originalMessage = eg.streamMessage(id: 243, stream: stream,
67-
content: "<p>Hello, world</p>",
68-
flags: [],
69-
);
70-
final messageList = await messageListViewWithMessages([originalMessage], stream, narrow);
71-
67+
content: "<p>Hello, world</p>");
7268
final updateEvent = UpdateMessageEvent(
7369
id: 1,
7470
messageId: originalMessage.id,
@@ -81,16 +77,17 @@ void main() async {
8177
renderingOnly: false,
8278
);
8379

80+
final messageList = await messageListViewWithMessages([originalMessage], stream, narrow);
81+
bool listenersNotified = false;
82+
messageList.addListener(() { listenersNotified = true; });
83+
8484
final message = messageList.messages.single;
8585
check(message)
8686
..content.not(it()..equals(updateEvent.renderedContent!))
8787
..lastEditTimestamp.isNull()
8888
..flags.not(it()..deepEquals(updateEvent.flags))
8989
..isMeMessage.not(it()..equals(updateEvent.isMeMessage!));
9090

91-
bool listenersNotified = false;
92-
messageList.addListener(() { listenersNotified = true; });
93-
9491
messageList.maybeUpdateMessage(updateEvent);
9592
check(listenersNotified).isTrue();
9693
check(messageList.messages.single)
@@ -104,8 +101,6 @@ void main() async {
104101
test('ignore when message not present', () async {
105102
final originalMessage = eg.streamMessage(id: 243, stream: stream,
106103
content: "<p>Hello, world</p>");
107-
final messageList = await messageListViewWithMessages([originalMessage], stream, narrow);
108-
109104
final updateEvent = UpdateMessageEvent(
110105
id: 1,
111106
messageId: originalMessage.id + 1,
@@ -117,6 +112,7 @@ void main() async {
117112
renderingOnly: false,
118113
);
119114

115+
final messageList = await messageListViewWithMessages([originalMessage], stream, narrow);
120116
bool listenersNotified = false;
121117
messageList.addListener(() { listenersNotified = true; });
122118

@@ -132,8 +128,6 @@ void main() async {
132128
final originalMessage = eg.streamMessage(id: 972, stream: stream,
133129
lastEditTimestamp: 78492,
134130
content: "<p>Hello, world</p>");
135-
final messageList = await messageListViewWithMessages([originalMessage], stream, narrow);
136-
137131
final updateEvent = UpdateMessageEvent(
138132
id: 1,
139133
messageId: originalMessage.id,
@@ -145,6 +139,8 @@ void main() async {
145139
userId: null,
146140
);
147141

142+
final messageList = await messageListViewWithMessages([originalMessage], stream, narrow);
143+
148144
final message = messageList.messages.single;
149145
messageList.maybeUpdateMessage(updateEvent);
150146
check(messageList.messages.single)

0 commit comments

Comments
 (0)