test: improve test coverage in child_process

PR-URL: #26282 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: Rich Trott <[email protected]>

Author: juanarbol

test/parallel/test-child-process-validate-stdio.js

@@ -5,14 +5,13 @@ const common = require('../common');
 constassert=require('assert');
 constgetValidStdio=require('internal/child_process').getValidStdio;
 
-constexpectedError=
-common.expectsError({code: 'ERR_INVALID_OPT_VALUE',type: TypeError},2);
+constexpectedError={code: 'ERR_INVALID_OPT_VALUE',type: TypeError};
 
 // Should throw if string and not ignore, pipe, or inherit
-assert.throws(()=>getValidStdio('foo'),expectedError);
+common.expectsError(()=>getValidStdio('foo'),expectedError);
 
 // Should throw if not a string or array
-assert.throws(()=>getValidStdio(600),expectedError);
+common.expectsError(()=>getValidStdio(600),expectedError);
 
 // Should populate stdio with undefined if len < 3
 {
@@ -30,6 +29,19 @@ common.expectsError(() => getValidStdio(stdio2, true),
 {code: 'ERR_IPC_SYNC_FORK',type: Error}
 );
 
+// Should throw if stdio is not a valid input
+{
+conststdio=['foo'];
+common.expectsError(()=>getValidStdio(stdio,false),
+{code: 'ERR_INVALID_SYNC_FORK_INPUT',type: TypeError}
+);
+}
+
+// Should throw if stdio is not a valid option
+{
+conststdio=[{foo: 'bar'}];
+common.expectsError(()=>getValidStdio(stdio),expectedError);
+}
 
 if(common.isMainThread){
 conststdio3=[process.stdin,process.stdout,process.stderr];