Skip to content

Commit d7c64cb

Browse files
committed
msglist test: Similarly tighten rendering-only test cases
1 parent 1e972e2 commit d7c64cb

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

test/model/message_list_test.dart

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -141,13 +141,9 @@ void main() async {
141141
Future<void> checkRenderingOnly({required bool legacy}) async {
142142
final store = await setupStore(stream);
143143

144-
const oldContent = "<p>Hello, world</p>";
145-
const oldTimestamp = 78492;
146-
const newContent = "<p>Hello, world</p> <div>Some link preview</div>";
147-
const newTimestamp = 99999;
148-
149-
final originalMessage = eg.streamMessage(id: 972, stream: stream, content: oldContent);
150-
originalMessage.lastEditTimestamp = oldTimestamp;
144+
final originalMessage = eg.streamMessage(id: 972, stream: stream,
145+
content: "<p>Hello, world</p>");
146+
originalMessage.lastEditTimestamp = 78492;
151147

152148
final messageList = await messageListViewWithMessages([originalMessage], store, narrow);
153149

@@ -156,17 +152,21 @@ void main() async {
156152
messageId: originalMessage.id,
157153
messageIds: [originalMessage.id],
158154
flags: originalMessage.flags,
159-
renderedContent: newContent,
160-
editTimestamp: newTimestamp,
155+
renderedContent: "<p>Hello, world</p> <div>Some link preview</div>",
156+
editTimestamp: 99999,
161157
renderingOnly: legacy ? null : true,
162158
userId: null,
163159
);
164160

165161
final message = messageList.messages.single;
166162
messageList.maybeUpdateMessage(updateEvent);
167-
check(message)
168-
..content.equals(newContent)
169-
..lastEditTimestamp.equals(oldTimestamp);
163+
check(messageList.messages.single)
164+
..identicalTo(message)
165+
// Content is updated...
166+
..content.equals(updateEvent.renderedContent!)
167+
// ... edit timestamp is not.
168+
..lastEditTimestamp.equals(originalMessage.lastEditTimestamp)
169+
..lastEditTimestamp.not(it()..equals(updateEvent.editTimestamp));
170170
}
171171

172172
test('rendering-only update does not change timestamp', () async {

0 commit comments

Comments
 (0)