model: update _handle_update_messages_event.

Uses the analyse_edit_history funciton from helper.py
The parameters passed to analyse_edit_history is slightly
diffrent from earlier as it gets the required values from event
information.
Tests adapted.

Author: Subhasish-Behera

tests/model/test_model.py

@@ -2150,7 +2150,8 @@ def test_notify_users_hides_PM_content_based_on_user_setting(
                     "topic_msg_ids": {
                         10: {"new subject": {1}, "old subject": {2}},
                     },
-                    "edited_messages": {1},
+                    "edited_messages": set(),
+                    "moved_messages": {1},
                     "topics": {10: []},
                 },
                 False,
@@ -2186,7 +2187,8 @@ def test_notify_users_hides_PM_content_based_on_user_setting(
                     "topic_msg_ids": {
                         10: {"new subject": {1, 2}, "old subject": set()},
                     },
-                    "edited_messages": {1},
+                    "edited_messages": set(),
+                    "moved_messages": {1},
                     "topics": {10: []},
                 },
                 False,
@@ -2292,6 +2294,7 @@ def test_notify_users_hides_PM_content_based_on_user_setting(
                         10: {"new subject": set(), "old subject": {1, 2}},
                     },
                     "edited_messages": {1},
+                    "moved_messages": set(),
                     "topics": {10: ["new subject", "old subject"]},
                 },
                 False,
@@ -2329,7 +2332,8 @@ def test_notify_users_hides_PM_content_based_on_user_setting(
                     "topic_msg_ids": {
                         10: {"new subject": {1}, "old subject": {2}},
                     },
-                    "edited_messages": {1},
+                    "edited_messages": set(),
+                    "moved_messages": {1},
                     "topics": {10: []},
                 },
                 False,
@@ -2363,6 +2367,7 @@ def test_notify_users_hides_PM_content_based_on_user_setting(
                         10: {"new subject": set(), "old subject": {1, 2}},
                     },
                     "edited_messages": {1},
+                    "moved_messages": set(),
                     "topics": {10: ["new subject", "old subject"]},
                 },
                 False,
@@ -2401,6 +2406,7 @@ def test_notify_users_hides_PM_content_based_on_user_setting(
                         10: {"new subject": {3}, "old subject": {1, 2}},
                     },
                     "edited_messages": set(),
+                    "moved_messages": set(),
                     "topics": {10: []},  # This resets the cache
                 },
                 False,
@@ -2439,6 +2445,7 @@ def test_notify_users_hides_PM_content_based_on_user_setting(
                         10: {"new subject": {3}, "old subject": {1, 2}},
                     },
                     "edited_messages": set(),
+                    "moved_messages": set(),
                     "topics": {10: ["new subject", "old subject"]},
                 },
                 True,
@@ -2476,7 +2483,8 @@ def test_notify_users_hides_PM_content_based_on_user_setting(
                     "topic_msg_ids": {
                         10: {"new subject": {1}, "old subject": {2}},
                     },
-                    "edited_messages": {1},
+                    "edited_messages": set(),
+                    "moved_messages": {1},
                     "topics": {10: ["new subject", "old subject"]},
                 },
                 True,
@@ -2512,6 +2520,7 @@ def test__handle_update_message_event(
                 10: {"new subject": set(), "old subject": {1, 2}},
             },
             "edited_messages": set(),
+            "moved_messages": set(),
             "topics": {10: ["new subject", "old subject"]},
         }
         mocker.patch(MODEL + "._update_rendered_view")

zulipterminal/model.py

@@ -64,6 +64,7 @@
     StreamData,
     TidiedUserInfo,
     UserStatus,
+    analyse_edit_history,
     asynch,
     canonicalize_color,
     classify_unread_counts,
@@ -1709,9 +1710,29 @@ def _handle_update_message_event(self, event: Event) -> None:
         #       they are not all marked as edited, as per server optimization
         message_id = event["message_id"]
         indexed_message = self.index["messages"].get(message_id, None)
-
+        current_topic = None
+        old_topic = None
         if indexed_message:
-            self.index["edited_messages"].add(message_id)
+            if "prev_content" in event:
+                content_changed = True
+            else:
+                content_changed = False
+            if "prev_stream" in event:
+                stream_changed = True
+            else:
+                stream_changed = False
+            if "subject" in event:
+                current_topic = event["subject"]
+                old_topic = event["orig_subject"]
+
+            analyse_edit_history(
+                message_id,
+                self.index,
+                content_changed,
+                stream_changed,
+                current_topic,
+                old_topic,
+            )
 
         # Update the rendered content, if the message is indexed
         if "rendered_content" in event and indexed_message: