let get_serialized_messages manage topic_head post

gip-inclusion · Nov 27, 2024 · 89d4bba · 89d4bba
1 parent c4ad74f
commit 89d4bba
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/lacommunaute/notification/tests/tests_utils.py b/lacommunaute/notification/tests/tests_utils.py
@@ -51,6 +51,18 @@ def test_order_by_date_joined(self):
 
 
 class TestGetSerializedMessages:
+    def test_post_is_topic_head(self, db):
+        topic = TopicFactory(with_post=True)
+        notifications = [NotificationFactory(post=topic.first_post)]
+        assert get_serialized_messages(notifications) == [
+            {
+                "poster": topic.first_post.poster_display_name,
+                "action": "a posé une nouvelle question",
+                "forum": topic.forum.name,
+                "url": topic.get_absolute_url(with_fqdn=True),
+            }
+        ]
+
     def test_post_is_not_topic_head(self, db):
         post = PostFactory(topic=TopicFactory(with_post=True))
         notifications = [NotificationFactory(post=post)]

diff --git a/lacommunaute/notification/utils.py b/lacommunaute/notification/utils.py
@@ -21,7 +21,7 @@ def get_serialized_messages(notifications):
     return [
         {
             "poster": n.post.poster_display_name,
-            "action": f"a répondu à '{n.post.subject}'",
+            "action": "a posé une nouvelle question" if n.post.is_topic_head else f"a répondu à '{n.post.subject}'",
             "forum": n.post.topic.forum.name,
             "url": n.post.topic.get_absolute_url(with_fqdn=True),
         }