diff options
author | alohaglenn <glenngonda@gmail.com> | 2017-04-22 10:53:56 -0700 |
---|---|---|
committer | Evan Lucas <evanlucas@me.com> | 2017-05-02 15:22:43 -0500 |
commit | 8e6d4402a04af20fba651663ae08a1b8a51a18f0 (patch) | |
tree | 429abe9f4ac1bd08299260caf13d613539c941d8 | |
parent | 660e58c7ebf98ca4a05570aeffe12e5940ffdc3b (diff) | |
download | node-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.js | 18 |
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. // |