commit 807a793aa10a01ce4031feb77042b5fcfc0f765b
parent 425cd7102957063690c6b06e37fc07c4abda5634
Author: I Mean I'm Just Saying <imeanimjustsaying@kiwifarms.st>
Date: Mon, 21 Sep 2026 03:08:31 -0400
e2e: Next's route announcer is also a role="alert"
`page.getByRole("alert")` matched the refusal AND
`__next-route-announcer__`, which every page ships. A strict-mode violation
there reads as "the message never appeared", which is the opposite of what
happened. Filtered by the text the assertion is about.
channel-storage.spec.ts: 7/7.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Diffstat:
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/editor/e2e/channel-storage.spec.ts b/editor/e2e/channel-storage.spec.ts
@@ -657,7 +657,12 @@ test("a move to an unmounted root refuses before it creates anything", async ({
// location id into the refusal.
await page.getByLabel("destination location").selectOption("cold");
await page.getByRole("button", { name: "Preview" }).click();
- const alert = page.getByRole("alert");
+ // FILTERED, because Next ships its own `role="alert"` route announcer on
+ // every page and a bare getByRole("alert") is a strict-mode violation that
+ // reads as "the message never appeared".
+ const alert = page
+ .getByRole("alert")
+ .filter({ hasText: /destination root/ });
await expect(alert).toContainText(/does not exist or is not a directory/, {
timeout: 15_000,
});