message test [nfc]: Move tests to a new home

While they are no longer integrations tests, we make it clearer that
these tests correspond to the code responsible for parsing move data.

This is also the more appropriate place to test them thoroughly,
by focusing on the parsed result.

Signed-off-by: Zixuan James Li <zixuan@zulip.com>

Author: PIG208

test/api/model/events_test.dart

@@ -162,6 +162,45 @@ void main() {
         ..origStreamId.equals(1)
         ..newStreamId.equals(1);
     });
+
+    test('no message move', () {
+      check(Event.fromJson({ ...baseJson,
+        'orig_content': 'foo',
+        'orig_rendered_content': 'foo',
+        'content': 'bar',
+        'rendered_content': 'bar',
+      })).isA<UpdateMessageEvent>().moveData.isNull();
+    });
+
+    test('stream move but no orig_subject', () {
+      check(() => Event.fromJson({ ...baseMoveJson,
+        'stream_id': 1,
+        'new_stream_id': 2,
+        'orig_subject': null,
+      })).throws<void>();
+    });
+
+    test('move but no subject or new_stream_id', () {
+      check(() => Event.fromJson({ ...baseMoveJson,
+        'new_stream_id': null,
+        'subject': null,
+      })).throws<FormatException>();
+    });
+
+    test('move but no orig_stream_id', () {
+      check(() => Event.fromJson({ ...baseMoveJson,
+        'stream_id': null,
+        'new_stream_id': 2,
+      })).throws<void>();
+    });
+
+    test('move but no propagate_mode', () {
+      check(() => Event.fromJson({ ...baseMoveJson,
+        'orig_subject': 'foo',
+        'subject': 'bar',
+        'propagate_mode': null,
+      })).throws<void>();
+    });
   });
 
   test('delete_message: require streamId and topic for stream messages', () {

test/model/message_test.dart

@@ -377,82 +377,6 @@ void main() {
     });
   });
 
-  group('message move', () {
-    final message = eg.streamMessage();
-    final baseJson = {
-      'id': 1,
-      'type': 'update_message',
-      'user_id': eg.selfUser.userId,
-      'rendering_only': false,
-      'message_id': message.id,
-      'message_ids': [message.id],
-      'flags': <String>[],
-      'edit_timestamp': 1718741351,
-      'stream_id': eg.stream().streamId,
-    };
-    final baseMoveJson = { ...baseJson,
-      'orig_subject': 'foo',
-      'propagate_mode': 'change_all',
-    };
-
-    Future<void> setupAndHandleEvent(Map<String, Object?> json) async {
-      await prepare();
-      await prepareMessages([message]);
-      await store.handleEvent(Event.fromJson(json) as UpdateMessageEvent);
-    }
-
-    test('smoke', () async {
-      await setupAndHandleEvent({ ...baseMoveJson,
-        'stream_id': 1,
-        'new_stream_id': 2,
-      });
-      checkNotified(count: 2);
-      check(store).messages[message.id].isA<StreamMessage>()
-        ..topic.equals(message.topic)
-        ..streamId.equals(2);
-    });
-
-    test('no message move', () async {
-      await setupAndHandleEvent({ ...baseJson,
-        'orig_content': 'foo',
-        'orig_rendered_content': 'foo',
-        'content': 'bar',
-        'rendered_content': 'bar',
-      });
-      checkNotifiedOnce();
-    });
-
-    test('stream move but no orig_subject', () async {
-      await check(setupAndHandleEvent({ ...baseMoveJson,
-        'stream_id': 1,
-        'new_stream_id': 2,
-        'orig_subject': null,
-      })).throws();
-    });
-
-    test('move but no subject or new_stream_id', () async {
-      await check(setupAndHandleEvent({ ...baseMoveJson,
-        'new_stream_id': null,
-        'subject': null,
-      })).throws<FormatException>();
-    });
-
-    test('move but no orig_stream_id', () async {
-      await check(setupAndHandleEvent({ ...baseMoveJson,
-        'stream_id': null,
-        'new_stream_id': 2,
-      })).throws();
-    });
-
-    test('move but no propagate_mode', () async {
-      await check(setupAndHandleEvent({ ...baseMoveJson,
-        'orig_subject': 'foo',
-        'subject': 'bar',
-        'propagate_mode': null,
-      })).throws();
-    });
-  });
-
   group('handleDeleteMessageEvent', () {
     test('delete an unknown message', () async {
       final message1 = eg.streamMessage();