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

[v16.x backport] test: add WPTRunner support for variants and generating WPT reports #46768

Closed
wants to merge 3 commits into from
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: fix default WPT titles
PR-URL: #46778
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Backport-PR-URL: #46768
panva committed Feb 22, 2023
commit b8a0697d3f3d7b8b47de7517abd4ac11e901089a
3 changes: 3 additions & 0 deletions test/common/wpt.js
Original file line number Diff line number Diff line change
@@ -722,6 +722,9 @@ class WPTRunner {
resultCallback(filename, test, reportResult) {
const status = this.getTestStatus(test.status);
const title = this.getTestTitle(filename);
if (/^Untitled( \d+)?$/.test(test.name)) {
test.name = `${title}${test.name.slice(8)}`;
}
console.log(`---- ${title} ----`);
if (status !== kPass) {
this.fail(filename, test, status, reportResult);
4 changes: 2 additions & 2 deletions test/wpt/status/dom/events.json
Original file line number Diff line number Diff line change
@@ -19,8 +19,8 @@
"Event-constructors.any.js": {
"fail": {
"expected": [
"Untitled 2",
"Untitled 3"
"Event constructors 2",
"Event constructors 3"
]
}
},