Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
test-cluster-worker-kill: use SIGKILL instead of SIGHUP
Browse files Browse the repository at this point in the history
In this test sending SIGKILL has the same effect as sending SIGHUP,
but SIGKILL has the advantage that it works on Windows too.
  • Loading branch information
piscisaureus committed Jun 19, 2012
1 parent 3795d77 commit 7a4dfb6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/simple/test-cluster-worker-kill.js
Expand Up @@ -21,7 +21,7 @@


// test-cluster-worker-kill.js
// verifies that, when a child process is killed (we use SIGHUP)
// verifies that, when a child process is killed (we use SIGKILL)
// - the parent receives the proper events in the proper order, no duplicates
// - the exitCode and signalCode are correct in the 'exit' event
// - the worker.suicide flag, and worker.state are correct
Expand All @@ -40,7 +40,7 @@ if (cluster.isWorker) {

} else if (cluster.isMaster) {

var KILL_SIGNAL = 'SIGHUP',
var KILL_SIGNAL = 'SIGKILL',
expected_results = {
cluster_emitDisconnect: [1, "the cluster did not emit 'disconnect'"],
cluster_emitExit: [1, "the cluster did not emit 'exit'"],
Expand Down

0 comments on commit 7a4dfb6

Please sign in to comment.