Browse Source

test: add check in test-signal-handler

* Check that the removed listener is not called.
* Opportunistic `==` -> `===` change.

PR-URL: https://github.com/nodejs/node/pull/8248
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
v4.x
Rich Trott 9 years ago
committed by Myles Borins
parent
commit
a48469f098
  1. 4
      test/parallel/test-signal-handler.js

4
test/parallel/test-signal-handler.js

@ -32,14 +32,14 @@ var i = 0;
setInterval(function() {
console.log('running process...' + ++i);
if (i == 5) {
if (i === 5) {
process.kill(process.pid, 'SIGUSR1');
}
}, 1);
// Test on condition where a watcher for SIGNAL
// has been previously registered, and `process.listeners(SIGNAL).length === 1`
process.on('SIGHUP', function() {});
process.on('SIGHUP', function() { common.fail('should not run'); });
process.removeAllListeners('SIGHUP');
process.on('SIGHUP', function() { sighup = true; });
process.kill(process.pid, 'SIGHUP');

Loading…
Cancel
Save