Skip to content

Commit

Permalink
test: fix test-child-process-spawn-typeerror
Browse files Browse the repository at this point in the history
You cannot spawn 'dir' on Windows because it's not an executable.  Also,
some people might have 'ls' on their path on Windows, so I changed
invalidCmd to something that's highly unlikely to exist.

Reviewed-by: Trevor Norris <[email protected]>
  • Loading branch information
orangemocha authored and trevnorris committed Oct 9, 2014
1 parent 47f119c commit 61dd74b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/simple/test-child-process-spawn-typeerror.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@
var spawn = require('child_process').spawn,
assert = require('assert'),
windows = (process.platform === 'win32'),
cmd = (windows) ? 'dir' : 'ls',
invalidcmd = (windows) ? 'ls' : 'dir',
cmd = (windows) ? 'rundll32' : 'ls',
invalidcmd = 'hopefully_you_dont_have_this_on_your_machine',
invalidArgsMsg = /Incorrect value of args option/,
invalidOptionsMsg = /options argument must be an object/,
errors = 0;
Expand Down

0 comments on commit 61dd74b

Please sign in to comment.