From b1f8c6645694a83c2a04e2d01cbc06008e7c2be7 Mon Sep 17 00:00:00 2001
From: Lily Shen <115414357+lilyshen0722@users.noreply.github.com>
Date: Sun, 1 Nov 2026 18:46:54 -0800
Subject: [PATCH] test(e2e): wrong-password login waits for the auth notice
 alert

#3297 replaced the login page's .v2-login__error div with V2AuthNotice
(.v2-login__notice, role="alert" for a failure, role="status" for a
confirmation), and Playwright on main has failed this one test since
c1af20e80. The spec now waits for the alert row; requiring role="alert"
keeps a status row from passing for an error.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015AKgpQ2pr6xRMhN5mcfoHF
---
 e2e/auth.spec.ts | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/e2e/auth.spec.ts b/e2e/auth.spec.ts
index 5395b8b351..019e4e154b 100644
--- a/e2e/auth.spec.ts
+++ b/e2e/auth.spec.ts
@@ -38,8 +38,9 @@ test.describe('Authentication', () => {
     await page.getByLabel('Password').fill('wrongpassword');
     await page.getByRole('button', { name: 'Sign in' }).click();
 
-    // v2 login surfaces the failure in a .v2-login__error div
-    await expect(page.locator('.v2-login__error')).toBeVisible({ timeout: 8000 });
+    // v2 login surfaces the failure in the auth notice row as an alert
+    // (V2AuthNotice); a confirmation uses the same row with role="status"
+    await expect(page.locator('.v2-login__notice[role="alert"]')).toBeVisible({ timeout: 8000 });
     // URL must still be the login page — not redirected
     expect(page.url()).toContain('/v2/login');
   });