test(lit-query/{useIsFetching,useIsMutating}): split filter tests into static filters and filters updated on host updates - #11763
Conversation
…o static filters and filters updated on host updates
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
|
View your CI Pipeline Execution ↗ for commit 388ec76
☁️ Nx Cloud last updated this comment at |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⚙️ Run configurationConfiguration used: Repository: TanStack/query/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 (2)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review. 📝 WalkthroughWalkthroughThe lit-query tests now separate static query and mutation filter counts from reactive filter updates. Reactive-filter tests retain assertions for filtered counts. ChangesLit Query filter count tests
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to This test-only split preserves static count coverage and reactive filter checks without changing runtime behavior. No actionable merge-blocking risk is identified; merge after normal checks pass. 🚥 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
Splits the filter test in
useIsFetching.test.tsanduseIsMutating.test.tsinto two tests each:should be able to filter: with static filters, the filtered count only includes the matching query or mutation as they finish.should apply updated filters on host updates: with filters from a function, the count follows the filters after each host update, from the first key to an unmatched key to the second key.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit