test: fix WPT state when process exits but workers are still running · nodejs/node@c923367 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 638
• Pull requests 758
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit c923367

Browse filesBrowse the repository at this point in the historyBrowse files

panva

authored and

targos

committed

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

PR-URL: #47826 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il>

1 parent 36f7cfa commit c923367
Copy full SHA for c923367

1 file changed

+2-5Lines changed: 2 additions & 5 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/common
  • wpt.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/common/wpt.js‎

Copy file name to clipboardExpand all lines: test/common/wpt.js

+2-5Lines changed: 2 additions & 5 deletions

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

0 commit comments

Comments

0 (0)