-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
tests: add web platform test runner #8990
Conversation
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.
A couple thoughts/nits
Co-Authored-By: Kitson Kelly <[email protected]>
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.
Awesome! LGTM.
This is really awesome! If you can output the test results to a JSON file and render them in |
Sounds like a good idea. Will investigate. |
Closes #6229
cc @kitsonk I only enabled a minimal set of tests for
streams
, because most of the rest does not work. We can start enabling them one by one in follow ups.Files are all concatenated together, because WPT tests are meant to work with scripts, not modules.
Once we support external source mapping, I will add source mapping for WPT tests.I have added source mapping. I have checked that it works with #8995.