test(query-persist-client-core/createPersister): add test for 'retrieveQuery' returning data without a restore callback#10883
Conversation
…veQuery' returning data without a restore callback
|
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)
📝 WalkthroughWalkthroughThis PR adds a test suite for the ChangesQuery Retrieval Test Coverage
Estimated Code Review Effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit fc2afcb
☁️ 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. |
size-limit report 📦
|
🎯 Changes
Adds a unit test for
retrieveQuery, which is part of the persister's public API but was previously only exercised indirectly throughpersisterFn. Calling it directly without a restore callback returns the persisted data, covering the branch where no after-restore callback is scheduled.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit