-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Port Test262’s RegExp lookbehind tests #37928
Port Test262’s RegExp lookbehind tests #37928
Conversation
There are no reviewers for this pull request. Please reach out on the chat room to get help with this. Thank you! |
7a74216
to
d12cfa8
Compare
Question to reviewers: stable Safari doesn’t yet support lookbehinds, and throws an early parse error when it encounters the first lookbehind in JS source. As a result, the test harness report page remains blank. Is this acceptable to convey the “fail” state? (If we wanted the harness to show up, we’d have to turn all RegExp literals using lookbehinds into a string literal passed to the |
This patch ports the tests from [1] into WPT. Any uses of the following Test262 harness APIs have been replaced with the corresponding WPT testharness.js API: - `assert.compareArray` → `assert_array_equals` - `assert.sameValue` → `assert_equals` The tests pass in Chrome and Firefox. [1]: https://github.com/tc39/test262/tree/main/test/built-ins/RegExp/lookBehind
d12cfa8
to
1f9fe74
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this!
Since it's already fixed in STP I think that's OK, yeah. It's certainly not the best failure more, but good enough for something that's being fixed. |
This patch ports the tests from https://github.com/tc39/test262/tree/main/test/built-ins/RegExp/lookBehind into WPT.
Any uses of the following Test262 harness APIs have been replaced with the corresponding WPT testharness.js API:
assert.compareArray
→assert_array_equals
assert.sameValue
→assert_equals
The tests pass in Chrome and Firefox.
This is motivated by the proposal to include RegExp lookbehind functionality in Interop 2023, which would require WPT coverage since Test262 results cannot yet be consumed / reported by the wpt.fyi & Interop dashboards.