changes SIGTERM to SIGKILL for child processes.

node-migrator-bot · May 14, 2012 · aa54b36 · aa54b36
1 parent 631a9ad
commit aa54b36
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,6 +1,9 @@
 Changes
 =======
 
+* Change SIGTERM to SIGKILL for child processes.
+   [Robert Chiniquy]
+
 * 11.05.2012 - v0.6.10:
 
 * Add 'spy' functionality to the 'test' object.

diff --git a/lib/process_runner/runner.js b/lib/process_runner/runner.js
@@ -415,7 +415,7 @@ ProcessRunner.prototype._stopProcess = function(process, callback) {
 
   if (!process.available_for_coverage) {
     if (!process.kill_script) {
-      process.process.kill('SIGTERM');
+      process.process.kill('SIGKILL');
     }
     else {
       exec(process.kill_script, function() {});

diff --git a/lib/run.js b/lib/run.js
@@ -126,7 +126,7 @@ TestRunner.prototype.runTests = function(testInitFile, chdir,
         'timeout': true
       };
 
-      child.kill('SIGTERM');
+      child.kill('SIGKILL');
       self._handleTestResult(filePath, resultObj);
     }
 
@@ -400,7 +400,7 @@ TestRunner.prototype.forceStop = function() {
       child = testFileData['child'];
       timeoutId = testFileData['timeout_id'];
       clearTimeout(timeoutId);
-      child.kill('SIGTERM');
+      child.kill('SIGKILL');
     }
   }
 };