test: add coverage for spawnSync() killSignal

This commit adds a test for the killSignal option to spawnSync(), and the other sync child process functions by extension. PR-URL: #8960 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>

Author: cjihrig

test/parallel/test-child-process-spawnsync-kill-signal.js

@@ -0,0 +1,51 @@
+'use strict';
+constcommon=require('../common');
+constassert=require('assert');
+constcp=require('child_process');
+
+if(process.argv[2]==='child'){
+setInterval(()=>{},1000);
+}else{
+constexitCode=common.isWindows ? 1 : 0;
+constSIGKILL=process.binding('constants').SIGKILL;
+
+functionspawn(killSignal){
+constchild=cp.spawnSync(process.execPath,
+[__filename,'child'],
+{killSignal,timeout: 100});
+
+assert.strictEqual(child.status,exitCode);
+assert.strictEqual(child.error.code,'ETIMEDOUT');
+returnchild;
+}
+
+// Verify that an error is thrown for unknown signals.
+assert.throws(()=>{
+spawn('SIG_NOT_A_REAL_SIGNAL');
+},/Error:Unknownsignal:SIG_NOT_A_REAL_SIGNAL/);
+
+// Verify that the default kill signal is SIGTERM.
+{
+constchild=spawn();
+
+assert.strictEqual(child.signal,'SIGTERM');
+assert.strictEqual(child.options.killSignal,undefined);
+}
+
+// Verify that a string signal name is handled properly.
+{
+constchild=spawn('SIGKILL');
+
+assert.strictEqual(child.signal,'SIGKILL');
+assert.strictEqual(child.options.killSignal,SIGKILL);
+}
+
+// Verify that a numeric signal is handled properly.
+{
+constchild=spawn(SIGKILL);
+
+assert.strictEqual(typeofSIGKILL,'number');
+assert.strictEqual(child.signal,'SIGKILL');
+assert.strictEqual(child.options.killSignal,SIGKILL);
+}
+}