forked from molecule-ai/molecule-core
fef664d6d0
The budget PR (#541) added budgetLimit: number | null as a required field on WorkspaceNodeData and budget_limit: number | null on WorkspaceData. Seven test fixture factories were not updated, causing tsc --noEmit to produce 34 TS2322/TS2345 errors (runtime tests still passed because Vitest transpiles via esbuild which strips types). Fixes: - canvas-events.test.ts: makeNode factory +budgetLimit: null - canvas-events-pan.test.ts: makeNode factory +budgetLimit: null - canvas-capabilities.test.ts: makeNodeData factory +budgetLimit: null - canvas-topology.test.ts: makeWS factory +budget_limit: null - canvas.test.ts: makeWS factory +budget_limit: null; two inline summarizeWorkspaceCapabilities args +budgetLimit: null; context-menu fixture +budgetLimit: null - ProvisioningTimeout.test.tsx: makeWS factory +budget_limit: null Also fixes 3 TS2348 errors in AuthGate.test.tsx: newer Vitest type defs resolve ReturnType<typeof vi.fn> to Mock<Procedure|Constructable> which TypeScript no longer considers directly callable in a vi.mock factory. Fix: intersect the mock variables with a plain function type so both the call expression and the mock API (mockReturnValue etc.) type-check. tsc --noEmit: 0 errors. npm test: 722/722. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| e2e | ||
| public | ||
| src | ||
| .env.example | ||
| .gitignore | ||
| Dockerfile | ||
| next.config.ts | ||
| package-lock.json | ||
| package.json | ||
| playwright.config.ts | ||
| postcss.config.js | ||
| tailwind.config.ts | ||
| tsconfig.json | ||
| vitest.config.ts | ||