test({react,preact}-query/useSuspenseQueries): add test for not suspending when all queries have fresh cached data#10202
Conversation
…nding when all queries have fresh cached data
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughTest cases added to both preact-query and react-query packages verifying that useSuspenseQueries doesn't suspend or trigger refetches when all queries have fresh cached data available. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested labels
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 f139851
☁️ Nx Cloud last updated this comment at |
🎯 Changes
Add a test to verify that
useSuspenseQueriesdoes not suspend when all queries already have fresh cached data.When all queries have cached data within
staleTime(min 1000ms in suspense mode), the component should:queryFnnot called)Added to both
react-queryandpreact-query.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit