Skip to content

Commit d3cc147

Browse files
committed
test: align responsive E2E test with actual mobile nav structure
"Toggle theme" doesn't exist as a standalone button in mobile nav — it's inside the More dropdown. Also DropdownMenuContent has no aria-label. Replace "Toggle theme" assertion with "More" button and remove name filter from menu role query.
1 parent ee9d679 commit d3cc147

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

apps/web/e2e/responsive.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ test.describe("Responsive Layout", () => {
1515

1616
// Verify mobile nav elements
1717
await expect(page.getByRole("link", { name: "Explore" })).toBeVisible();
18-
await expect(page.getByRole("button", { name: "Toggle theme" })).toBeVisible();
18+
await expect(page.getByRole("button", { name: "More" })).toBeVisible();
1919
await expect(page.getByRole("button", { name: "Sign in" })).toBeVisible();
2020
});
2121

@@ -33,7 +33,7 @@ test.describe("Responsive Layout", () => {
3333
await moreButton.click();
3434

3535
// Verify menu appears with Docs and Pricing
36-
const menu = page.getByRole("menu", { name: "More" });
36+
const menu = page.getByRole("menu");
3737
await expect(menu).toBeVisible();
3838

3939
// Verify Docs menuitem

0 commit comments

Comments
 (0)