From 190416d76549e1b47049a7b49be53bb36c8f5c1b Mon Sep 17 00:00:00 2001
From: Shivam Mishra <scm.mymail@gmail.com>
Date: Thu, 23 Apr 2026 12:55:46 +0530
Subject: [PATCH] test: handle setActiveChat

---
 .../specs/conversations/actions.spec.js       | 22 ++++++++++++-------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/app/javascript/dashboard/store/modules/specs/conversations/actions.spec.js b/app/javascript/dashboard/store/modules/specs/conversations/actions.spec.js
index fa052ec1b..dc2714167 100644
--- a/app/javascript/dashboard/store/modules/specs/conversations/actions.spec.js
+++ b/app/javascript/dashboard/store/modules/specs/conversations/actions.spec.js
@@ -736,16 +736,19 @@ describe('#addMentions', () => {
         [types.CLEAR_ALL_MESSAGES_LOADED, 42],
         [types.SET_CHAT_DATA_FETCHED, 42],
       ]);
-      expect(localDispatch).toHaveBeenCalledWith('fetchPreviousMessages', {
-        after: 99,
-        before: 100,
-        conversationId: 42,
-      });
+      expect(localDispatch.mock.calls).toEqual([
+        [
+          'fetchPreviousMessages',
+          { after: 99, before: 100, conversationId: 42 },
+        ],
+        ['setConversationLastMessageId', { conversationId: 42 }],
+        ['syncActiveConversationMessages', { conversationId: 42 }],
+      ]);
     });
 
-    it('should not dispatch fetchPreviousMessages if dataFetched is already set', async () => {
+    it('should skip fetchPreviousMessages but still sync if dataFetched is already set', async () => {
       const localCommit = vi.fn();
-      const localDispatch = vi.fn();
+      const localDispatch = vi.fn().mockResolvedValue();
       const data = { id: 42, messages: [{ id: 100 }], dataFetched: true };
 
       await actions.setActiveChat(
@@ -757,7 +760,10 @@ describe('#addMentions', () => {
         [types.SET_CURRENT_CHAT_WINDOW, data],
         [types.CLEAR_ALL_MESSAGES_LOADED, 42],
       ]);
-      expect(localDispatch).not.toHaveBeenCalled();
+      expect(localDispatch.mock.calls).toEqual([
+        ['setConversationLastMessageId', { conversationId: 42 }],
+        ['syncActiveConversationMessages', { conversationId: 42 }],
+      ]);
     });
 
     it('should commit SET_CHAT_DATA_FETCHED by ID, not mutate the data object directly (race condition fix)', async () => {