Skip to content

Commit 6d0a2a9

Browse files
authored
test(react-query/useSuspenseQuery): remove unnecessary 'act' from background refetch, 'refetchInterval', and post-unmount assertions (#10212)
1 parent 9ec0fc6 commit 6d0a2a9

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ describe('useSuspenseQuery', () => {
200200

201201
fireEvent.click(rendered.getByLabelText('toggle'))
202202
expect(rendered.queryByText('loading')).not.toBeInTheDocument()
203-
await act(() => vi.advanceTimersByTimeAsync(10))
203+
await vi.advanceTimersByTimeAsync(10)
204204
expect(rendered.queryByText('rendered')).not.toBeInTheDocument()
205205
expect(queryCache.find({ queryKey: key })?.getObserversCount()).toBe(0)
206206
})
@@ -811,7 +811,7 @@ describe('useSuspenseQuery', () => {
811811
// refetch
812812
fireEvent.click(rendered.getByRole('button', { name: 'refetch' }))
813813
// we are now in error state but still have data to show
814-
await act(() => vi.advanceTimersByTimeAsync(11))
814+
await vi.advanceTimersByTimeAsync(11)
815815
expect(rendered.getByText('rendered data error')).toBeInTheDocument()
816816

817817
consoleMock.mockRestore()
@@ -922,9 +922,9 @@ describe('useSuspenseQuery', () => {
922922
expect(rendered.getByText('loading')).toBeInTheDocument()
923923
await act(() => vi.advanceTimersByTimeAsync(10))
924924
expect(rendered.getByText('count: 1')).toBeInTheDocument()
925-
await act(() => vi.advanceTimersByTimeAsync(21))
925+
await vi.advanceTimersByTimeAsync(21)
926926
expect(rendered.getByText('count: 2')).toBeInTheDocument()
927-
await act(() => vi.advanceTimersByTimeAsync(21))
927+
await vi.advanceTimersByTimeAsync(21)
928928
expect(rendered.getByText('count: 3')).toBeInTheDocument()
929929

930930
expect(count).toBeGreaterThanOrEqual(3)

0 commit comments

Comments
 (0)