fixup! test_runner: validate timeout option

Author: aduh95

lib/internal/test_runner/test.js

@@ -51,7 +51,7 @@ function testTimeout(promise, timeout) {
     return promise;
   }
   return SafePromiseRace([
-    promise,
+    PromiseResolve(promise),
     PromisePrototypeThen(setTimeout(timeout, null, { ref: false }), () => {
       throw new ERR_TEST_FAILURE(
         `test timed out after ${timeout}ms`,

test/parallel/test-runner-option-validation.js

@@ -4,12 +4,12 @@ const assert = require('assert');
 const test = require('node:test');
 
 [Symbol(), {}, [], () => {}, 1n, true, '1'].forEach((timeout) => {
-  assert.throws(() => test('name', { timeout }), { code: 'ERR_INVALID_ARG_TYPE' });
+  assert.throws(() => test({ timeout }), { code: 'ERR_INVALID_ARG_TYPE' });
 });
-[-1, 2 ** 33, 1.1].forEach((timeout) => {
-  assert.throws(() => test('name', { timeout }), { code: 'ERR_OUT_OF_RANGE' });
+[-1, 2 ** 33, 1.1, -Infinity, NaN].forEach((timeout) => {
+  assert.throws(() => test({ timeout }), { code: 'ERR_OUT_OF_RANGE' });
 });
 [null, undefined, Infinity, 0, 1].forEach((timeout) => {
   // Valid values should not throw.
-  test('name', { timeout });
+  test({ timeout });
 });