Permalink
Browse files

test: add check in test-signal-handler

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

PR-URL: #8248
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information...
Trott committed Aug 24, 2016
1 parent 0482d6d commit a6d53c67795bf957b8b57d96814520f98069e80e
Showing with 2 additions and 2 deletions.
  1. +2 −2 test/parallel/test-signal-handler.js
@@ -22,14 +22,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', common.mustCall(function() {}));
process.kill(process.pid, 'SIGHUP');

0 comments on commit a6d53c6

Please sign in to comment.