fix: notification read state for delayNotificationState

Signed-off-by: Adam Setch <[email protected]>

Author: setchy

src/renderer/components/notifications/NotificationRow.test.tsx

@@ -84,6 +84,28 @@ describe('renderer/components/notifications/NotificationRow.tsx', () => {
     expect(tree).toMatchSnapshot();
   });
 
+  it('should render itself & its children - notification is read', async () => {
+    jest
+      .spyOn(global.Date, 'now')
+      .mockImplementation(() => new Date('2024').valueOf());
+
+    const props = {
+      notification: {
+        ...mockSingleNotification,
+        unread: false,
+      },
+      account: mockGitHubCloudAccount,
+    };
+
+    const tree = render(
+      <AppContext.Provider value={{ settings: mockSettings }}>
+        <NotificationRow {...props} />
+      </AppContext.Provider>,
+    );
+
+    expect(tree).toMatchSnapshot();
+  });
+
   describe('notification interactions', () => {
     it('should open a notification in the browser - click', async () => {
       const markNotificationsAsRead = jest.fn();

src/renderer/components/notifications/RepositoryNotifications.test.tsx

@@ -39,6 +39,20 @@ describe('renderer/components/notifications/RepositoryNotifications.tsx', () =>
     expect(tree).toMatchSnapshot();
   });
 
+  it('should render itself & its children - all notifications are read', () => {
+    for (const n of props.repoNotifications) {
+      n.unread = false;
+    }
+
+    const tree = render(
+      <AppContext.Provider value={{}}>
+        <RepositoryNotifications {...props} />
+      </AppContext.Provider>,
+    );
+
+    expect(tree).toMatchSnapshot();
+  });
+
   it('should open the browser when clicking on the repo name', async () => {
     const openExternalLinkMock = jest
       .spyOn(comms, 'openExternalLink')