From de2b220c36d3f4cf3602eafed2cb3cad611392d4 Mon Sep 17 00:00:00 2001 From: Yuneng Jiang Date: Sat, 15 Aug 2026 17:50:40 -0700 Subject: [PATCH] test(e2e/ui): assert the log drawer chevrons by their lucide classes The log details drawer moved off Ant Design in 03d2b16bc, so its section header renders lucide ChevronUp/ChevronDown rather than antd's UpOutlined and DownOutlined. The collapse test still waited on .anticon-up and .anticon-down, which no longer exist anywhere under view_logs, so it failed on every run and burned all three attempts identically. Point the three assertions at .lucide-chevron-up and .lucide-chevron-down, matching how the dashboard's other suites address lucide icons. --- tests/e2e/ui/tests/logs/logs.spec.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/e2e/ui/tests/logs/logs.spec.ts b/tests/e2e/ui/tests/logs/logs.spec.ts index c1182a7ca5c..b29a72cbf81 100644 --- a/tests/e2e/ui/tests/logs/logs.spec.ts +++ b/tests/e2e/ui/tests/logs/logs.spec.ts @@ -129,17 +129,17 @@ test.describe("Logs page", () => { // box, so the wrapper reads as hidden while the clipped text node inside it does not. const header = sectionHeader(drawer, "Input"); const body = header.locator("xpath=following-sibling::div[1]"); - await expect(header.locator(".anticon-up")).toBeVisible(); + await expect(header.locator(".lucide-chevron-up")).toBeVisible(); await expect(body).toBeVisible(); await header.click(); - await expect(header.locator(".anticon-down")).toBeVisible({ + await expect(header.locator(".lucide-chevron-down")).toBeVisible({ timeout: 10_000, }); await expect(body).toBeHidden({ timeout: 10_000 }); await header.click(); - await expect(header.locator(".anticon-up")).toBeVisible({ + await expect(header.locator(".lucide-chevron-up")).toBeVisible({ timeout: 10_000, }); await expect(body).toBeVisible({ timeout: 10_000 });