Skip to content

Commit bfcff0b

Browse files
sukvvonmanudeli
andauthored
test(react-query/ssr): replace 'advanceTimersByTimeAsync' with 'sleep' in 'queryFn', remove 'await' from 'fetchQuery' and related methods (#9443)
Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com>
1 parent 4daf694 commit bfcff0b

File tree

1 file changed

+17
-8
lines changed

1 file changed

+17
-8
lines changed

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

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import * as React from 'react'
22
import { renderToString } from 'react-dom/server'
33
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
4-
import { queryKey } from '@tanstack/query-test-utils'
4+
import { queryKey, sleep } from '@tanstack/query-test-utils'
55
import {
66
QueryCache,
77
QueryClient,
@@ -56,11 +56,18 @@ describe('Server Side Rendering', () => {
5656

5757
it('should add prefetched data to cache', async () => {
5858
const key = queryKey()
59-
const fetchFn = () => Promise.resolve('data')
60-
const data = await queryClient.fetchQuery({
59+
60+
const promise = queryClient.fetchQuery({
6161
queryKey: key,
62-
queryFn: fetchFn,
62+
queryFn: async () => {
63+
await sleep(10)
64+
return 'data'
65+
},
6366
})
67+
await vi.advanceTimersByTimeAsync(10)
68+
69+
const data = await promise
70+
6471
expect(data).toBe('data')
6572
expect(queryCache.find({ queryKey: key })?.state.data).toBe('data')
6673
queryCache.clear()
@@ -69,7 +76,7 @@ describe('Server Side Rendering', () => {
6976
it('should return existing data from the cache', async () => {
7077
const key = queryKey()
7178
const queryFn = vi.fn(async () => {
72-
await vi.advanceTimersByTimeAsync(10)
79+
await sleep(10)
7380
return 'data'
7481
})
7582

@@ -85,7 +92,8 @@ describe('Server Side Rendering', () => {
8592
)
8693
}
8794

88-
await queryClient.prefetchQuery({ queryKey: key, queryFn })
95+
queryClient.prefetchQuery({ queryKey: key, queryFn })
96+
await vi.advanceTimersByTimeAsync(10)
8997

9098
const markup = renderToString(
9199
<QueryClientProvider client={queryClient}>
@@ -132,7 +140,7 @@ describe('Server Side Rendering', () => {
132140
it('useInfiniteQuery should return the correct state', async () => {
133141
const key = queryKey()
134142
const queryFn = vi.fn(async () => {
135-
await vi.advanceTimersByTimeAsync(5)
143+
await sleep(5)
136144
return 'page 1'
137145
})
138146

@@ -148,11 +156,12 @@ describe('Server Side Rendering', () => {
148156
)
149157
}
150158

151-
await queryClient.prefetchInfiniteQuery({
159+
queryClient.prefetchInfiniteQuery({
152160
queryKey: key,
153161
queryFn,
154162
initialPageParam: 0,
155163
})
164+
await vi.advanceTimersByTimeAsync(5)
156165

157166
const markup = renderToString(
158167
<QueryClientProvider client={queryClient}>

0 commit comments

Comments
 (0)