diff --git a/packages/app/e2e/browser/settings-navigation.spec.ts b/packages/app/e2e/browser/settings-navigation.spec.ts index 3ec921a663..77aad8f250 100644 --- a/packages/app/e2e/browser/settings-navigation.spec.ts +++ b/packages/app/e2e/browser/settings-navigation.spec.ts @@ -313,12 +313,12 @@ test.describe("Settings sidebar navigation", () => { await test.step("a dropdown owns Escape", async () => { await openSettingsSection(page, "appearance"); - await page.getByLabel(/Theme:/).click(); - await expect(page.getByRole("menuitem", { name: "System", exact: true })).toBeVisible(); + await page.getByLabel(/Highlight theme:/).click(); + await expect(page.getByRole("menuitem", { name: "GitHub", exact: true })).toBeVisible(); await page.keyboard.press("Escape"); - await expect(page.getByRole("menuitem", { name: "System", exact: true })).toHaveCount(0); + await expect(page.getByRole("menuitem", { name: "GitHub", exact: true })).toHaveCount(0); await expect(page).toHaveURL(/\/settings(\/|$)/); }); diff --git a/packages/app/e2e/mobile/system-bars/android.mjs b/packages/app/e2e/mobile/system-bars/android.mjs index 8201f3dc12..acc7458c01 100644 --- a/packages/app/e2e/mobile/system-bars/android.mjs +++ b/packages/app/e2e/mobile/system-bars/android.mjs @@ -38,12 +38,9 @@ async function expectDarkIcons(expected, context) { assert.equal(windowState(), expected, context); } +// Settings > Appearance shows every theme as a tile in one radio group. function selectTheme(theme) { - const current = ad("snapshot", "-i").match(/"Theme: ([^"]+)"/)?.[1]; - assert(current, "Start on Settings > Appearance"); - ad("press", `label="Theme: ${current}"`, "--settle"); ad("press", `label="${theme}"`, "--settle"); - ad("wait", "text", `Theme: ${theme}`, "5000"); } ad("settings", "appearance", "light");