Files
OmniRoute/tests/unit/ui/use-session-recorder.test.tsx
Diego Rodrigues de Sa e Souza a5cad5ab2a fix(tests): vitest UI suite back to green (69 fails triaged — WS6.1) (#7127)
test:vitest:ui was advisory/parked with 70 failing tests across 30 files (of
159 total). Triaged by grouping failures by root cause instead of fixing
one-by-one:

- 15 files (use-virtual-list, use-traffic-stream, use-system-proxy-exit-guard,
  use-session-recorder, use-resizable-panels, traffic-inspector-page,
  timing-i18n, stats-tab, session-recorder-bar, same-context-filter,
  historic-session-banner, conversation-tab, conversation-tab-separators,
  cli-tools-no-mitm-tab, agent-bridge-server-card-a11y) were authored against
  node:test but live under tests/unit/ui/*.test.tsx, which vitest.config.ts
  collects but test:unit's glob (only *.test.ts) never does — orphaned. Fixed
  by switching their describe/it/beforeEach imports to "vitest".
- jsdom does not implement window.matchMedia, and several dashboard
  components read it via useTheme() (directly, or transitively through
  ProviderIcon). Added tests/_setup/vitestUiPolyfills.ts (wired into
  vitest.config.ts) with a minimal MediaQueryList polyfill — fixed
  providerCascadeNode, ProviderIcon-icon-url, CliAgentsPage, playground-studio,
  comboLiveStudio, memories-tab, home-topology-hidden, ProxyRegistryManager-tdz.
- playground-build-tab.test.tsx (9 tests) and compressionHub*.test.tsx (2
  tests) asserted against pre-redesign UI: BuildTab now sits behind a 3-step
  BuildWizard (mode picker -> configure -> run), and CompressionHub is a
  Phase-2 thin overview without the old master toggle/mode selector/pipeline
  list. Rewrote the build-tab test to drive the wizard, and removed the two
  compressionHub.test.tsx assertions already superseded by
  compressionHub-active-selector.test.tsx. compressionHub-context-editing.test.tsx
  asserted stale Portuguese copy against a component that deliberately uses
  literal English strings (documented hydration workaround) — aligned to the
  real text.
- search-tools-compare-tab.test.tsx: the D22 4-provider cap documented in
  docs/frameworks/SEARCH_TOOLS_STUDIO.md was never implemented in CompareTab —
  fixed the component (disable extra toggles + cap selectAll + warning
  message) since the test was correct and the component was the bug. Also
  fixed an assertion looking for a <table> that never existed (the results
  panel is a div-based side-by-side layout).
- CliAgentsPage.test.tsx: the agent-tool catalog grew from 6 to 8 (omp, letta
  added) since the test was written — updated the fixture and expected count.
- memories-tab.test.tsx: a call-order-dependent fetch mock
  (mockResolvedValueOnce + fallback) broke once MemoriesTab started firing an
  immediate health check that raced its 300ms-debounced list fetch — switched
  to a URL-keyed mock like the rest of the file.
- home-topology-hidden-4596.test.tsx: useLiveDashboard now runs an async
  handshake fetch before opening the WebSocket — stubbed fetch and awaited it.
- same-context-filter.test.tsx: the filter branch moved from
  useTrafficStream.applyFilter into the extracted, reusable
  matchesTrafficFilter() helper — updated the source-grep target.
- tests/unit/ui/provider-plan-config.test.tsx deleted: it tested
  ProviderPlanConfigClient, which tests/unit/quota-plans-route-retired.test.ts
  proves was deliberately retired (Plans screen removed).

Result: test:vitest:ui 158/158 files, 870/870 tests passing (was 30 failed /
159, 70 failed / 743). test:vitest (MCP/autoCombo) still green at 28/28,
253/253. Not promoted to blocking in this PR per the task — the owner
promotes after reviewing the green suite.
2026-07-14 16:24:19 -03:00

141 lines
4.4 KiB
TypeScript

/**
* Tests for useSessionRecorder (R5-5 frontend half)
*
* Verifies that during recording, new traffic WS events trigger
* POST to /api/tools/traffic-inspector/sessions/{id}/requests.
*/
import { describe, it } from "vitest";
import assert from "node:assert/strict";
import fs from "node:fs";
import path from "node:path";
import { fileURLToPath } from "node:url";
const __dirname = path.dirname(fileURLToPath(import.meta.url));
const HOOK_SRC = fs.readFileSync(
path.resolve(
__dirname,
"../../../src/app/(dashboard)/dashboard/tools/traffic-inspector/hooks/useSessionRecorder.ts"
),
"utf8"
);
describe("useSessionRecorder R5-5 source assertions", () => {
it("opens a WebSocket during start() for traffic capture", () => {
assert.ok(
HOOK_SRC.includes("new WebSocket(wsUrl)"),
"should open a WebSocket connection inside start()"
);
});
it("POSTs to /sessions/{id}/requests on new WS event", () => {
assert.ok(
HOOK_SRC.includes("/requests"),
"should POST to sessions/{id}/requests endpoint"
);
assert.ok(
HOOK_SRC.includes(`method: "POST"`),
"should use POST method"
);
assert.ok(
HOOK_SRC.includes("payload"),
"should send payload in body"
);
});
it("buffers events and flushes in batches", () => {
assert.ok(
HOOK_SRC.includes("SNAPSHOT_FLUSH_BATCH"),
"should define SNAPSHOT_FLUSH_BATCH constant"
);
assert.ok(
HOOK_SRC.includes("SNAPSHOT_FLUSH_MS"),
"should define SNAPSHOT_FLUSH_MS constant for debounce"
);
assert.ok(
HOOK_SRC.includes("pendingSnapshotsRef"),
"should use a pendingSnapshotsRef buffer"
);
});
it("stops WS and flushes on stop()", () => {
assert.ok(
HOOK_SRC.includes("stopRecordingWs()"),
"stop() should call stopRecordingWs to clean up the WS"
);
assert.ok(
HOOK_SRC.includes("await flushSnapshots(sid)"),
"stop() should await a final flush before sending PATCH"
);
});
it("handles POST failure gracefully (does not throw)", () => {
// The fetch call must be wrapped in try/catch
const fetchBlock = HOOK_SRC.slice(HOOK_SRC.indexOf("flushSnapshots"));
assert.ok(
fetchBlock.includes("} catch {"),
"POST fetch should be wrapped in try/catch to handle failures gracefully"
);
});
it("only pushes 'new' event type to snapshots", () => {
assert.ok(
HOOK_SRC.includes(`event.type !== "new"`),
"should early-return for non-new events"
);
});
});
describe("useSessionRecorder snapshot flush logic (unit)", () => {
it("batch threshold triggers immediate flush instead of timer", () => {
// Simulate the flush decision logic
const SNAPSHOT_FLUSH_BATCH = 10;
const pendingSnapshots: string[] = [];
let immediateFlushCalled = false;
let scheduleFlushCalled = false;
const flushSnapshots = () => { immediateFlushCalled = true; };
const scheduleFlush = () => { scheduleFlushCalled = true; };
// Below threshold
pendingSnapshots.push(JSON.stringify({ id: "req-1" }));
if (pendingSnapshots.length >= SNAPSHOT_FLUSH_BATCH) {
flushSnapshots();
} else {
scheduleFlush();
}
assert.equal(immediateFlushCalled, false);
assert.equal(scheduleFlushCalled, true);
// At threshold
immediateFlushCalled = false;
scheduleFlushCalled = false;
for (let i = 0; i < SNAPSHOT_FLUSH_BATCH - 1; i++) {
pendingSnapshots.push(JSON.stringify({ id: `req-${i + 2}` }));
}
assert.equal(pendingSnapshots.length, SNAPSHOT_FLUSH_BATCH);
if (pendingSnapshots.length >= SNAPSHOT_FLUSH_BATCH) {
flushSnapshots();
} else {
scheduleFlush();
}
assert.equal(immediateFlushCalled, true);
assert.equal(scheduleFlushCalled, false);
});
it("POST URL is correct format", () => {
const sessionId = "test-session-123";
const url = `/api/tools/traffic-inspector/sessions/${encodeURIComponent(sessionId)}/requests`;
assert.equal(url, "/api/tools/traffic-inspector/sessions/test-session-123/requests");
});
it("POST body contains stringified payload", () => {
const req = { id: "req-1", host: "api.openai.com", method: "POST" };
const payload = JSON.stringify(req);
const body = JSON.stringify({ payload });
const parsed = JSON.parse(body) as { payload: string };
assert.equal(parsed.payload, payload);
const reparsed = JSON.parse(parsed.payload) as typeof req;
assert.equal(reparsed.id, "req-1");
});
});