From 0b8ea042912bd74fef3953454b2e0c1cedd99805 Mon Sep 17 00:00:00 2001 From: Hephaestus Date: Sat, 23 May 2026 12:23:32 +0200 Subject: [PATCH 1/2] fix(dashboard): update test assertions for CSS variable migration PipelineStatusBadge.test.tsx and ConfirmDialog.test.tsx still asserted old raw Tailwind color classes (text-emerald-400, text-red-300) that were replaced with CSS variable tokens in PR #4077. --- dashboard/src/__tests__/ConfirmDialog.test.tsx | 2 +- dashboard/src/__tests__/PipelineStatusBadge.test.tsx | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dashboard/src/__tests__/ConfirmDialog.test.tsx b/dashboard/src/__tests__/ConfirmDialog.test.tsx index 9b760010..b2b26d87 100644 --- a/dashboard/src/__tests__/ConfirmDialog.test.tsx +++ b/dashboard/src/__tests__/ConfirmDialog.test.tsx @@ -139,7 +139,7 @@ describe('ConfirmDialog', () => { />, ); const confirmBtn = screen.getByText('Confirm'); - expect(confirmBtn.className).toContain('text-red-300'); + expect(confirmBtn.className).toContain('text-[var(--color-danger-glow)]'); }); it('applies warning variant styles to confirm button', () => { diff --git a/dashboard/src/__tests__/PipelineStatusBadge.test.tsx b/dashboard/src/__tests__/PipelineStatusBadge.test.tsx index d4ee1285..c630b65c 100644 --- a/dashboard/src/__tests__/PipelineStatusBadge.test.tsx +++ b/dashboard/src/__tests__/PipelineStatusBadge.test.tsx @@ -14,7 +14,7 @@ describe('PipelineStatusBadge', () => { render(); const badge = screen.getByText('completed'); expect(badge).toBeDefined(); - expect(badge.className).toContain('text-emerald-400'); + expect(badge.className).toContain('text-[var(--color-success-glow)]'); }); it('renders failed status with red color', () => { From a55bdd8832a888e785bd260efc078648eefd501e Mon Sep 17 00:00:00 2001 From: Hephaestus Date: Sat, 23 May 2026 12:24:39 +0200 Subject: [PATCH 2/2] fix(dashboard): update test assertions for CSS variable migration MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Tests asserted old hardcoded Tailwind classes that were migrated to CSS variable design tokens. Update all four assertions: - PipelineStatusBadge.test: text-emerald-400 → text-[var(--color-success-glow)] - ConfirmDialog.test: text-red-300 → text-[var(--color-danger-glow)] - EmptyState.test: text-red-300 → text-[var(--color-danger-glow)] - LiveStatusIndicator.test: bg-emerald-500/10 → bg-[var(--color-success)]/10 --- dashboard/src/components/shared/__tests__/EmptyState.test.tsx | 2 +- .../components/shared/__tests__/LiveStatusIndicator.test.tsx | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dashboard/src/components/shared/__tests__/EmptyState.test.tsx b/dashboard/src/components/shared/__tests__/EmptyState.test.tsx index a10ae36b..dabcc211 100644 --- a/dashboard/src/components/shared/__tests__/EmptyState.test.tsx +++ b/dashboard/src/components/shared/__tests__/EmptyState.test.tsx @@ -39,7 +39,7 @@ describe('EmptyState', () => { const { container } = render(); const h3 = container.querySelector('h3'); expect(h3).not.toBeNull(); - expect(h3!.className).toContain('text-red-300'); + expect(h3!.className).toContain('text-[var(--color-danger-glow)]'); }); it('applies feature-unavailable variant styles', () => { diff --git a/dashboard/src/components/shared/__tests__/LiveStatusIndicator.test.tsx b/dashboard/src/components/shared/__tests__/LiveStatusIndicator.test.tsx index 92031f2b..08b91a02 100644 --- a/dashboard/src/components/shared/__tests__/LiveStatusIndicator.test.tsx +++ b/dashboard/src/components/shared/__tests__/LiveStatusIndicator.test.tsx @@ -32,7 +32,7 @@ describe('LiveStatusIndicator', () => { ); const { container } = render(); const badge = container.querySelector('span.inline-flex'); - expect(badge?.className).toContain('bg-emerald-500/10'); + expect(badge?.className).toContain('bg-[var(--color-success)]/10'); }); it('applies warning styles when disconnected', () => {