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

nodejs / node Public

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

• Code
• Issues 632
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit f1e1312

Browse filesBrowse the repository at this point in the historyBrowse files

panva

authored and

MoLow

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 951cc7b commit f1e1312
Copy full SHA for f1e1312

1 file changed

+2-4Lines changed: 2 additions & 4 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-4Lines changed: 2 additions & 4 deletions

Original file line number	Diff line number	Diff line change
@@ -613,10 +613,8 @@ class WPTRunner {
613	613	}
614	614
615	615	process.on('exit', () => {
616		- if (this.inProgress.size > 0) {
617		- for (const filename of this.inProgress) {
618		- this.fail(filename, { name: 'Unknown' }, kIncomplete);
619		- }
	616	+ for (const spec of this.inProgress) {
	617	+ this.fail(spec, { name: 'Unknown' }, kIncomplete);
620	618	}
621	619	inspect.defaultOptions.depth = Infinity;
622	620	// Sorts the rules to have consistent output

0 commit comments

Comments

0 (0)