test(solid-query/useQuery): add test for 'isRestoring' transition from true to false triggering refetch#10168
Conversation
…m true to false triggering refetch
|
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughWalkthroughA new Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit 8e8b776
☁️ Nx Cloud last updated this comment at |
More templates
@tanstack/angular-query-experimental
@tanstack/eslint-plugin-query
@tanstack/preact-query
@tanstack/query-async-storage-persister
@tanstack/query-broadcast-client-experimental
@tanstack/query-core
@tanstack/query-devtools
@tanstack/query-persist-client-core
@tanstack/query-sync-storage-persister
@tanstack/react-query
@tanstack/react-query-devtools
@tanstack/react-query-next-experimental
@tanstack/react-query-persist-client
@tanstack/solid-query
@tanstack/solid-query-devtools
@tanstack/solid-query-persist-client
@tanstack/svelte-query
@tanstack/svelte-query-devtools
@tanstack/svelte-query-persist-client
@tanstack/vue-query
@tanstack/vue-query-devtools
commit: |
🎯 Changes
Add a test to cover the
isRestoringsignal transition (true→false) triggering a refetch inuseBaseQuery.ts(lines 338-339).The test verifies that:
isRestoringistrue, the query stays inpendingstatus withidlefetchStatus and does not callqueryFnisRestoringtransitions tofalse, the query refetches and resolves tosuccessThis mirrors the behavior of
PersistQueryClientProviderinsolid-query-persist-client(src/PersistQueryClientProvider.tsx:20,33).✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit
New Features
Tests