1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: add check in test-signal-handler

Browse Source 
* 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>
This commit is contained in:
Rich Trott 2016-08-23 23:18:23 -07:00
parent 0482d6d592
commit a6d53c6779
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -22,14 +22,14 @@ var i = 0;
setInterval(function() { setInterval(function() {
console.log('running process...' + ++i); console.log('running process...' + ++i);
if (i == 5) { if (i === 5) {
process.kill(process.pid, 'SIGUSR1'); process.kill(process.pid, 'SIGUSR1');
} }
}, 1); }, 1);
// Test on condition where a watcher for SIGNAL // Test on condition where a watcher for SIGNAL
// has been previously registered, and `process.listeners(SIGNAL).length === 1` // 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.removeAllListeners('SIGHUP');
process.on('SIGHUP', common.mustCall(function() {})); process.on('SIGHUP', common.mustCall(function() {}));
process.kill(process.pid, 'SIGHUP'); process.kill(process.pid, 'SIGHUP');