diff options
author | cjihrig <cjihrig@gmail.com> | 2014-12-22 16:39:51 -0500 |
---|---|---|
committer | Julien Gilli <julien.gilli@joyent.com> | 2015-01-12 16:51:29 -0800 |
commit | 10703774f01a4005fc74d6b4131c713259e81657 (patch) | |
tree | 722562ab3965a809672a2f67d7e1a51c04e1658e | |
parent | e64ee2b3f7b4067101b0291f1add842353cd6865 (diff) | |
download | node-10703774f01a4005fc74d6b4131c713259e81657.tar.gz |
test: use executable in spawn() test
Currently, the test-child-process-spawn-typeerror.js is
calling execFile() on a JavaScript source file, which is
causing failures on Windows. This commit switches to calling
spawn() on an actual executable.
Reviewed-by: Sam Roberts <sam@strongloop.com>
Reviewed-by: Julien Gilli <julien.gilli@joyent.com>
-rw-r--r-- | test/simple/test-child-process-spawn-typeerror.js | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/test/simple/test-child-process-spawn-typeerror.js b/test/simple/test-child-process-spawn-typeerror.js index 27de35500..d05c20b2d 100644 --- a/test/simple/test-child-process-spawn-typeerror.js +++ b/test/simple/test-child-process-spawn-typeerror.js @@ -100,10 +100,11 @@ assert.doesNotThrow(function() { spawn(cmd, a, o); }); assert.doesNotThrow(function() { spawn(cmd, o); }); // Variants of undefined as explicit 'no argument' at a position -assert.doesNotThrow(function() { execFile(empty, u, o); }); -assert.doesNotThrow(function() { execFile(empty, a, u); }); -assert.doesNotThrow(function() { execFile(empty, n, o); }); -assert.doesNotThrow(function() { execFile(empty, a, n); }); +assert.doesNotThrow(function() { spawn(cmd, u, o); }); +assert.doesNotThrow(function() { spawn(cmd, a, u); }); + +assert.throws(function() { spawn(cmd, n, o); }, TypeError); +assert.throws(function() { spawn(cmd, a, n); }, TypeError); assert.throws(function() { spawn(cmd, s); }, TypeError); assert.doesNotThrow(function() { spawn(cmd, a, s); }, TypeError); |