test(react-query/QueryResetErrorBoundary): use precise time in 'advanceTimersByTimeAsync', and add 'expect' using 'toBeInTheDocument'

Author: sukvvon

packages/react-query/src/__tests__/QueryResetErrorBoundary.test.tsx

@@ -471,7 +471,7 @@ describe('QueryErrorResetBoundary', () => {
       shouldReset = false
       fireEvent.click(rendered.getByText('retry'))
       await vi.advanceTimersByTimeAsync(11)
-      rendered.getByText('error boundary')
+      expect(rendered.getByText('error boundary')).toBeInTheDocument()
       consoleMock.mockRestore()
     })
 
@@ -523,15 +523,15 @@ describe('QueryErrorResetBoundary', () => {
       )
 
       await vi.advanceTimersByTimeAsync(11)
-      rendered.getByText('error boundary')
-      rendered.getByText('retry')
+      expect(rendered.getByText('error boundary')).toBeInTheDocument()
+      expect(rendered.getByText('retry')).toBeInTheDocument()
       fireEvent.click(rendered.getByText('retry'))
       await vi.advanceTimersByTimeAsync(11)
-      rendered.getByText('error boundary')
-      rendered.getByText('retry')
+      expect(rendered.getByText('error boundary')).toBeInTheDocument()
+      expect(rendered.getByText('retry')).toBeInTheDocument()
       fireEvent.click(rendered.getByText('retry'))
       await vi.advanceTimersByTimeAsync(11)
-      rendered.getByText('error boundary')
+      expect(rendered.getByText('error boundary')).toBeInTheDocument()
       expect(fetchCount).toBe(3)
       consoleMock.mockRestore()
     })
@@ -590,16 +590,16 @@ describe('QueryErrorResetBoundary', () => {
         </QueryErrorResetBoundary>,
       )
 
-      await act(() => vi.advanceTimersByTimeAsync(11))
-      rendered.getByText('error boundary')
-      rendered.getByText('retry')
+      await act(() => vi.advanceTimersByTimeAsync(10))
+      expect(rendered.getByText('error boundary')).toBeInTheDocument()
+      expect(rendered.getByText('retry')).toBeInTheDocument()
       fireEvent.click(rendered.getByText('retry'))
-      await act(() => vi.advanceTimersByTimeAsync(11))
-      rendered.getByText('error boundary')
-      rendered.getByText('retry')
+      await act(() => vi.advanceTimersByTimeAsync(10))
+      expect(rendered.getByText('error boundary')).toBeInTheDocument()
+      expect(rendered.getByText('retry')).toBeInTheDocument()
       fireEvent.click(rendered.getByText('retry'))
-      await act(() => vi.advanceTimersByTimeAsync(11))
-      rendered.getByText('data')
+      await act(() => vi.advanceTimersByTimeAsync(10))
+      expect(rendered.getByText('data')).toBeInTheDocument()
       expect(fetchCount).toBe(3)
       expect(renders).toBe(1)
       consoleMock.mockRestore()
@@ -813,12 +813,12 @@ describe('QueryErrorResetBoundary', () => {
         </QueryErrorResetBoundary>,
       )
 
-      await act(() => vi.advanceTimersByTimeAsync(11))
+      await act(() => vi.advanceTimersByTimeAsync(10))
       expect(rendered.getByText('error boundary')).toBeInTheDocument()
       expect(rendered.getByText('retry')).toBeInTheDocument()
       succeed = true
       fireEvent.click(rendered.getByText('retry'))
-      await act(() => vi.advanceTimersByTimeAsync(11))
+      await act(() => vi.advanceTimersByTimeAsync(10))
       expect(rendered.getByText('data')).toBeInTheDocument()
       consoleMock.mockRestore()
     })