mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-08-01 04:42:10 +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.
95 lines
3.0 KiB
TypeScript
95 lines
3.0 KiB
TypeScript
/**
|
|
* Asserts that StatsTab lazy-loads StatsCharts via next/dynamic (ssr: false)
|
|
* and does NOT statically import anything from "recharts".
|
|
*/
|
|
import { describe, it } from "vitest";
|
|
import assert from "node:assert/strict";
|
|
import { readFileSync } from "node:fs";
|
|
import { fileURLToPath } from "node:url";
|
|
import path from "node:path";
|
|
|
|
const __dirname = path.dirname(fileURLToPath(import.meta.url));
|
|
const TABS_DIR = path.resolve(
|
|
__dirname,
|
|
"../../../src/app/(dashboard)/dashboard/tools/traffic-inspector/components/tabs",
|
|
);
|
|
|
|
const statsTabSrc = readFileSync(path.join(TABS_DIR, "StatsTab.tsx"), "utf-8");
|
|
const statsChartsSrc = readFileSync(path.join(TABS_DIR, "StatsCharts.tsx"), "utf-8");
|
|
|
|
describe("StatsTab — C3 lazy bundle split", () => {
|
|
it("imports dynamic from next/dynamic", () => {
|
|
assert.ok(
|
|
statsTabSrc.includes('import dynamic from "next/dynamic"'),
|
|
"StatsTab must import dynamic from next/dynamic",
|
|
);
|
|
});
|
|
|
|
it("calls dynamic() at module level with import('./StatsCharts')", () => {
|
|
assert.ok(
|
|
statsTabSrc.includes("dynamic(() => import(\"./StatsCharts\")"),
|
|
"StatsTab must call dynamic(() => import('./StatsCharts')) at module level",
|
|
);
|
|
});
|
|
|
|
it("passes ssr: false to dynamic()", () => {
|
|
assert.ok(
|
|
statsTabSrc.includes("ssr: false"),
|
|
"StatsTab dynamic() call must include ssr: false",
|
|
);
|
|
});
|
|
|
|
it("does NOT statically import any name from recharts", () => {
|
|
// A static import of recharts would look like: import ... from "recharts"
|
|
assert.ok(
|
|
!statsTabSrc.includes("from \"recharts\""),
|
|
"StatsTab must not contain a static import from recharts",
|
|
);
|
|
});
|
|
|
|
it("does NOT contain a useEffect that imports recharts", () => {
|
|
assert.ok(
|
|
!statsTabSrc.includes("import(\"recharts\")"),
|
|
"StatsTab must not dynamically import recharts via useEffect",
|
|
);
|
|
});
|
|
|
|
it("does NOT contain the discarded _rechartsPreload no-op pattern", () => {
|
|
assert.ok(
|
|
!statsTabSrc.includes("_rechartsPreload"),
|
|
"StatsTab must not contain the orphaned _rechartsPreload variable",
|
|
);
|
|
});
|
|
});
|
|
|
|
describe("StatsCharts — recharts imports live here", () => {
|
|
it("imports recharts components statically", () => {
|
|
assert.ok(
|
|
statsChartsSrc.includes("from \"recharts\""),
|
|
"StatsCharts must statically import from recharts",
|
|
);
|
|
});
|
|
|
|
it("imports ResponsiveContainer from recharts", () => {
|
|
assert.ok(
|
|
statsChartsSrc.includes("ResponsiveContainer"),
|
|
"StatsCharts must import ResponsiveContainer",
|
|
);
|
|
});
|
|
|
|
it("imports BarChart from recharts", () => {
|
|
assert.ok(statsChartsSrc.includes("BarChart"), "StatsCharts must import BarChart");
|
|
});
|
|
|
|
it("imports LineChart from recharts", () => {
|
|
assert.ok(statsChartsSrc.includes("LineChart"), "StatsCharts must import LineChart");
|
|
});
|
|
|
|
it("exports a default component", () => {
|
|
assert.ok(
|
|
statsChartsSrc.includes("export default function StatsCharts"),
|
|
"StatsCharts must export a default function",
|
|
);
|
|
});
|
|
});
|