test(form-core): add coverage for Map and Set comparison in evaluate#2009
test(form-core): add coverage for Map and Set comparison in evaluate#2009Kyujenius wants to merge 1 commit intoTanStack:mainfrom
Conversation
Add tests for the evaluate utility function to cover Map and Set comparison logic, including edge cases for equality, empty collections, different sizes, and different values.
|
|
View your CI Pipeline Execution ↗ for commit 0be8209
☁️ Nx Cloud last updated this comment at |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2009 +/- ##
==========================================
- Coverage 90.35% 89.96% -0.39%
==========================================
Files 38 48 +10
Lines 1752 1953 +201
Branches 444 491 +47
==========================================
+ Hits 1583 1757 +174
- Misses 149 179 +30
+ Partials 20 17 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@LeCarbonator |
🎯 Changes
Add test Coverage for the
evaluateutility function - Map and Set comparison logic.Test cases added:
This improves
utils.tsstatement coverage from 90.34% to 97.15%.Why not 100% coverage?
The remaining 3% uncovered lines are intentionally excluded for:
setByedge casedeleteByunreachable errormakePathArrayarray inputThese would be better addressed in separate PRs if needed.
✅ Checklist
pnpm test:pr.🚀 Release Impact