Skip to content

Commit

Permalink
test: fix test-process-kill-pid on Windows
Browse files Browse the repository at this point in the history
Disabling the part of the test that relies on dispatching SIGHUP,
because sending SIGHUP is not supported on Windows.

Signed-off-by: Timothy J Fontaine <tjfontaine@gmail.com>
  • Loading branch information
orangemocha authored and tjfontaine committed Aug 8, 2014
1 parent 4516e6d commit 2a41535
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions test/simple/test-process-kill-pid.js
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,13 @@ assert.throws(function() { process.kill(+'not a number'); }, TypeError);
assert.throws(function() { process.kill(1/0); }, TypeError);
assert.throws(function() { process.kill(-1/0); }, TypeError);

/* Sending SIGHUP is not supported on Windows */
if (process.platform === 'win32') {
pass = true;
clearInterval(wait);
return;
}

process.once('SIGHUP', function() {
process.once('SIGHUP', function() {
pass = true;
Expand Down

0 comments on commit 2a41535

Please sign in to comment.