Updating unreadcount only on message.new event

Author: vishalnarkhede

src/components/ChannelPreview/ChannelPreview.js

@@ -14,7 +14,10 @@ const ChannelPreview = (props) => {
   useEffect(() => {
     const handleEvent = (e) => {
       setLastMessage(e.message);
-      setUnread(channel.countUnread());
+
+      if (e.type === 'message.new') {
+        setUnread(channel.countUnread());
+      }
     };
 
     channel.on('message.new', handleEvent);

src/components/ChannelPreview/__tests__/ChannelPreview.test.js

@@ -97,14 +97,15 @@ describe('ChannelPreview', () => {
       expect(getNodeText(getByTestId('unread-count'))).toBe('0');
     });
   });
+
   const eventCases = [
     ['message.new', dispatchMessageNewEvent],
     ['message.updated', dispatchMessageUpdatedEvent],
     ['message.deleted', dispatchMessageDeletedEvent],
   ];
 
   it.each(eventCases)(
-    'should update the last event message & unreadCount on %s event',
+    'should update the last event message',
     async (eventType, dispatcher) => {
       const c = generateChannel();
       await initializeChannel(c);
@@ -117,8 +118,6 @@ describe('ChannelPreview', () => {
         user: clientUser,
       });
 
-      channel.countUnread = () => 10;
-
       act(() => {
         dispatcher(chatClient, message, channel);
       });
@@ -127,8 +126,30 @@ describe('ChannelPreview', () => {
         expect(getNodeText(getByTestId('last-event-message'))).toBe(
           message.text,
         );
-        expect(getNodeText(getByTestId('unread-count'))).toBe('10');
       });
     },
   );
+
+  it('should update the unread count on "message.new" event', async () => {
+    const c = generateChannel();
+    await initializeChannel(c);
+
+    const { getByTestId } = render(getComponent());
+
+    await waitForElement(() => getByTestId('channel-id'));
+
+    const message = generateMessage({
+      user: clientUser,
+    });
+
+    channel.countUnread = () => 10;
+
+    act(() => {
+      dispatchMessageNewEvent(chatClient, message, channel);
+    });
+
+    await wait(() => {
+      expect(getNodeText(getByTestId('unread-count'))).toBe('10');
+    });
+  });
 });