-
Notifications
You must be signed in to change notification settings - Fork 471
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
Add tests for regexp-match-indices #2309
Conversation
@leobalter I take it there's currently no way to specify that a specific host argument is required for a test? The feature is currently enabled in V8 v7.8 under the |
Test262 is agostic with the host usage and there isn't any mapping to arguments used by any. A test runner consuming test262 could easily do this mapping from the features flags but there isn't anything already doing it. |
test/built-ins/RegExp/match-indices/indices-array-unicode-match.js
Outdated
Show resolved
Hide resolved
0b040c7
to
85db19d
Compare
The `assert.deepEqual` utility method was originally implemented in support of two tests for the "RegExp Match Indices" proposal [1]. In the years since that time, it has been referenced by one additional test, where it is interchangeable with the considerably simpler `assert.compareArray`. Refactor the RegExp tests to use a simpler inline helper function that verifies the same semantics and preserves meaningful error messages. [1] tc39#2309
Adds tests for https://github.com/tc39/proposal-regexp-match-indices