Skip to content

Commit

Permalink
test: assert that invalidcmd throws error code
Browse files Browse the repository at this point in the history
Update invalidcmd test case in test-child-process-spawn-typeerror to
assert on specific expected error code.

PR-URL: #23942
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
jeromecovington authored and MylesBorins committed Dec 3, 2018
1 parent 52c9209 commit b3ab546
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions test/parallel/test-child-process-spawn-typeerror.js
Expand Up @@ -33,12 +33,11 @@ const invalidArgValueError =
common.expectsError({ code: 'ERR_INVALID_ARG_VALUE', type: TypeError }, 14); common.expectsError({ code: 'ERR_INVALID_ARG_VALUE', type: TypeError }, 14);


const invalidArgTypeError = const invalidArgTypeError =
common.expectsError({ code: 'ERR_INVALID_ARG_TYPE', type: TypeError }, 12); common.expectsError({ code: 'ERR_INVALID_ARG_TYPE', type: TypeError }, 13);


assert.throws(function() { assert.throws(function() {
const child = spawn(invalidcmd, 'this is not an array'); spawn(invalidcmd, 'this is not an array');
child.on('error', common.mustNotCall()); }, invalidArgTypeError);
}, TypeError);


// Verify that valid argument combinations do not throw. // Verify that valid argument combinations do not throw.
spawn(cmd); spawn(cmd);
Expand Down

0 comments on commit b3ab546

Please sign in to comment.