Skip to content
/ node Public
forked from nodejs/node

Commit

Permalink
test: fix default WPT titles
Browse files Browse the repository at this point in the history
PR-URL: nodejs#46778
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Backport-PR-URL: nodejs#46768
  • Loading branch information
panva committed Feb 22, 2023
1 parent 0dbc4c2 commit b8a0697
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions test/common/wpt.js
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
4 changes: 2 additions & 2 deletions test/wpt/status/dom/events.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@
"Event-constructors.any.js": {
"fail": {
"expected": [
"Untitled 2",
"Untitled 3"
"Event constructors 2",
"Event constructors 3"
]
}
},
Expand Down

0 comments on commit b8a0697

Please sign in to comment.