fix: regression in delay notification state after tailwind animations

gitify-app · Jun 14, 2024 · 9dd817b · 9dd817b
1 parent 13c57df
commit 9dd817b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/src/hooks/useNotifications.test.ts b/src/hooks/useNotifications.test.ts
@@ -2,6 +2,7 @@ import { act, renderHook, waitFor } from '@testing-library/react';
 import axios, { AxiosError } from 'axios';
 import nock from 'nock';
 
+import { mockSingleAccountNotifications } from '../__mocks__/notifications-mocks';
 import {
   mockAuth,
   mockGitHubCloudAccount,
@@ -299,6 +300,10 @@ describe('hooks/useNotifications.ts', () => {
 
   describe('removeNotificationFromState', () => {
     it('should remove a notification from state', async () => {
+      const mockElement = document.createElement('div');
+      mockElement.id = mockSingleAccountNotifications[0].notifications[0].id;
+      jest.spyOn(document, 'getElementById').mockReturnValue(mockElement);
+
       const notifications = [
         { id: 1, title: 'This is a notification.' },
         { id: 2, title: 'This is another one.' },

diff --git a/src/utils/remove-notification.test.ts b/src/utils/remove-notification.test.ts
@@ -6,6 +6,10 @@ import { removeNotification } from './remove-notification';
 
 describe('utils/remove-notification.ts', () => {
   it('should remove a notification if it exists', () => {
+    const mockElement = document.createElement('div');
+    mockElement.id = mockSingleAccountNotifications[0].notifications[0].id;
+    jest.spyOn(document, 'getElementById').mockReturnValue(mockElement);
+
     expect(mockSingleAccountNotifications[0].notifications.length).toBe(1);
 
     const result = removeNotification(