test_runner: fix env option validation

Signed-off-by: hanityx <hanityx@gmail.com>
PR-URL: #64865
Reviewed-By: Aviv Keller <me@aviv.sh>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

Author: hanityx

lib/internal/test_runner/runner.js

@@ -910,7 +910,7 @@ function run(options = kEmptyObject) {
   }
 
   if (env != null) {
-    validateObject(env);
+    validateObject(env, 'options.env');
 
     if (isolation === 'none') {
       throw new ERR_INVALID_ARG_VALUE('options.env', env, 'is not supported with isolation=\'none\'');

test/parallel/test-runner-run.mjs

@@ -673,6 +673,14 @@ describe('require(\'node:test\').run', { concurrency: true }, () => {
         }));
     });
 
+    it('should only allow object in options.env', () => {
+      [Symbol(), [], () => {}, 0, 1, 0n, 1n, '', '1', true, false]
+        .forEach((env) => assert.throws(() => run({ files: [], env }), {
+          code: 'ERR_INVALID_ARG_TYPE',
+          message: /The "options\.env" property must be of type object\./
+        }));
+    });
+
     it('should not allow files and globPatterns used together', () => {
       assert.throws(() => run({ files: ['a.js'], globPatterns: ['*.js'] }), {
         code: 'ERR_INVALID_ARG_VALUE'