Skip to content

test(query-core/{hydration,queriesObserver}): remove unnecessary 'async' from tests without 'await' #11035

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sukvvon merged 1 commit into from
Jul 8, 2026
+5 −5
Merged

test(query-core/{hydration,queriesObserver}): remove unnecessary 'async' from tests without 'await' #11035

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions packages/query-core/src/__tests__/hydration.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1643,7 +1643,7 @@ describe('dehydration and rehydration', () => {
// synchronous thenable resolution must also produce status: 'success'.
// Previously the if (query) branch would spread status: 'pending' from the
// server state without correcting it for the resolved data.
it('should set status to success when rehydrating an existing pending query with a synchronously resolved promise', async () => {
it('should set status to success when rehydrating an existing pending query with a synchronously resolved promise', () => {
const key = queryKey()
// --- server ---

Expand Down Expand Up @@ -1811,7 +1811,7 @@ describe('dehydration and rehydration', () => {
serverQueryClient.clear()
})

it('should set dataUpdatedAt when hydrating a resolved streamed query into a new cache entry', async () => {
it('should set dataUpdatedAt when hydrating a resolved streamed query into a new cache entry', () => {
const key = queryKey()

// --- server ---
Expand Down Expand Up @@ -1856,7 +1856,7 @@ describe('dehydration and rehydration', () => {
serverQueryClient.clear()
})

it('should set dataUpdatedAt when hydrating a resolved streamed query into an existing cache entry', async () => {
it('should set dataUpdatedAt when hydrating a resolved streamed query into an existing cache entry', () => {
const key = queryKey()

// --- server ---
Expand Down
4 changes: 2 additions & 2 deletions packages/query-core/src/__tests__/queriesObserver.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -473,7 +473,7 @@ describe('queriesObserver', () => {
expect(newCombined.count).toBe(2)
})

it('should skip combine notifications while suspense queries have no data', async () => {
it('should skip combine notifications while suspense queries have no data', () => {
const key = queryKey()
const combine = vi.fn((results: Array<QueryObserverResult>) =>
results.map((result) => result.data),
Expand Down Expand Up @@ -506,7 +506,7 @@ describe('queriesObserver', () => {
unsubscribe()
})

it('should skip combine notifications after suspense is enabled without structural changes', async () => {
it('should skip combine notifications after suspense is enabled without structural changes', () => {
const key = queryKey()
const combine = vi.fn((results: Array<QueryObserverResult>) =>
results.map((result) => result.data),
Expand Down
Loading