test(vue-query/useQueries): add test for outside scope warning in development mode#10240
test(vue-query/useQueries): add test for outside scope warning in development mode#10240sukvvon wants to merge 1 commit intoTanStack:mainfrom
Conversation
|
|
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 (1)
📝 WalkthroughWalkthroughAdds two test cases to the useQueries test suite that validate warning behavior in development mode. Both tests verify that a warning is emitted when useQueries is invoked outside a valid setup function or running effect scope. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 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 99f2a2f
☁️ Nx Cloud last updated this comment at |
🎯 Changes
Add test for
useQueriesoutside scope warning in development mode, coveringuseQueries.tslines 254-259.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit
useQueriesoutside of proper execution scopes in development mode.