chore: fix test warnings (TanStack#2070)

* chore: fix eslint warnings by including refetch in the dependency array

the refetch function is stable

* chore: mockConsoleError for tests that expect errors to be thrown

to avoid error output on the console during tests

Author: TkDodo

src/core/tests/queryObserver.test.tsx

@@ -241,6 +241,7 @@ describe('queryObserver', () => {
   })
 
   test('should always run the selector again if selector throws an error', async () => {
+    const consoleMock = mockConsoleError()
     const key = queryKey()
     const results: QueryObserverResult[] = []
     const select = () => {
@@ -284,6 +285,7 @@ describe('queryObserver', () => {
       isFetching: false,
       data: undefined,
     })
+    consoleMock.mockRestore()
   })
 
   test('should structurally share the selector', async () => {

src/react/tests/useQuery.test.tsx

@@ -3611,14 +3611,16 @@ describe('useQuery', () => {
 
       states.push(state)
 
+      const { refetch } = state
+
       React.useEffect(() => {
         setActTimeout(() => {
-          state.refetch()
+          refetch()
         }, 0)
         setActTimeout(() => {
           queryClient.resetQueries(key)
         }, 50)
-      }, [])
+      }, [refetch])
 
       return null
     }