Skip to content
Browse files

Merge pull request #36 from robert-chiniquy/master

Change SIGTERM to SIGKILL for child processes
  • Loading branch information...
2 parents 631a9ad + bbadbc8 commit 66bb309e5a23c989bf5416afeadc887027e388b0 @Kami Kami committed May 14, 2012
Showing with 6 additions and 3 deletions.
  1. +3 −0 CHANGES.md
  2. +1 −1 lib/process_runner/runner.js
  3. +2 −2 lib/run.js
View
3 CHANGES.md
@@ -1,6 +1,9 @@
Changes
=======
+* Send the SIGKILL signal instead of SIGTERM when killing child processes managed by the process runner.
+ [Robert Chiniquy]
+
* 11.05.2012 - v0.6.10:
* Add 'spy' functionality to the 'test' object.
View
2 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() {});
View
4 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');
}
}
};

0 comments on commit 66bb309

Please sign in to comment.
Something went wrong with that request. Please try again.