Uh oh!
There was an error while loading. Please reload this page.
test({react,preact}-query/usePrefetchQuery): assert the 'Loading...' fallback is rendered before the prefetch resolves - #11075
Conversation
View your CI Pipeline Execution ↗ for commit 78dbec4
☁️ 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 (2)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughThe React and Preact ChangesPrefetch fallback tests
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
…fallback is rendered before the prefetch resolves
78dbec4 to
092776cComparesize-limit report 📦
|
Uh oh!
There was an error while loading. Please reload this page.
🎯 Changes
Adds an assertion right after the initial render that the Suspense
'Loading...'fallback is shown while the prefetched query is still in flight, inshould prefetch query if query state does not existfor bothreact-queryandpreact-query. The same file'sshould not create a suspense waterfall if prefetch is firedtest already follows this convention (checkingfetchStatus/fallback text before advancing timers); this test was missing it.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit