test(svelte-query): replace 'async/await sleep' with 'sleep().then()' in test 'queryFn' and 'mutationFn' by sukvvon · Pull Request #10397 · TanStack/query

sukvvon · 2026-04-06T01:32:37Z

🎯 Changes

Replace async () => { await sleep(N); return value } with () => sleep(N).then(() => value) in test queryFn and mutationFn
Applied to createQuery.svelte.test.ts, useIsFetching/Query.svelte, and useIsMutating/Query.svelte

✅ Checklist

I have followed the steps in the Contributing guide.
I have tested this code locally with pnpm run test:pr.

🚀 Release Impact

This change affects published code, and I have generated a changeset.
This change is docs/CI/dev-only (no release).

Summary by CodeRabbit

Tests
- Refactored test implementations to use promise chaining patterns while maintaining test behavior and assertions.

… in test 'queryFn' and 'mutationFn'

coderabbitai · 2026-04-06T01:32:55Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d7c57dd3-b128-4b31-9ed7-524e1d692928

📥 Commits

Reviewing files that changed from the base of the PR and between e76d476 and a9c3c4b.

📒 Files selected for processing (3)

packages/svelte-query/tests/createQuery.svelte.test.ts
packages/svelte-query/tests/useIsFetching/Query.svelte
packages/svelte-query/tests/useIsMutating/Query.svelte

📝 Walkthrough

Walkthrough

This PR refactors async function implementations across three test files, replacing async/await syntax with promise chain patterns using .then(). All test logic, assertions, and timing calls remain unchanged. Net reduction of approximately 12 lines.

Changes

Cohort / File(s)	Summary
Test Function Refactoring `packages/svelte-query/tests/createQuery.svelte.test.ts`, `packages/svelte-query/tests/useIsFetching/Query.svelte`, `packages/svelte-query/tests/useIsMutating/Query.svelte`	Converted `async`/`await` function syntax to promise chains using `.then()` for `queryFn` and `mutationFn` implementations. No changes to test logic or control flow assertions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Async awaits hop away,
.then() chains come out to play,
Promise dances, clean and bright,
Tests are sparkling, all feels right! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: replacing async/await sleep patterns with promise-chain .then() in test functions across specified Svelte Query test files.
Description check	✅ Passed	The description follows the template structure with clear 'Changes' section detailing the refactoring, completed checklist items, and proper release impact designation as dev-only.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch test/svelte-query-replace-async-await-sleep-with-then-chaining

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

nx-cloud · 2026-04-06T01:33:07Z

View your CI Pipeline Execution ↗ for commit a9c3c4b

Command	Status	Duration	Result
`nx affected --targets=test:sherif,test:knip,tes...`	✅ Succeeded	1m 57s	View ↗
`nx run-many --target=build --exclude=examples/*...`	✅ Succeeded	<1s	View ↗

☁️ Nx Cloud last updated this comment at 2026-04-06 01:35:44 UTC

github-actions · 2026-04-06T01:33:31Z

🚀 Changeset Version Preview

No changeset entries found. Merging this PR will not cause a version bump for any packages.

pkg-pr-new · 2026-04-06T01:34:04Z

More templates

@tanstack/angular-query-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-experimental@10397

@tanstack/eslint-plugin-query

npm i https://pkg.pr.new/@tanstack/eslint-plugin-query@10397

@tanstack/preact-query

npm i https://pkg.pr.new/@tanstack/preact-query@10397

@tanstack/preact-query-devtools

npm i https://pkg.pr.new/@tanstack/preact-query-devtools@10397

@tanstack/preact-query-persist-client

npm i https://pkg.pr.new/@tanstack/preact-query-persist-client@10397

@tanstack/query-async-storage-persister

npm i https://pkg.pr.new/@tanstack/query-async-storage-persister@10397

@tanstack/query-broadcast-client-experimental

npm i https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@10397

@tanstack/query-core

npm i https://pkg.pr.new/@tanstack/query-core@10397

@tanstack/query-devtools

npm i https://pkg.pr.new/@tanstack/query-devtools@10397

@tanstack/query-persist-client-core

npm i https://pkg.pr.new/@tanstack/query-persist-client-core@10397

@tanstack/query-sync-storage-persister

npm i https://pkg.pr.new/@tanstack/query-sync-storage-persister@10397

@tanstack/react-query

npm i https://pkg.pr.new/@tanstack/react-query@10397

@tanstack/react-query-devtools

npm i https://pkg.pr.new/@tanstack/react-query-devtools@10397

@tanstack/react-query-next-experimental

npm i https://pkg.pr.new/@tanstack/react-query-next-experimental@10397

@tanstack/react-query-persist-client

npm i https://pkg.pr.new/@tanstack/react-query-persist-client@10397

@tanstack/solid-query

npm i https://pkg.pr.new/@tanstack/solid-query@10397

@tanstack/solid-query-devtools

npm i https://pkg.pr.new/@tanstack/solid-query-devtools@10397

@tanstack/solid-query-persist-client

npm i https://pkg.pr.new/@tanstack/solid-query-persist-client@10397

@tanstack/svelte-query

npm i https://pkg.pr.new/@tanstack/svelte-query@10397

@tanstack/svelte-query-devtools

npm i https://pkg.pr.new/@tanstack/svelte-query-devtools@10397

@tanstack/svelte-query-persist-client

npm i https://pkg.pr.new/@tanstack/svelte-query-persist-client@10397

@tanstack/vue-query

npm i https://pkg.pr.new/@tanstack/vue-query@10397

@tanstack/vue-query-devtools

npm i https://pkg.pr.new/@tanstack/vue-query-devtools@10397

commit: a9c3c4b

github-actions · 2026-04-06T01:34:23Z

size-limit report 📦

Path	Size
react full	11.98 KB (0%)
react minimal	9.01 KB (0%)

test(svelte-query): replace 'async/await sleep' with 'sleep().then()'…

a9c3c4b

… in test 'queryFn' and 'mutationFn'

github-actions Bot added the package: svelte-query label Apr 6, 2026

sukvvon self-assigned this Apr 6, 2026

sukvvon merged commit abf8547 into main Apr 6, 2026
10 checks passed

sukvvon deleted the test/svelte-query-replace-async-await-sleep-with-then-chaining branch April 6, 2026 01:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

test(svelte-query): replace 'async/await sleep' with 'sleep().then()' in test 'queryFn' and 'mutationFn'#10397

test(svelte-query): replace 'async/await sleep' with 'sleep().then()' in test 'queryFn' and 'mutationFn'#10397
sukvvon merged 1 commit intomainfrom
test/svelte-query-replace-async-await-sleep-with-then-chaining

sukvvon commented Apr 6, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Apr 6, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Poem

Uh oh!

nx-cloud Bot commented Apr 6, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Apr 6, 2026

Uh oh!

pkg-pr-new Bot commented Apr 6, 2026

Uh oh!

github-actions Bot commented Apr 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

sukvvon commented Apr 6, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🎯 Changes

✅ Checklist

🚀 Release Impact

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Apr 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Poem

Uh oh!

nx-cloud Bot commented Apr 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Apr 6, 2026

🚀 Changeset Version Preview

Uh oh!

pkg-pr-new Bot commented Apr 6, 2026

Uh oh!

github-actions Bot commented Apr 6, 2026

size-limit report 📦

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

sukvvon commented Apr 6, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Apr 6, 2026 •

edited

Loading

nx-cloud Bot commented Apr 6, 2026 •

edited

Loading