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

[wptrunner] Support testdriver.js in {ref,print-ref,crash}tests #48486

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jonathan-j-lee
Copy link
Contributor

@jonathan-j-lee jonathan-j-lee commented Oct 5, 2024

Implements the wpt {manifest,run} changes proposed in web-platform-tests/rfcs#211.

This will let wptrunner gracefully skip testdriver tests if the executor
doesn't support them.
Non-testharness executors that don't support testdriver simply skip
those tests.
@jonathan-j-lee jonathan-j-lee force-pushed the wptrunner/testdriver-reftest branch from a0230ef to 0efaa69 Compare December 10, 2024 21:55
@jonathan-j-lee jonathan-j-lee marked this pull request as ready for review December 10, 2024 22:10
@wpt-pr-bot wpt-pr-bot added docs infra manifest wptrunner The automated test runner, commonly called through ./wpt run labels Dec 10, 2024
@jonathan-j-lee jonathan-j-lee removed the request for review from sideshowbarker December 10, 2024 22:10
@jonathan-j-lee
Copy link
Contributor Author

PTAL @WeizhongX @jgraham. Chrome and Safari pass the new infrastructure/ tests.

@sideshowbarker sideshowbarker removed their assignment Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs infra manifest wptrunner The automated test runner, commonly called through ./wpt run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants