Uh oh!
There was an error while loading. Please reload this page.
test({react,preact,solid}-query/useInfiniteQuery): inline the shared 'fetchItems' helper into the tests that use it - #11082
Conversation
…'fetchItems' helper into the tests that use it
View your CI Pipeline Execution ↗ for commit 2856b6c
☁️ 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. |
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe Preact, React, and Solid infinite-query tests remove shared ChangesInfinite-query test updates
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
size-limit report 📦
|
🎯 Changes
Inlines the
fetchItemshelper (a plain async mock queryFn generator, defined at the top of each file but used in only 1–2 tests) directly into theitblock(s) that use it, matching the dominant pattern in these files where each test defines its ownqueryFninline. The 4th (noPrev) parameter is dropped since no call site ever passed it.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit