summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authoralohaglenn <glenngonda@gmail.com>2017-04-22 10:53:56 -0700
committerEvan Lucas <evanlucas@me.com>2017-05-02 15:22:43 -0500
commit8e6d4402a04af20fba651663ae08a1b8a51a18f0 (patch)
tree429abe9f4ac1bd08299260caf13d613539c941d8
parent660e58c7ebf98ca4a05570aeffe12e5940ffdc3b (diff)
downloadnode-new-8e6d4402a04af20fba651663ae08a1b8a51a18f0.tar.gz
test: improve test-process-kill-pid
PR-URL: https://github.com/nodejs/node/pull/12588 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
-rw-r--r--test/parallel/test-process-kill-pid.js18
1 files changed, 11 insertions, 7 deletions
diff --git a/test/parallel/test-process-kill-pid.js b/test/parallel/test-process-kill-pid.js
index 79dc00163b..959c07b58f 100644
--- a/test/parallel/test-process-kill-pid.js
+++ b/test/parallel/test-process-kill-pid.js
@@ -17,16 +17,20 @@ const assert = require('assert');
//
// process.pid, String(process.pid): ourself
-assert.throws(function() { process.kill('SIGTERM'); }, TypeError);
-assert.throws(function() { process.kill(null); }, TypeError);
-assert.throws(function() { process.kill(undefined); }, TypeError);
-assert.throws(function() { process.kill(+'not a number'); }, TypeError);
-assert.throws(function() { process.kill(1 / 0); }, TypeError);
-assert.throws(function() { process.kill(-1 / 0); }, TypeError);
+assert.throws(function() { process.kill('SIGTERM'); },
+ /^TypeError: invalid pid$/);
+assert.throws(function() { process.kill(null); }, /^TypeError: invalid pid$/);
+assert.throws(function() { process.kill(undefined); },
+ /^TypeError: invalid pid$/);
+assert.throws(function() { process.kill(+'not a number'); },
+ /^TypeError: invalid pid$/);
+assert.throws(function() { process.kill(1 / 0); }, /^TypeError: invalid pid$/);
+assert.throws(function() { process.kill(-1 / 0); }, /^TypeError: invalid pid$/);
// Test that kill throws an error for invalid signal
-assert.throws(function() { process.kill(1, 'test'); }, Error);
+assert.throws(function() { process.kill(1, 'test'); },
+ /^Error: Unknown signal: test$/);
// Test kill argument processing in valid cases.
//