fix: restart on change for non-default signals (#1409) (#1430)

Fixes #1409

Author: aaronjensen

lib/monitor/run.js

@@ -243,9 +243,9 @@ function run(options) {
         process.stdin.unpipe(child.stdin);
       }
 
-      if (utils.isWindows) {
-        // For the on('exit', ...) handler above the following looks like a
-        // crash, so we set the killedAfterChange flag
+      // For the on('exit', ...) handler above the following looks like a
+      // crash, so we set the killedAfterChange flag if a restart is planned
+      if (!noRestart) {
         killedAfterChange = true;
       }
 

test/fixtures/app.js

@@ -1,3 +1,5 @@
 // process.title = 'node (fixtures/app)';
 // console.log('starting up @ ' + (process.env.PORT || 8000));
-require('http').createServer(function (req, res) { console.log('Request in'); res.end('ok'); }).listen(process.env.PORT || 8000);
+require('http').createServer(function (req, res) { console.log('Request in'); res.end('ok'); }).listen(process.env.PORT || 8000);
+
+process.on('SIGINT', function() { process.exit(130) })

test/lib/require.test.js

@@ -54,9 +54,34 @@ describe('require-able', function () {
       setTimeout(function () {
         touch.sync(appjs);
       }, 1000);
+    }).on('start', function() {
+      if (restarted) {
+        setTimeout(function() { nodemon.emit('quit') });
+      }
+    }).on('restart', function () {
+      restarted = true;
+    }).on('quit', function () {
+      assert(restarted, 'nodemon restarted and quit properly');
+      nodemon.reset(done);
+    }).on('log', function (event) {
+      // console.log(event.message);
+    });
+  });
+
+  it('should restart on file change with custom signal', function (done) {
+    var restarted = false;
+
+    utils.port++;
+    nodemon({ script: appjs, verbose: true, env: { PORT: utils.port }, signal: 'SIGINT' }).on('start', function () {
+      setTimeout(function () {
+        touch.sync(appjs);
+      }, 1000);
+    }).on('start', function() {
+      if (restarted) {
+        setTimeout(function() { nodemon.emit('quit') });
+      }
     }).on('restart', function () {
       restarted = true;
-      nodemon.emit('quit');
     }).on('quit', function () {
       assert(restarted, 'nodemon restarted and quit properly');
       nodemon.reset(done);