fix: make header command quoting tests cross-platform

Extract quoteCommand into shared platform utils and use it in
cliExec and cliConfig tests so assertions match Windows double
quotes vs Unix single quotes.

Author: EhabY

test/unit/cliConfig.test.ts

@@ -14,7 +14,7 @@ import {
 } from "@/settings/cli";
 
 import { MockConfigurationProvider } from "../mocks/testHelpers";
-import { isWindows } from "../utils/platform";
+import { quoteCommand } from "../utils/platform";
 
 vi.mock("node:os");
 
@@ -320,9 +320,3 @@ describe("cliConfig", () => {
 		});
 	});
 });
-
-function quoteCommand(value: string): string {
-	// Used to escape environment variables in commands. See `getHeaderArgs` in src/headers.ts
-	const quote = isWindows() ? '"' : "'";
-	return `${quote}${value}${quote}`;
-}

test/unit/core/cliExec.test.ts

@@ -4,7 +4,7 @@ import path from "path";
 import { beforeAll, describe, expect, it, vi } from "vitest";
 
 import { MockConfigurationProvider } from "../../mocks/testHelpers";
-import { isWindows, writeExecutable } from "../../utils/platform";
+import { isWindows, quoteCommand, writeExecutable } from "../../utils/platform";
 
 import type { CliEnv } from "@/core/cliExec";
 
@@ -147,7 +147,7 @@ describe("cliExec", () => {
 				"--url",
 				"http://localhost:3000",
 				"--header-command",
-				"'my-header-cmd'",
+				quoteCommand("my-header-cmd"),
 				"speedtest",
 				"owner/workspace",
 				"--output",
@@ -192,7 +192,7 @@ describe("cliExec", () => {
 				"--url",
 				"http://localhost:3000",
 				"--header-command",
-				"'my-header-cmd'",
+				quoteCommand("my-header-cmd"),
 				"support",
 				"bundle",
 				"owner/workspace",

test/utils/platform.ts

@@ -101,6 +101,15 @@ export function shimExecFile<
 	return Object.assign({}, mod, { execFile });
 }
 
+/**
+ * Wraps a value in the platform-appropriate quote character,
+ * matching the escaping in {@link getHeaderArgs} from src/settings/headers.ts.
+ */
+export function quoteCommand(value: string): string {
+	const quote = isWindows() ? '"' : "'";
+	return `${quote}${value}${quote}`;
+}
+
 export function expectPathsEqual(actual: string, expected: string) {
 	expect(normalizePath(actual)).toBe(normalizePath(expected));
 }