Refactor fetch implementation and error handling

Author: Awad-de

src/commands/test.rerun.spec.ts

@@ -10,7 +10,6 @@ import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { describe, expect, it } from 'vitest';
 import { ApiError, RequestTimeoutError } from '../lib/errors.js';
-
 import type { RunResponse, RerunResponse, BatchRerunResponse } from '../lib/runs.types.js';
 import type { FetchImpl } from '../lib/http.js';
 import { runTestRerun, resolveWaitRequestTimeoutMs } from './test.js';
@@ -4836,7 +4835,7 @@ describe('[finding-5] batch rerun --wait: RequestTimeoutError during fan-out pol
       closure: { byProject: [] },
     };
 
-    const fetchImpl = makeFetch(url => {
+    const fetchImpl = makeFetch((url) => {
       if (url.includes('/tests/batch/rerun')) {
         return { status: 202, body: batchResp };
       }
@@ -4868,15 +4867,15 @@ describe('[finding-5] batch rerun --wait: RequestTimeoutError during fan-out pol
         stdout: (line) => stdoutLines.push(line),
         stderr: () => undefined,
       },
-    ).catch(e => e);
+    ).catch((e) => e);
 
     expect(err).toMatchObject({ exitCode: 7 });
     const parsed = JSON.parse(stdoutLines.join('\n')) as {
       accepted: Array<{ testId: string; runId: string; status: string }>;
     };
     expect(parsed.accepted).toHaveLength(2);
-    expect(parsed.accepted.map(r => r.runId).sort()).toEqual(['run_b1', 'run_b2']);
-    expect(parsed.accepted.every(r => r.status === 'timeout')).toBe(true);
+    expect(parsed.accepted.map((r) => r.runId).sort()).toEqual(['run_b1', 'run_b2']);
+    expect(parsed.accepted.every((r) => r.status === 'timeout')).toBe(true);
   });
 });
 
@@ -4939,10 +4938,10 @@ describe('[finding-4] single FE rerun --wait: TimeoutError writes partial JSON t
         ...creds,
         sleep: instantSleep,
         fetchImpl: fetchImpl as unknown as FetchImpl,
-        stdout: line => stdoutLines.push(line),
+        stdout: (line) => stdoutLines.push(line),
         stderr: () => undefined,
       },
-    ).catch(e => e);
+    ).catch((e) => e);
 
     expect(err).toMatchObject({ exitCode: 7 });
     expect(stdoutLines.length).toBeGreaterThan(0);