Skip to content

Commit

Permalink
test: fix WPT state when process exits but workers are still running
Browse files Browse the repository at this point in the history
PR-URL: nodejs#47826
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
  • Loading branch information
panva authored and MoLow committed Jul 6, 2023
1 parent 951cc7b commit f1e1312
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions test/common/wpt.js
Original file line number Diff line number Diff line change
Expand Up @@ -613,10 +613,8 @@ class WPTRunner {
}

process.on('exit', () => {
if (this.inProgress.size > 0) {
for (const filename of this.inProgress) {
this.fail(filename, { name: 'Unknown' }, kIncomplete);
}
for (const spec of this.inProgress) {
this.fail(spec, { name: 'Unknown' }, kIncomplete);
}
inspect.defaultOptions.depth = Infinity;
// Sorts the rules to have consistent output
Expand Down

0 comments on commit f1e1312

Please sign in to comment.