test: update test262#20841
Conversation
|
This PR is packaged and the instant preview is available (a553f61). Install it locally:
npm i -D webpack@https://pkg.pr.new/webpack@a553f61
yarn add -D webpack@https://pkg.pr.new/webpack@a553f61
pnpm add -D webpack@https://pkg.pr.new/webpack@a553f61 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #20841 +/- ##
=======================================
Coverage 91.43% 91.43%
=======================================
Files 562 562
Lines 55438 55438
Branches 14636 14636
=======================================
Hits 50688 50688
Misses 4750 4750
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
There was a problem hiding this comment.
Pull request overview
Updates the local Test262 spec test runner configuration to account for newly added/updated upstream Test262 cases that currently fail under V8/this harness.
Changes:
- Add
module-code/namespace/internals/super-access-to-tdz-binding.jsto theknownV8Bugsallowlist (ignored when it errors). - Add three
import/import-defer/evaluation-triggers/*super-property-set-exported*.jscases to theknownBugsskip list.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merging this PR will degrade performance by 21.18%
Performance Changes
Comparing Footnotes
|
Summary
test - update 262, these tests are not supported by V8 currently
What kind of change does this PR introduce?
test
Did you add tests for your changes?
Yes
Does this PR introduce a breaking change?
No
If relevant, what needs to be documented once your changes are merged or what have you already documented?
Nothing
Use of AI
No