mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-08-07 07:42:13 +03:00
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.
203 lines
6.2 KiB
TypeScript
203 lines
6.2 KiB
TypeScript
/**
|
|
* Tests for useTrafficStream — WebSocket snapshot/new/update/clear + reconnect backoff
|
|
*/
|
|
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/useTrafficStream.ts"
|
|
),
|
|
"utf8"
|
|
);
|
|
|
|
// Minimal EventEmitter-based mock WebSocket
|
|
class MockWebSocket {
|
|
static instances: MockWebSocket[] = [];
|
|
static OPEN = 1;
|
|
static CLOSING = 2;
|
|
static CLOSED = 3;
|
|
|
|
readyState = MockWebSocket.OPEN;
|
|
onopen: (() => void) | null = null;
|
|
onmessage: ((ev: { data: string }) => void) | null = null;
|
|
onclose: (() => void) | null = null;
|
|
onerror: ((ev: unknown) => void) | null = null;
|
|
url: string;
|
|
|
|
constructor(url: string) {
|
|
this.url = url;
|
|
MockWebSocket.instances.push(this);
|
|
}
|
|
|
|
close() {
|
|
this.readyState = MockWebSocket.CLOSED;
|
|
this.onclose?.();
|
|
}
|
|
|
|
simulateOpen() {
|
|
this.onopen?.();
|
|
}
|
|
|
|
simulateMessage(data: unknown) {
|
|
this.onmessage?.({ data: JSON.stringify(data) });
|
|
}
|
|
}
|
|
|
|
describe("useTrafficStream core logic", () => {
|
|
it("initializes with empty state", () => {
|
|
// The hook itself relies on React, but we can test the filter logic
|
|
const requests: Array<{ id: string; detectedKind: string }> = [];
|
|
assert.equal(requests.length, 0);
|
|
});
|
|
|
|
it("applies llm profile filter correctly", () => {
|
|
const applyFilter = (req: { detectedKind?: string }, profile: string) => {
|
|
if (profile === "llm" && req.detectedKind !== "llm") return false;
|
|
return true;
|
|
};
|
|
|
|
assert.equal(applyFilter({ detectedKind: "llm" }, "llm"), true);
|
|
assert.equal(applyFilter({ detectedKind: "app" }, "llm"), false);
|
|
assert.equal(applyFilter({ detectedKind: "unknown" }, "llm"), false);
|
|
assert.equal(applyFilter({ detectedKind: "app" }, "all"), true);
|
|
});
|
|
|
|
it("applies host filter correctly", () => {
|
|
const applyFilter = (req: { host: string }, hostFilter?: string) => {
|
|
if (hostFilter && !req.host.includes(hostFilter)) return false;
|
|
return true;
|
|
};
|
|
|
|
assert.equal(applyFilter({ host: "api.openai.com" }, "openai"), true);
|
|
assert.equal(applyFilter({ host: "api.anthropic.com" }, "openai"), false);
|
|
assert.equal(applyFilter({ host: "api.openai.com" }, undefined), true);
|
|
});
|
|
|
|
it("applies status filter 2xx correctly", () => {
|
|
const applyStatusFilter = (status: number | string, filter?: string): boolean => {
|
|
if (!filter) return true;
|
|
if (typeof status === "number") {
|
|
const cat = `${Math.floor(status / 100)}xx`;
|
|
return cat === filter;
|
|
}
|
|
return filter === "error" && status === "error";
|
|
};
|
|
|
|
assert.equal(applyStatusFilter(200, "2xx"), true);
|
|
assert.equal(applyStatusFilter(201, "2xx"), true);
|
|
assert.equal(applyStatusFilter(404, "2xx"), false);
|
|
assert.equal(applyStatusFilter(500, "5xx"), true);
|
|
assert.equal(applyStatusFilter("error", "error"), true);
|
|
assert.equal(applyStatusFilter("error", "2xx"), false);
|
|
});
|
|
|
|
it("backoff doubles on reconnect up to max", () => {
|
|
const INITIAL = 500;
|
|
const MAX = 30_000;
|
|
const MULT = 2;
|
|
|
|
let backoff = INITIAL;
|
|
const delays: number[] = [];
|
|
|
|
for (let i = 0; i < 10; i++) {
|
|
delays.push(Math.min(backoff, MAX));
|
|
backoff = Math.min(backoff * MULT, MAX);
|
|
}
|
|
|
|
assert.equal(delays[0], 500);
|
|
assert.equal(delays[1], 1000);
|
|
assert.equal(delays[2], 2000);
|
|
// Eventually capped at MAX
|
|
const maxDelay = delays[delays.length - 1];
|
|
assert.ok(maxDelay <= MAX, `Expected max delay ${MAX}, got ${maxDelay}`);
|
|
});
|
|
|
|
it("handles snapshot event correctly", () => {
|
|
const requests: Array<{ id: string; detectedKind: string; host: string }> = [];
|
|
|
|
const snapshot = [
|
|
{ id: "1", detectedKind: "llm", host: "api.openai.com" },
|
|
{ id: "2", detectedKind: "app", host: "example.com" },
|
|
];
|
|
|
|
// Simulate snapshot handling with llm profile filter
|
|
const applyFilter = (req: { detectedKind: string }) => req.detectedKind === "llm";
|
|
requests.push(...snapshot.filter(applyFilter));
|
|
|
|
assert.equal(requests.length, 1);
|
|
assert.equal(requests[0].id, "1");
|
|
});
|
|
|
|
it("handles new event with deduplication up to 1000", () => {
|
|
const requests: string[] = [];
|
|
const maxSize = 1000;
|
|
|
|
// Simulate adding 1001 items
|
|
for (let i = 0; i <= maxSize; i++) {
|
|
requests.unshift(`req-${i}`);
|
|
if (requests.length > maxSize) requests.splice(maxSize);
|
|
}
|
|
|
|
assert.equal(requests.length, maxSize);
|
|
assert.equal(requests[0], `req-${maxSize}`);
|
|
});
|
|
|
|
it("handles update event correctly", () => {
|
|
const requests = [
|
|
{ id: "1", status: "in-flight" },
|
|
{ id: "2", status: 200 },
|
|
];
|
|
|
|
const update = { id: "1", status: 200 };
|
|
const updated = requests.map((r) => (r.id === update.id ? { ...r, ...update } : r));
|
|
|
|
assert.equal(updated[0].status, 200);
|
|
assert.equal(updated[1].status, 200);
|
|
});
|
|
|
|
it("handles clear event", () => {
|
|
let requests = [{ id: "1" }, { id: "2" }];
|
|
requests = [];
|
|
assert.equal(requests.length, 0);
|
|
});
|
|
|
|
it("buffers events when paused", () => {
|
|
const pending: Array<{ id: string }> = [];
|
|
const paused = true;
|
|
|
|
const newEvent = { id: "3", type: "new" };
|
|
if (paused) pending.push({ id: newEvent.id });
|
|
|
|
assert.equal(pending.length, 1);
|
|
assert.equal(pending[0].id, "3");
|
|
});
|
|
|
|
it("TrafficStreamState interface includes pendingCount field (R5-9)", () => {
|
|
assert.ok(
|
|
HOOK_SRC.includes("pendingCount"),
|
|
"TrafficStreamState should expose pendingCount"
|
|
);
|
|
});
|
|
|
|
it("pendingCount increments when paused and new event arrives (R5-9)", () => {
|
|
// Verify the source contains the setPendingCount call when pushing to pendingRef
|
|
assert.ok(
|
|
HOOK_SRC.includes("setPendingCount(pendingRef.current.length)"),
|
|
"should call setPendingCount when adding to pendingRef"
|
|
);
|
|
});
|
|
|
|
it("pendingCount resets to 0 on resume (R5-9)", () => {
|
|
assert.ok(
|
|
HOOK_SRC.includes("setPendingCount(0)"),
|
|
"should reset pendingCount to 0 on resume and clear"
|
|
);
|
|
});
|
|
});
|