test: remove error message checking in test-worker-init-failure · nodejs/node@9a79020 · GitHub

nodejs / node Public

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

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

Additional navigation options

Commit 9a79020

Browse filesBrowse the repository at this point in the historyBrowse files

Trott

authored and

BethGriggs

committed

test: remove error message checking in test-worker-init-failure

Let the check for the error code suffice and don't check for a particular string in the message. PR-URL: #34727 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com>

1 parent aed8237 commit 9a79020
Copy full SHA for 9a79020

1 file changed

-1Lines changed: 0 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-worker-init-failure.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-worker-init-failure.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-worker-init-failure.js

-1Lines changed: 0 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -35,7 +35,6 @@ if (process.argv[2] === 'child') {
35	35	// (i.e. single cpu) `ulimit` may not lead to such an error.
36	36
37	37	worker.on('error', (e) => {
38		- assert.match(e.message, /EMFILE/);
39	38	assert.ok(e.code === 'ERR_WORKER_INIT_FAILED' || e.code === 'EMFILE');
40	39	});
41	40	}

0 commit comments

Comments

0 (0)