From 58cd8458fd067469437585c8f995bc22b59ef58e Mon Sep 17 00:00:00 2001 From: Brennan Benson <79079362+brennanb2025@users.noreply.github.com> Date: Thu, 13 Aug 2026 11:41:41 -0700 Subject: [PATCH] test(dashboard): retarget the agent-row question assertion at the shared token MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit DashboardAgentRow renders the glyph through AgentStateDot, so the component already moved with the token — only its assertion still pinned text-amber-500. Missed locally because I ran components/dashboard-popout but not components/dashboard. --- .../src/components/dashboard/DashboardAgentRow.test.tsx | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/renderer/src/components/dashboard/DashboardAgentRow.test.tsx b/src/renderer/src/components/dashboard/DashboardAgentRow.test.tsx index 93238b8fc56..0fb978de62b 100644 --- a/src/renderer/src/components/dashboard/DashboardAgentRow.test.tsx +++ b/src/renderer/src/components/dashboard/DashboardAgentRow.test.tsx @@ -247,13 +247,14 @@ describe('DashboardAgentRow', () => { expect(classes.every((className) => !/\bgroup-hover:/.test(className))).toBe(true) }) - it('renders waiting rows with the amber question glyph', () => { + it('renders waiting rows with the shared question glyph', () => { const markup = renderRow(makeAgent({}, { state: 'waiting' })) const tokens = classTokens(markup) expect(markup).toContain('aria-label="Waiting for input"') expect(markup).toContain('lucide-message-circle-question-mark') - expect(tokens).toContain('text-amber-500') + expect(tokens).toContain('text-agent-question') + expect(tokens).not.toContain('text-amber-500') expect(tokens).not.toContain('bg-red-500') })