Commit b7ef829
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
2 files changed
Lines changed: 6 additions & 14 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -389,6 +389,11 @@ class WPTRunner { | |||
| 389 | 389 | }); | |
| 390 | 390 | ||
| 391 | 391 | 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 | + } | ||
| 392 | 397 | this.fail( | |
| 393 | 398 | testFileName, | |
| 394 | 399 | { | |
| Original file line number | Diff line number | Diff 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