fix: skip PDF tool tests on Windows (process.platform check)

hasCommand() now returns false immediately on win32 instead of
trying to run 'which' (which doesn't exist on Windows). Also adds
pdfinfo to the skip check in pdf-content.test.ts and warns on Linux
when tools are missing.

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>

Author: simongdavies

tests/pdf-content.test.ts

@@ -26,16 +26,31 @@ function extractText(doc: any): string {
   }
 }
 
-function hasPdftotext(): boolean {
+/** Check if a command-line tool is available (cross-platform). */
+function hasCommand(cmd: string): boolean {
+  if (process.platform === "win32") return false;
   try {
-    execSync("which pdftotext", { stdio: "ignore" });
+    execSync(`which ${cmd}`, { stdio: "ignore" });
     return true;
   } catch {
     return false;
   }
 }
 
-const skip = !hasPdftotext();
+const skip = !hasCommand("pdftotext") || !hasCommand("pdfinfo");
+
+if (!skip) {
+  // Tools available
+} else if (process.platform !== "win32") {
+  const missing = ["pdftotext", "pdfinfo"]
+    .filter((cmd) => !hasCommand(cmd))
+    .join(", ");
+  console.warn(
+    `\n⚠️  WARNING: missing PDF tools (${missing}) — skipping content extraction tests.` +
+      "\n   Install with: sudo apt-get install poppler-utils\n",
+  );
+}
+
 const itP = skip ? it.skip : it;
 
 describe("PDF content extraction (pdftotext)", () => {

tests/pdf-fonts.test.ts

@@ -15,8 +15,9 @@ const pdf: any = await import("../builtin-modules/pdf.js");
 
 // ── Tool / Font Availability ─────────────────────────────────────────
 
-/** Check if a command-line tool is available on this system. */
+/** Check if a command-line tool is available (cross-platform). */
 function hasCommand(cmd: string): boolean {
+  if (process.platform === "win32") return false;
   try {
     execSync(`which ${cmd}`, { stdio: "ignore" });
     return true;

tests/pdf-visual.test.ts

@@ -28,8 +28,9 @@ const MAX_DIFF_PIXELS = 50; // fail if more than this many pixels differ
 
 // ── Tool Availability ────────────────────────────────────────────────
 
-/** Check if a command-line tool is available on this system. */
+/** Check if a command-line tool is available (cross-platform). */
 function hasCommand(cmd: string): boolean {
+  if (process.platform === "win32") return false;
   try {
     execSync(`which ${cmd}`, { stdio: "ignore" });
     return true;