test(query-core/queriesObserver): add test for 'getObservers' method#10248
test(query-core/queriesObserver): add test for 'getObservers' method#10248sukvvon wants to merge 2 commits 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)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds a new test case to QueriesObserver tests that creates two queries, initializes a QueriesObserver, subscribes, advances timers, calls getObservers, asserts two QueryObserver instances are returned, then unsubscribes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 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 4787cca
☁️ Nx Cloud last updated this comment at |
e2b4147 to
c1703c9
Compare
🎯 Changes
Add a test for
queriesObserver.tslines 168-169 — thegetObservers()method that returns the internal observers array.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit