Fix tests

Author: sindresorhus

lib/arguments/options.js

@@ -14,6 +14,8 @@ import {normalizeCwd} from './cwd.js';
 import {normalizeFileUrl} from './file-url.js';
 import {normalizeFdSpecificOptions} from './specific.js';
 
+const cmdExeRegExp = /^cmd(?:\.exe)?$/i;
+
 // Normalize the options object, and sometimes also the file paths and arguments.
 // Applies default values, validate allowed options, normalize them.
 export const normalizeOptions = (filePath, rawArguments, rawOptions) => {
@@ -37,7 +39,8 @@ export const normalizeOptions = (filePath, rawArguments, rawOptions) => {
 	options.forceKillAfterDelay = normalizeForceKillAfterDelay(options.forceKillAfterDelay);
 	options.lines = options.lines.map((lines, fdNumber) => lines && !BINARY_ENCODINGS.has(options.encoding) && options.buffer[fdNumber]);
 
-	if (process.platform === 'win32' && path.basename(file, '.exe') === 'cmd') {
+	// The file is now an absolute path resolved via `PATHEXT`, so its extension might be uppercase (`cmd.EXE`)
+	if (process.platform === 'win32' && cmdExeRegExp.test(path.basename(file))) {
 		// #116
 		commandArguments.unshift('/q');
 	}

test/terminate/timeout.js

@@ -21,7 +21,8 @@ test('timeout kills the subprocess if it times out, in sync mode', async t => {
 	t.true(isTerminated);
 	t.is(signal, 'SIGTERM');
 	t.true(timedOut);
-	t.is(originalMessage, 'spawnSync node ETIMEDOUT');
+	// On Windows, the command is spawned using the absolute path resolved via `PATHEXT`, so it appears in Node.js' own error message
+	t.regex(originalMessage, /^spawnSync .*node(?:\.[A-Za-z]+)? ETIMEDOUT$/);
 	t.is(shortMessage, `Command timed out after 1 milliseconds: node forever.js\n${originalMessage}`);
 	t.is(message, shortMessage);
 });