Merge pull request #91 from carnun/master

Change kill signal and add unit test

lib/killer.js

@@ -21,10 +21,10 @@ module.exports = function kill(child) {
     } else {
       if (hasPS) {
         psTree(child.pid, function(err, kids) {
-          spawn('kill', ['-s', KILL_SIGNAL, child.pid].concat( kids.map(p => p.PID) )).on('close', resolve);
+          spawn('kill', ['-' + KILL_SIGNAL, child.pid].concat( kids.map(p => p.PID) )).on('close', resolve);
         });
       } else {
-        exec('kill -s ' + KILL_SIGNAL + ' ' + child.pid, resolve);
+        exec('kill -' + KILL_SIGNAL + ' ' + child.pid, resolve);
       }
     }
   });

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tsc-watch",
-  "version": "4.2.6",
+  "version": "4.2.7",
   "description": "The TypeScript compiler with onSuccess command",
   "scripts": {
     "test": "mocha test/**/*.js"

test/client.it.js

@@ -3,7 +3,6 @@ const sinon = require('sinon');
 const mochaEventually = require('mocha-eventually');
 const eventually = fn => mochaEventually(fn, 8000, 50);
 const TscWatchClient = require('../lib/client');
-const { driver: tscWatchDriver } = require('./driver.js');
 
 describe('Client Events', () => {
   let watchClient;

test/runner.it.js

@@ -0,0 +1,23 @@
+const { expect } = require('chai');
+const runner = require('../lib/runner');
+
+describe.only('Runner', () => {
+  it('Should start a long running process and kill it before it finishes', (done) => {
+    const kill = runner('sleep 10')
+    setTimeout(function() {
+      kill().then(function(result) {
+        expect(result).to.deep.equal([0,null])
+        done()
+      })
+    }, 1)
+  })
+  it('Should start a short running process and not kill it before it finishes', (done) => {
+    const kill = runner('echo')
+    setTimeout(function() {
+      kill().then(function(result) {
+        expect(result).to.deep.equal([1,0])
+        done()
+      })
+    }, 5)
+  })
+})