test: try to stabilize tests

Author: TkDodo

src/core/tests/queryObserver.test.tsx

@@ -249,7 +249,10 @@ describe('queryObserver', () => {
   test('should always run the selector again if selector throws an error and selector is not referentially stable', async () => {
     const key = queryKey()
     const results: QueryObserverResult[] = []
-    const queryFn = () => ({ count: 1 })
+    const queryFn = async () => {
+      await sleep(10)
+      return { count: 1 }
+    }
     const observer = new QueryObserver(queryClient, {
       queryKey: key,
       queryFn,
@@ -260,10 +263,10 @@ describe('queryObserver', () => {
     const unsubscribe = observer.subscribe(result => {
       results.push(result)
     })
-    await sleep(1)
+    await sleep(50)
     await observer.refetch()
     unsubscribe()
-    expect(results.length).toBe(4)
+    expect(results.length).toBe(5)
     expect(results[0]).toMatchObject({
       status: 'loading',
       isFetching: true,
@@ -284,6 +287,11 @@ describe('queryObserver', () => {
       isFetching: false,
       data: undefined,
     })
+    expect(results[4]).toMatchObject({
+      status: 'error',
+      isFetching: false,
+      data: undefined,
+    })
   })
 
   test('should return stale data if selector throws an error', async () => {
@@ -293,7 +301,11 @@ describe('queryObserver', () => {
     const error = new Error('select error')
     const observer = new QueryObserver(queryClient, {
       queryKey: key,
-      queryFn: () => (shouldError ? 2 : 1),
+      retry: 0,
+      queryFn: async () => {
+        await sleep(10)
+        return shouldError ? 2 : 1
+      },
       select: num => {
         if (shouldError) {
           throw error
@@ -306,11 +318,11 @@ describe('queryObserver', () => {
     const unsubscribe = observer.subscribe(result => {
       results.push(result)
     })
-    await sleep(10)
+    await sleep(50)
     await observer.refetch()
     unsubscribe()
 
-    expect(results.length).toBe(4)
+    expect(results.length).toBe(5)
     expect(results[0]).toMatchObject({
       status: 'loading',
       isFetching: true,
@@ -335,6 +347,12 @@ describe('queryObserver', () => {
       data: '1',
       error,
     })
+    expect(results[4]).toMatchObject({
+      status: 'error',
+      isFetching: false,
+      data: '1',
+      error,
+    })
   })
 
   test('should structurally share the selector', async () => {