Skip to content

Commit 2a41535

Browse files
orangemochatjfontaine
authored andcommitted
test: fix test-process-kill-pid on Windows
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>
1 parent 4516e6d commit 2a41535

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

test/simple/test-process-kill-pid.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,13 @@ assert.throws(function() { process.kill(+'not a number'); }, TypeError);
5555
assert.throws(function() { process.kill(1/0); }, TypeError);
5656
assert.throws(function() { process.kill(-1/0); }, TypeError);
5757

58+
/* Sending SIGHUP is not supported on Windows */
59+
if (process.platform === 'win32') {
60+
pass = true;
61+
clearInterval(wait);
62+
return;
63+
}
64+
5865
process.once('SIGHUP', function() {
5966
process.once('SIGHUP', function() {
6067
pass = true;

0 commit comments

Comments
 (0)