fix existing tests

reduxjs · Apr 7, 2022 · 776e24f · 776e24f
1 parent 169d359
commit 776e24f
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 9 deletions.
diff --git a/packages/toolkit/src/query/tests/buildHooks.test.tsx b/packages/toolkit/src/query/tests/buildHooks.test.tsx
@@ -195,7 +195,7 @@ describe('hooks tests', () => {
         expect(screen.getByTestId('isLoading').textContent).toBe('false')
       )
       // We call a refetch, should still be `false`
-      act(() => refetch())
+      act(() => void refetch())
       await waitFor(() =>
         expect(screen.getByTestId('isFetching').textContent).toBe('true')
       )
@@ -251,7 +251,7 @@ describe('hooks tests', () => {
       expect(getRenderCount()).toBe(5)
 
       // We call a refetch, should set `isFetching` to true, then false when complete/errored
-      act(() => refetchMe())
+      act(() => void refetchMe())
       await waitFor(() => {
         expect(screen.getByTestId('isLoading').textContent).toBe('false')
         expect(screen.getByTestId('isFetching').textContent).toBe('true')

diff --git a/packages/toolkit/src/query/tests/errorHandling.test.tsx b/packages/toolkit/src/query/tests/errorHandling.test.tsx
@@ -145,7 +145,7 @@ describe('query error handling', () => {
       )
     )
 
-    act(result.current.refetch)
+    act(() => void result.current.refetch())
 
     await hookWaitFor(() => expect(result.current.isFetching).toBeFalsy())
     expect(result.current).toEqual(
@@ -191,7 +191,7 @@ describe('query error handling', () => {
       })
     )
 
-    act(result.current.refetch)
+    act(() => void result.current.refetch())
 
     await hookWaitFor(() => expect(result.current.isFetching).toBeFalsy())
     expect(result.current).toEqual(

diff --git a/packages/toolkit/src/query/tests/unionTypes.test.ts b/packages/toolkit/src/query/tests/unionTypes.test.ts
@@ -343,7 +343,7 @@ describe.skip('TS only tests', () => {
   })
   // pre41-remove-end
 
-  test('queryHookResult (without selector) union', () => {
+  test('queryHookResult (without selector) union', async () => {
     const useQueryStateResult = api.endpoints.test.useQueryState()
     const useQueryResult = api.endpoints.test.useQuery()
     const useQueryStateWithSelectFromResult = api.endpoints.test.useQueryState(
@@ -353,12 +353,15 @@ describe.skip('TS only tests', () => {
       }
     )
 
-    const { refetch: _omit, ...useQueryResultWithoutMethods } = useQueryResult
+    const { refetch, ...useQueryResultWithoutMethods } = useQueryResult
     expectExactType(useQueryStateResult)(useQueryResultWithoutMethods)
     expectExactType(useQueryStateWithSelectFromResult)(
       // @ts-expect-error
       useQueryResultWithoutMethods
     )
+    expectType<ReturnType<ReturnType<typeof api.endpoints.test.select>>>(
+      await refetch()
+    )
   })
 
   test('useQueryState (with selectFromResult)', () => {
@@ -391,8 +394,8 @@ describe.skip('TS only tests', () => {
     })(result)
   })
 
-  test('useQuery (with selectFromResult)', () => {
-    const result = api.endpoints.test.useQuery(undefined, {
+  test('useQuery (with selectFromResult)', async () => {
+    const { refetch, ...result } = api.endpoints.test.useQuery(undefined, {
       selectFromResult({
         data,
         isLoading,
@@ -418,8 +421,11 @@ describe.skip('TS only tests', () => {
       isFetching: true,
       isSuccess: false,
       isError: false,
-      refetch: () => {},
     })(result)
+
+    expectType<ReturnType<ReturnType<typeof api.endpoints.test.select>>>(
+      await refetch()
+    )
   })
 
   test('useMutation union', () => {