non-deterministic tests fix

Author: chpy04

src/frontend/src/tests/hooks/ChangeRequests.hooks.test.tsx

@@ -20,7 +20,7 @@ describe('change request hooks', () => {
     mockedGetAllChangeRequests.mockReturnValue(mockPromiseAxiosResponse<ChangeRequest[]>(exampleAllChangeRequests));
 
     const { result } = renderHook(() => useAllChangeRequests(), { wrapper });
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleAllChangeRequests);
   });
 
@@ -29,7 +29,7 @@ describe('change request hooks', () => {
     mockedGetSingleChangeRequest.mockReturnValue(mockPromiseAxiosResponse<ChangeRequest>(exampleStageGateChangeRequest));
 
     const { result } = renderHook(() => useSingleChangeRequest('1'), { wrapper });
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleStageGateChangeRequest);
   });
 });

src/frontend/src/tests/hooks/Projects.hooks.test.tsx

@@ -21,7 +21,7 @@ describe('project hooks', () => {
     mockedGetAllProjects.mockReturnValue(mockPromiseAxiosResponse<Project[]>(exampleAllProjects));
 
     const { result } = renderHook(() => useAllProjectsGantt(), { wrapper });
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleAllProjects);
   });
 
@@ -30,7 +30,7 @@ describe('project hooks', () => {
     mockedGetSingleProject.mockReturnValue(mockPromiseAxiosResponse<Project>(exampleProject1));
 
     const { result } = renderHook(() => useSingleProject(exampleWbsProject1), { wrapper });
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleProject1);
   });
 });

src/frontend/src/tests/hooks/Users.hooks.test.tsx

@@ -22,7 +22,7 @@ describe('user hooks', () => {
     mockedGetAllOrgUsers.mockReturnValue(mockPromiseAxiosResponse<User[]>(exampleAllUsers));
 
     const { result } = renderHook(() => useAllUsers(), { wrapper });
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleAllUsers);
   });
 
@@ -31,7 +31,7 @@ describe('user hooks', () => {
     mockedGetSingleUser.mockReturnValue(mockPromiseAxiosResponse<User>(exampleAdminUser));
 
     const { result } = renderHook(() => useSingleUser('1'), { wrapper });
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleAdminUser);
   });
 
@@ -47,7 +47,7 @@ describe('user hooks', () => {
       result.current.mutate(exampleAdminUser.email);
     });
 
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleAdminUser);
   });
 });

src/frontend/src/tests/hooks/WorkPackages.hooks.test.tsx

@@ -21,7 +21,7 @@ describe('work package hooks', () => {
     mockedGetAllWorkPackages.mockReturnValue(mockPromiseAxiosResponse<WorkPackage[]>(exampleAllWorkPackages));
 
     const { result } = renderHook(() => useAllWorkPackages(), { wrapper });
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleAllWorkPackages);
   });
 
@@ -32,7 +32,7 @@ describe('work package hooks', () => {
     const { result } = renderHook(() => useSingleWorkPackage(exampleWbsWorkPackage1), {
       wrapper
     });
-    await waitFor(() => result.current.isSuccess);
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
     expect(result.current.data).toEqual(exampleResearchWorkPackage);
   });
 });