fix(tests): stabilize shell mocks across bun test suite

Author: roodboi

tests/global-command.test.ts

@@ -43,8 +43,9 @@ mock.module("../src/lib/shell.ts", () => ({
     runCalls.push([...cmd]);
     return 0;
   },
-  findExecutableInPath: async (name?: string) =>
+  findExecutableInPath: (name?: string) =>
     name === "hack" ? "/usr/local/bin/hack" : "/usr/bin/mkcert",
+  CommandError: class CommandError extends Error {},
 }));
 
 mock.module("../src/lib/os.ts", () => ({

tests/log-backend.test.ts

@@ -1,22 +1,19 @@
 import { beforeEach, expect, mock, test } from "bun:test";
 
-const runCalls: string[][] = [];
 const dockerJsonCalls: Record<string, unknown>[] = [];
+const dockerPlainCalls: Record<string, unknown>[] = [];
 const dockerPrettyCalls: Record<string, unknown>[] = [];
 const lokiCalls: Record<string, unknown>[] = [];
 
-mock.module("../src/lib/shell.ts", () => ({
-  run: async (cmd: readonly string[]) => {
-    runCalls.push([...cmd]);
-    return 0;
-  },
-}));
-
 mock.module("../src/ui/docker-logs.ts", () => ({
   dockerComposeLogsJson: async (opts: Record<string, unknown>) => {
     dockerJsonCalls.push(opts);
     return 0;
   },
+  dockerComposeLogsPlain: async (opts: Record<string, unknown>) => {
+    dockerPlainCalls.push(opts);
+    return 0;
+  },
   dockerComposeLogsPretty: async (opts: Record<string, unknown>) => {
     dockerPrettyCalls.push(opts);
     return 0;
@@ -37,8 +34,8 @@ import {
 } from "../src/backends/log-backend.ts";
 
 beforeEach(() => {
-  runCalls.length = 0;
   dockerJsonCalls.length = 0;
+  dockerPlainCalls.length = 0;
   dockerPrettyCalls.length = 0;
   lokiCalls.length = 0;
 });
@@ -81,21 +78,15 @@ test("composeLogBackend routes plain output to docker compose logs", async () =>
     profiles: ["ops"],
   });
 
-  expect(runCalls[0]).toEqual([
-    "docker",
-    "compose",
-    "-p",
-    "proj",
-    "-f",
-    "docker-compose.yml",
-    "--profile",
-    "ops",
-    "logs",
-    "-f",
-    "--tail",
-    "10",
-    "api",
-  ]);
+  expect(dockerPlainCalls.length).toBe(1);
+  expect(dockerPlainCalls[0]).toMatchObject({
+    composeFile: "docker-compose.yml",
+    follow: true,
+    tail: 10,
+    service: "api",
+    composeProject: "proj",
+    profiles: ["ops"],
+  });
 });
 
 test("lokiLogBackend.isAvailable proxies canReachLoki", async () => {

tests/runtime-backend.test.ts

@@ -8,10 +8,16 @@ mock.module("../src/lib/shell.ts", () => ({
     execCalls.push([...cmd]);
     return { exitCode: 0, stdout: "", stderr: "" };
   },
+  execOrThrow: async (cmd: readonly string[]) => {
+    execCalls.push([...cmd]);
+    return { exitCode: 0, stdout: "", stderr: "" };
+  },
   run: async (cmd: readonly string[]) => {
     runCalls.push([...cmd]);
     return 0;
   },
+  findExecutableInPath: () => "/usr/bin/docker",
+  CommandError: class CommandError extends Error {},
 }));
 
 import { composeRuntimeBackend } from "../src/backends/runtime-backend.ts";