From e87a8220a6ce10dc86a4ddea2e3bb5ab9b3d9ce9 Mon Sep 17 00:00:00 2001 From: Claude Date: Sat, 31 Jan 2026 07:01:33 +0000 Subject: [PATCH] fix: resolve test file type errors - Add missing 'vi' import in techTree.test.ts - Fix mockHealth in unitBehaviors.test.ts by moving spread before mock functions to prevent overwriting, and remove cast to preserve types https://claude.ai/code/session_01665xH61KterutQ8q8iP8Qo --- tests/data/techTree.test.ts | 2 +- tests/engine/ai/unitBehaviors.test.ts | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/data/techTree.test.ts b/tests/data/techTree.test.ts index a8fcf8f2..a1660dc2 100644 --- a/tests/data/techTree.test.ts +++ b/tests/data/techTree.test.ts @@ -1,4 +1,4 @@ -import { describe, it, expect } from 'vitest'; +import { describe, it, expect, vi } from 'vitest'; import { BUILDING_DISPLAY_NAMES, BUILDING_RESEARCH_MAP, diff --git a/tests/engine/ai/unitBehaviors.test.ts b/tests/engine/ai/unitBehaviors.test.ts index 81902de2..808b811a 100644 --- a/tests/engine/ai/unitBehaviors.test.ts +++ b/tests/engine/ai/unitBehaviors.test.ts @@ -74,12 +74,13 @@ function createMockEntity( current: overrides.health?.current ?? 100, max: overrides.health?.max ?? 100, armor: overrides.health?.armor ?? 0, + ...overrides.health, + // Mock functions must be after spread to avoid being overwritten getHealthPercent: vi .fn() .mockReturnValue((overrides.health?.current ?? 100) / (overrides.health?.max ?? 100)), isDead: vi.fn().mockReturnValue(false), - ...overrides.health, - } as unknown as Health; + }; const mockSelectable = { playerId: overrides.selectable?.playerId ?? 'player1',