test({react,preact,angular}-query/{queryOptions,infiniteQueryOptions,mutationOptions}): replace 'toStrictEqual' with 'toBe' and add type annotations#10192
Conversation
…mutationOptions}): replace 'toStrictEqual' with 'toBe' for identity function assertion and add explicit type annotations to 'mutationOptions'
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (9)
📝 WalkthroughWalkthroughThis PR updates test assertions across multiple packages to validate object reference identity instead of deep structural equality, and adds explicit type annotations to test objects in mutation option tests. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
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 c6acd98
☁️ Nx Cloud last updated this comment at |
🎯 Changes
toStrictEqualwithtoBefor identity function assertions inqueryOptions,infiniteQueryOptions, andmutationOptionstests acrossreact-query,preact-query, andangular-query-experimentalUseMutationOptions/CreateMutationOptions) tomutationOptionstest objects for consistency withqueryOptionsandinfiniteQueryOptionstestsSince these options functions are identity functions that return the same reference,
toBe(reference equality) is more precise thantoStrictEqual(deep equality). This aligns with the change made in #10135 forsolid-query.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit