test: fix WPT state when process exits but workers are still running

panva · targos · commit c9233679e8c7 · 2023-05-12T11:56:59.000+02:00
PR-URL: #47826 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
diff --git a/test/common/wpt.js b/test/common/wpt.js
@@ -678,11 +678,8 @@ class WPTRunner {
     }
 
     process.on('exit', () => {
-      if (this.inProgress.size > 0) {
-        for (const id of this.inProgress) {
-          const spec = this.specs.get(id);
-          this.fail(spec, { 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

Original file line number	Diff line number	Diff line change
`@@ -678,11 +678,8 @@ class WPTRunner {`
`678`	`678`	`}`
`679`	`679`
`680`	`680`	`process.on('exit', () => {`
`681`		`- if (this.inProgress.size > 0) {`
`682`		`- for (const id of this.inProgress) {`
`683`		`- const spec = this.specs.get(id);`
`684`		`- this.fail(spec, { name: 'Unknown' }, kIncomplete);`
`685`		`- }`
	`681`	`+ for (const spec of this.inProgress) {`
	`682`	`+ this.fail(spec, { name: 'Unknown' }, kIncomplete);`
`686`	`683`	`}`
`687`	`684`	`inspect.defaultOptions.depth = Infinity;`
`688`	`685`	`// Sorts the rules to have consistent output`