fix: ensure window.api is undefined in API tests

ThomasK33 · ThomasK33 · commit 1e1bbdb78dae · 2025-12-10T11:27:55.000+01:00
The CI environment may have window.api defined, causing the APIProvider
to use the Electron client path instead of the WebSocket path, resulting
in no MockWebSocket instances being created.

Change-Id: Ic971f4e0fe8dd6efc997b04a42181eaae626b2b8
Signed-off-by: Thomas Kosiewski &lt;tk@coder.com&gt;
diff --git a/src/browser/contexts/API.test.tsx b/src/browser/contexts/API.test.tsx
@@ -93,6 +93,8 @@ describe("API reconnection", () => {
     globalThis.window = window as unknown as Window & typeof globalThis;
     globalThis.document = window.document as unknown as Document;
     globalThis.WebSocket = MockWebSocket as unknown as typeof WebSocket;
+    // Ensure we're not in Electron mode (would skip WebSocket)
+    (globalThis.window as unknown as Record<string, unknown>).api = undefined;
     // Mock import.meta.env
     (globalThis as Record<string, unknown>).import = {
       meta: { env: { VITE_BACKEND_URL: "http://localhost:3000" } },
