Skip to content
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

[idlharness.js] Support subsetTestByKey in idlharness #11414

Merged
merged 6 commits into from Jun 13, 2018

Conversation

zcorpan
Copy link
Member

@zcorpan zcorpan commented Jun 7, 2018

Fixes #11413.

@zcorpan zcorpan removed the quirks label Jun 8, 2018
@gsnedders
Copy link
Member

Will this not easily result in subtests moving, given idlharness.js adding/removing tests will change the number produced? A bit painful for expectation management, IMO.

@zcorpan
Copy link
Member Author

zcorpan commented Jun 8, 2018

Yes. Could split more semantically.

@zcorpan
Copy link
Member Author

zcorpan commented Jun 12, 2018

@gsnedders PTAL

@zcorpan zcorpan changed the title [idlharness.js] Support subsetTest in idlharness [idlharness.js] Support subsetTestByName in idlharness Jun 12, 2018
Copy link
Member

@gsnedders gsnedders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM. Would be interested in opinions from others, though?

@gsnedders
Copy link
Member

One quick thought: I wonder if subtestByName is really the best name? To me, that made me think it used test.name at first glance, which it doesn't. I wonder if something like subtestByKey(key, callback, ...) makes more sense?

@wpt-pr-bot
Copy link
Collaborator

There are no owners for this pull request. Please reach out on W3C's irc server (irc.w3.org, port 6665) on channel #testing (web client) to get help with this. Thank you!

@zcorpan
Copy link
Member Author

zcorpan commented Jun 13, 2018

Renamed to key.

@gsnedders gsnedders merged commit 9bf1daa into master Jun 13, 2018
@gsnedders gsnedders deleted the zcorpan/idlharness-subsettests branch June 13, 2018 09:13
@zcorpan zcorpan changed the title [idlharness.js] Support subsetTestByName in idlharness [idlharness.js] Support subsetTestByKey in idlharness Jun 18, 2018
zcorpan added a commit that referenced this pull request Jun 21, 2018
gsnedders pushed a commit that referenced this pull request Jun 21, 2018
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Jul 11, 2018
…y function, a=testonly

Automatic update from web-platform-tests[docs] Document subsetTestByKey() utility function (#11610)

Added in web-platform-tests/wpt#11414
--

wpt-commits: 973012935e322f9238878a0246ea9937a68c37b6
wpt-pr: 11610
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 3, 2019
…y function, a=testonly

Automatic update from web-platform-tests[docs] Document subsetTestByKey() utility function (#11610)

Added in web-platform-tests/wpt#11414
--

wpt-commits: 973012935e322f9238878a0246ea9937a68c37b6
wpt-pr: 11610

UltraBlame original commit: 8c42f515c0e37082762d79a2bcc9a5ecd7c0f7db
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 3, 2019
…y function, a=testonly

Automatic update from web-platform-tests[docs] Document subsetTestByKey() utility function (#11610)

Added in web-platform-tests/wpt#11414
--

wpt-commits: 973012935e322f9238878a0246ea9937a68c37b6
wpt-pr: 11610

UltraBlame original commit: 8c42f515c0e37082762d79a2bcc9a5ecd7c0f7db
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 3, 2019
…y function, a=testonly

Automatic update from web-platform-tests[docs] Document subsetTestByKey() utility function (#11610)

Added in web-platform-tests/wpt#11414
--

wpt-commits: 973012935e322f9238878a0246ea9937a68c37b6
wpt-pr: 11610

UltraBlame original commit: 8c42f515c0e37082762d79a2bcc9a5ecd7c0f7db
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants