From 7ecfe9fe0b7ddbe5888ab14924104ac9b08f019b Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 25 Dec 2025 13:57:39 +0000 Subject: [PATCH] Fix shared-components test to use correct JSON field names The validate-dimensions-coverage.sh script outputs `total_critical_missing` at the root level and `missing` under `files[0]` to support multi-file mode, but the test was looking for `critical_missing` and `missing` at the root. --- tests/e2e/shared-components.test.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/e2e/shared-components.test.ts b/tests/e2e/shared-components.test.ts index 2d50102..86f5363 100644 --- a/tests/e2e/shared-components.test.ts +++ b/tests/e2e/shared-components.test.ts @@ -477,10 +477,12 @@ describe('Shared + Validate Integration', () => { const validateData = JSON.parse(validateResult); // Should report the unique element as missing (critical) - expect(validateData.critical_missing).toBe(1); + // The script outputs total_critical_missing at root level for multi-file support + expect(validateData.total_critical_missing).toBe(1); // Should NOT include the shared exit button instances in missing - const missingIds = validateData.missing?.map((m: { id: string }) => m.id) || []; + // Missing IDs are nested under files[0].missing for multi-file support + const missingIds = validateData.files?.[0]?.missing?.map((m: { id: string }) => m.id) || []; expect(missingIds).not.toContain('I237:2417;2708:1961'); expect(missingIds).not.toContain('I237:2572;2708:1961');