test: add edge case coverage for Array.prototype.map#5220
test: add edge case coverage for Array.prototype.map#5220ParthMozarkar wants to merge 4 commits intoboa-dev:mainfrom
Conversation
hansl
left a comment
There was a problem hiding this comment.
We don't need to do all Array functions. Test262 are doing every function in the spec and all edge cases already.
Test262 conformance changes
Tested main commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5220 +/- ##
===========================================
+ Coverage 47.24% 59.79% +12.55%
===========================================
Files 476 582 +106
Lines 46892 63460 +16568
===========================================
+ Hits 22154 37948 +15794
- Misses 24738 25512 +774 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Thanks for the approval and the feedback! Understood I won't add |
I added test cases for Array.prototype.map covering some important
edge cases that weren't tested before:
No existing tests were removed or modified.
Raising this as a draft for review — let me know if any changes
are needed!