7f1b7e21c1
Extend CLI quarantine validation across the entire ledger. - Normalize package-relative Vitest exclusions to repository paths. - Require each CLI configuration exclusion and ledger entry exactly once. - Validate ISO 8601 quarantine timestamps. Files changed: .../src/__tests__/lockstep-cli-quarantine.test.ts | 52 +++++++++++----------- packages/cli/vitest.config.ts | 4 +- 2 files changed, 27 insertions(+), 29 deletions(-) Fusion-Task-Id: FN-8223 Fusion-Task-Lineage: 7abe719b-63bf-4dfa-8db6-e3dd7547c7fe Co-authored-by: Fusion (runfusion.ai) <noreply@runfusion.ai>
63 lines
2.6 KiB
TypeScript
63 lines
2.6 KiB
TypeScript
import { readFileSync } from "node:fs";
|
|
import { resolve } from "node:path";
|
|
import { describe, expect, it } from "vitest";
|
|
|
|
type QuarantineEntry = {
|
|
file: string;
|
|
reason: string;
|
|
quarantinedAt: string;
|
|
};
|
|
|
|
const repoRoot = resolve(import.meta.dirname!, "../../../..");
|
|
const configPath = resolve(repoRoot, "packages/cli/vitest.config.ts");
|
|
const ledgerPath = resolve(repoRoot, "scripts/lib/test-quarantine.json");
|
|
const cliPathPrefix = "packages/cli/";
|
|
const iso8601Timestamp = /^\d{4}-\d{2}-\d{2}(?:T\d{2}:\d{2}:\d{2}(?:\.\d+)?(?:Z|[+-]\d{2}:\d{2}))?$/;
|
|
|
|
function parseQuarantinedCliTests(configSource: string): string[] {
|
|
const declaration = configSource.match(/const quarantinedCliTests: string\[\] = \[([\s\S]*?)\n\];/);
|
|
expect(declaration, "quarantinedCliTests declaration must remain statically parseable").not.toBeNull();
|
|
|
|
return [...declaration![1].matchAll(/"([^"\n]+)"/g)].map((match) => match[1]);
|
|
}
|
|
|
|
function countByPath(paths: string[]): Map<string, number> {
|
|
return paths.reduce((counts, path) => counts.set(path, (counts.get(path) ?? 0) + 1), new Map<string, number>());
|
|
}
|
|
|
|
function normalizeConfigPath(path: string): string {
|
|
return `${cliPathPrefix}${path}`;
|
|
}
|
|
|
|
describe("CLI quarantine ledger lockstep", () => {
|
|
/*
|
|
FNXC:CliTests 2026-07-17-10:45:
|
|
FN-8223 widens this guard to the full CLI quarantine surface after FN-8219
|
|
deleted its five expired paths and FN-8210 rescued package-config.test.ts.
|
|
Every package-relative Vitest exclude and repo-relative packages/cli ledger
|
|
row must now match exactly once, preventing future one-sided quarantine drift.
|
|
*/
|
|
it("keeps all CLI config excludes and ledger rows in bidirectional lockstep", () => {
|
|
const configPaths = parseQuarantinedCliTests(readFileSync(configPath, "utf8")).map(normalizeConfigPath);
|
|
const ledger = JSON.parse(readFileSync(ledgerPath, "utf8")) as { entries: QuarantineEntry[] };
|
|
const ledgerEntries = ledger.entries.filter((entry) => entry.file.startsWith(cliPathPrefix));
|
|
const ledgerPaths = ledgerEntries.map((entry) => entry.file);
|
|
const configCounts = countByPath(configPaths);
|
|
const ledgerCounts = countByPath(ledgerPaths);
|
|
|
|
for (const count of configCounts.values()) {
|
|
expect(count).toBe(1);
|
|
}
|
|
for (const count of ledgerCounts.values()) {
|
|
expect(count).toBe(1);
|
|
}
|
|
expect(configCounts).toEqual(ledgerCounts);
|
|
|
|
for (const entry of ledgerEntries) {
|
|
expect(entry.reason.trim()).not.toBe("");
|
|
expect(entry.quarantinedAt).toMatch(iso8601Timestamp);
|
|
expect(Number.isNaN(Date.parse(entry.quarantinedAt))).toBe(false);
|
|
}
|
|
});
|
|
});
|