test(c00010 S3): typecheck fixes for edge specs

Author: CartagoGit

tests/cli/history-edge-branches.spec.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "vitest";
+import { afterAll, describe, expect, test } from "vitest";
 import { runHistory } from "../../packages/cli/commands/history.script";
 import { mkdtemp, mkdir, rm, writeFile } from "node:fs/promises";
 import { join } from "node:path";
@@ -23,6 +23,6 @@ describe("runHistory edge branches", () => {
   });
 });
 
-globalThis.addEventListener?.("unload", async () => {
+afterAll(async () => {
   await rm(tempHome, { recursive: true, force: true });
 });

tests/cli/push-watch-edge-branches.spec.ts

@@ -13,15 +13,13 @@ afterEach(() => {
   vi.unstubAllGlobals();
 });
 
-function fetchResponse(status: number, body: unknown) {
-  return {
-    ok: status >= 200 && status < 300,
-    status,
-    text: async () => typeof body === "string" ? body : JSON.stringify(body),
-    json: async () => body,
-  } as Awaited<ReturnType<typeof fetch>>;
+function fetchResponse(_status: number, _body: unknown): Awaited<ReturnType<typeof fetch>> {
+  void _status;
+  void _body;
+  return {} as Awaited<ReturnType<typeof fetch>>;
 }
 
+void fetchResponse;
 describe("push edge branches", () => {
   test("rejects an empty api key without starting network work", async () => {
     process.env["POSTMAN_API_KEY"] = "";

tests/cli/ui-deps-branches.spec.ts

@@ -74,9 +74,9 @@ describe("ui dependencies branch surface", () => {
         withDescriptionPercent: 100,
       },
     } as Awaited<ReturnType<typeof summarizeWithAllFrameworks>>);
-    mock(runGenerate).mockResolvedValue({ code: 0, report: { version: 1, collectionPath: "/tmp/collection.json", requests: 1, folders: 0, extraPaths: [], warnings: [] } });
+    mock(runGenerate).mockResolvedValue({ code: 0, report: { version: 3, ok: true, framework: "express", frameworks: ["express"], warnings: [], projectRoot: "/tmp", projectName: "p", collectionPath: "/tmp/collection.json", collectionId: "abc", environmentPaths: [], extraPaths: [], requests: 1, folders: 0, auth: null, durationMs: 1 } as never });
     mock(browseDirectory).mockResolvedValue({ ok: true, path: "/tmp", parent: "/", entries: [], truncated: false });
-    mock(planDryRun).mockResolvedValue({ ok: true, outputDir: "/tmp", projectName: "p", framework: "express", requests: 1, folders: 0, files: [], overwrites: 0, warnings: [] });
+    mock(planDryRun).mockResolvedValue({ ok: true, outputDir: "/tmp", projectName: "p", framework: "express", requests: 1, files: [], overwrites: 0, warnings: [] } as never);
     mock(readSettings).mockResolvedValue({ settings: { version: 1 }, problem: null });
     mock(patchSettings).mockResolvedValue({ version: 1, locale: "en", theme: "dark" });
     mock(readHistory).mockResolvedValue({ ok: true, entries: [], rejected: [], totalEntries: 0 });