fix: cross-platform tests for Windows CI (clipboard + formatHomeRelativePath)

Author: Lellansin

src/tests/clipboard.test.ts

@@ -36,40 +36,44 @@ test("readClipboardImage returns null when no clipboard helpers are installed",
   assert.equal(result, null);
 });
 
-test("readClipboardImage uses osascript fallback on macOS when pngpaste is missing", async () => {
-  const binDir = fs.mkdtempSync(path.join(os.tmpdir(), "deepcode-clipboard-test-bin-"));
-  try {
-    fs.writeFileSync(path.join(binDir, "pngpaste"), "#!/bin/sh\nexit 1\n", { mode: 0o755 });
-    fs.writeFileSync(
-      path.join(binDir, "osascript"),
-      [
-        "#!/bin/sh",
-        'for arg in "$@"; do',
-        '  case "$arg" in',
-        "    *'open for access POSIX file " + '"' + "'*)",
-        '      path_part=${arg#*POSIX file \\"}',
-        '      out_path=${path_part%%\\"*}',
-        '      printf fakepng > "$out_path"',
-        "      exit 0",
-        "      ;;",
-        "  esac",
-        "done",
-        "exit 1",
-        "",
-      ].join("\n"),
-      { mode: 0o755 }
-    );
+test(
+  "readClipboardImage uses osascript fallback on macOS when pngpaste is missing",
+  { skip: process.platform !== "darwin" },
+  async () => {
+    const binDir = fs.mkdtempSync(path.join(os.tmpdir(), "deepcode-clipboard-test-bin-"));
+    try {
+      fs.writeFileSync(path.join(binDir, "pngpaste"), "#!/bin/sh\nexit 1\n", { mode: 0o755 });
+      fs.writeFileSync(
+        path.join(binDir, "osascript"),
+        [
+          "#!/bin/sh",
+          'for arg in "$@"; do',
+          '  case "$arg" in',
+          "    *'open for access POSIX file " + '"' + "'*)",
+          '      path_part=${arg#*POSIX file \\"}',
+          '      out_path=${path_part%%\\"*}',
+          '      printf fakepng > "$out_path"',
+          "      exit 0",
+          "      ;;",
+          "  esac",
+          "done",
+          "exit 1",
+          "",
+        ].join("\n"),
+        { mode: 0o755 }
+      );
 
-    const moduleUrl = new URL(`../ui/clipboard.ts?t=${Date.now()}`, import.meta.url).href;
-    const { readClipboardImage } = (await import(moduleUrl)) as ClipboardModule;
+      const moduleUrl = new URL(`../ui/clipboard.ts?t=${Date.now()}`, import.meta.url).href;
+      const { readClipboardImage } = (await import(moduleUrl)) as ClipboardModule;
 
-    process.env.PATH = binDir;
-    const result = withPlatform("darwin", () => readClipboardImage());
-    assert.equal(result?.mimeType, "image/png");
-    assert.equal(result?.dataUrl, `data:image/png;base64,${Buffer.from("fakepng").toString("base64")}`);
-  } finally {
-    process.env.PATH = ORIGINAL_PATH;
-    Object.defineProperty(process, "platform", { value: ORIGINAL_PLATFORM });
-    fs.rmSync(binDir, { recursive: true, force: true });
+      process.env.PATH = binDir;
+      const result = withPlatform("darwin", () => readClipboardImage());
+      assert.equal(result?.mimeType, "image/png");
+      assert.equal(result?.dataUrl, `data:image/png;base64,${Buffer.from("fakepng").toString("base64")}`);
+    } finally {
+      process.env.PATH = ORIGINAL_PATH;
+      Object.defineProperty(process, "platform", { value: ORIGINAL_PLATFORM });
+      fs.rmSync(binDir, { recursive: true, force: true });
+    }
   }
-});
+);

src/tests/welcomeScreen.test.ts

@@ -1,17 +1,23 @@
 import { test } from "node:test";
 import assert from "node:assert/strict";
+import * as os from "node:os";
+import * as path from "node:path";
 import { buildWelcomeTips, formatHomeRelativePath } from "../ui";
 
+const home = os.homedir();
+
 test("formatHomeRelativePath returns tilde for the home directory", () => {
-  assert.equal(formatHomeRelativePath("/Users/example", "/Users/example"), "~");
+  assert.equal(formatHomeRelativePath(home, home), "~");
 });
 
 test("formatHomeRelativePath shortens paths inside the home directory", () => {
-  assert.equal(formatHomeRelativePath("/Users/example/dev/project", "/Users/example"), "~/dev/project");
+  const subdir = path.join(home, "dev", "project");
+  assert.equal(formatHomeRelativePath(subdir, home), `~${path.sep}dev${path.sep}project`);
 });
 
 test("formatHomeRelativePath keeps paths outside the home directory absolute", () => {
-  assert.equal(formatHomeRelativePath("/tmp/project", "/Users/example"), "/tmp/project");
+  const outside = process.platform === "win32" ? "C:\\tmp\\project" : "/tmp/project";
+  assert.equal(formatHomeRelativePath(outside, home), path.resolve(outside));
 });
 
 test("buildWelcomeTips includes built-in slash commands and loaded skills", () => {