Update tap, touch, make sure all our processes respect SIGTERM

Author: bjornstar

.gitignore

@@ -1,4 +1,5 @@
 .DS_Store
+.nyc_output
 .vagrant
 node_modules
 npm-debug.log

.npmignore

@@ -1,2 +1,3 @@
+.nyc_output
 package-lock.json
 test

package.json

@@ -43,7 +43,7 @@
     "eslint": "^5.16.0",
     "eslint-config-airbnb-base": "^13.1.0",
     "eslint-plugin-import": "^2.17.2",
-    "tap": "^12.6.2",
-    "touch": "^1.0.0"
+    "tap": "^13.1.2",
+    "touch": "^3.1.0"
   }
 }

test/fixture/cluster.js

@@ -1,19 +1,30 @@
 var cluster = require('cluster');
 
-function createWorker() {
+function createWorker(i) {
   var worker = cluster.fork();
   worker.on('message', function (msg) {
-    console.log('Message from worker:', msg);
+    console.log('Message from worker', i, ':', msg);
   });
+  worker.on('disconnect', function () {
+    console.log('Worker disconnected', i);
+  });
+  return worker;
 }
 
+var workers = [];
+
 if (cluster.isMaster) {
   for (var i = 0; i < 2; i += 1) {
     console.log('Forking worker', i);
-    createWorker();
+    workers.push(createWorker(i));
   }
+  process.on('SIGTERM', function () {
+    console.log('Master received SIGTERM');
+    workers.forEach(function (worker) {
+      worker.disconnect();
+    });
+  });
 } else {
-  console.log('Worker started.');
   process.send('Hello');
   require('./server');
 }

test/fixture/ipc-server.js

@@ -24,3 +24,7 @@ process.on('message', function (data) {
 process.on('exit', function () {
   console.log('exit');
 });
+
+process.on('SIGTERM', function () {
+  server.close();
+});

test/fixture/pid.js

@@ -1,2 +1,3 @@
 require('http').createServer().listen(0);
 console.log(process.pid);
+process.on('SIGTERM', function () { process.exit(); });

test/fixture/server.coffee

@@ -12,3 +12,5 @@ server.listen 8080
 
 console.log 'Server running at http://localhost:8080/'
 console.log message
+
+process.on 'SIGTERM', -> server.close()