Skip to content

Commit

Permalink
test: allow ENOENT in test-worker-init-failure
Browse files Browse the repository at this point in the history
PR-URL: #34769
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Denys Otrishko <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Ricky Zhou <[email protected]>
  • Loading branch information
Trott authored and addaleax committed Sep 22, 2020
1 parent 238d01f commit 5352cde
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/parallel/test-worker-init-failure.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,14 +30,14 @@ if (process.argv[2] === 'child') {
});

// We want to test that if there is an error in a constrained running
// environment, it will be one of `ENFILE`, `EMFILE`, or
// environment, it will be one of `ENFILE`, `EMFILE`, 'ENOENT', or
// `ERR_WORKER_INIT_FAILED`.
const allowableCodes = ['ERR_WORKER_INIT_FAILED', 'EMFILE', 'ENFILE'];
const expected = ['ERR_WORKER_INIT_FAILED', 'EMFILE', 'ENFILE', 'ENOENT'];

// `common.mustCall*` cannot be used here as in some environments
// (i.e. single cpu) `ulimit` may not lead to such an error.
worker.on('error', (e) => {
assert.ok(allowableCodes.includes(e.code), `${e.code} not expected`);
assert.ok(expected.includes(e.code), `${e.code} not expected`);
});
}

Expand Down

0 comments on commit 5352cde

Please sign in to comment.