explicitly kill the child process after the test has run

Author: dotnetCarpenter

bin/codecov

@@ -23,7 +23,7 @@ var args = argv.option([
 
 // listen for piped input for 500ms
 var listeningTime = 500;
-/*process.stdin.setEncoding('utf8');
+process.stdin.setEncoding('utf8');
 process.stdin.on('data', function(report) {
   if (report !== null) {
     clearTimeout(timer);
@@ -34,5 +34,5 @@ process.stdin.on('data', function(report) {
 process.stdin.on('end', function() {
   codecov.upload(args);
 });
-*/
+
 var timer = setTimeout(codecov.upload, listeningTime, args)

test/index.js

@@ -128,9 +128,10 @@ describe("Codecov", function(){
 
   it("can read piped reports", function(done){
     var exec = require('child_process').exec;
-    exec('cat test/example.coverage.txt | bin/codecov --dump --disable=gcov', function(err, stdout, stderr){
+    var childProcess = exec('cat test/example.coverage.txt | bin/codecov --dump --disable=gcov', function(err, stdout, stderr){
       expect(stdout.toString()).to.contain('path=piped');
       expect(stdout.toString()).to.contain('this file is intentionally left blank');
+      childProcess.kill();
       done();
     });
   });