test(vue-query/useQueries): assert the other query is not refetched when refetching a single query - #11608
Conversation
…hen refetching a single query
|
View your CI Pipeline Execution ↗ for commit d17fc1e
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: TanStack/query/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe Vue Query test now checks that refetching the first query does not leave the second query in a fetching state. ChangesRefetch isolation
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Merge Risk: ⚪ Minimal · up to The test now detects an unintended refetch of the second query. No actionable merge-blocking risk is established. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
size-limit report 📦
|
🎯 Changes
In
should refetch only the specific query without affecting others, the second query'squeryFntakes20ms while only10ms elapse afterrefetch(), so itsdatastays'posts-1'even if it were refetched.Adds
expect(queriesState.value[1].isFetching).toBe(false)so the test fails if refetching the first query also refetches the second.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit