Skip to content

Commit b7ef829

Browse files
targosdanielleadams
authored andcommitted
test: ignore WPT worker errors after tests finished
This can happen normally, for example in timers tests. PR-URL: #37626 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
1 parent 257b1ab commit b7ef829

File tree

2 files changed

+6
-14
lines changed

2 files changed

+6
-14
lines changed

test/common/wpt.js

+5
Original file line numberDiff line numberDiff line change
@@ -389,6 +389,11 @@ class WPTRunner {
389389
});
390390

391391
worker.on('error', (err) => {
392+
if (!this.inProgress.has(testFileName)) {
393+
// The test is already finished. Ignore errors that occur after it.
394+
// This can happen normally, for example in timers tests.
395+
return;
396+
}
392397
this.fail(
393398
testFileName,
394399
{
+1-14
Original file line numberDiff line numberDiff line change
@@ -1,14 +1 @@
1-
{
2-
"negative-settimeout.any.js": {
3-
"fail": "assert_unreached: Reached unreachable code"
4-
},
5-
"type-long-setinterval.any.js": {
6-
"fail": "assert_unreached: Reached unreachable code"
7-
},
8-
"type-long-settimeout.any.js": {
9-
"fail": "assert_unreached: Reached unreachable code"
10-
},
11-
"negative-setinterval.any.js": {
12-
"fail": "assert_unreached: Reached unreachable code"
13-
}
14-
}
1+
{}

0 commit comments

Comments
 (0)