Change kill signal and add unit test

lib/killer.js

@@ -2,7 +2,7 @@ const psTree = require('ps-tree');
 const spawn = require('cross-spawn');
 const { exec } = require('child_process');
 
-let KILL_SIGNAL = '15'; // SIGTERM
+let KILL_SIGNAL = '-15'; // SIGTERM
 let hasPS = true;
 
 const isWindows = process.platform === 'win32';
@@ -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);
       }
     }
   });

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)
+  })
+})