fix: resolve 6 failing Playwright E2E tests

- Add custom "Invalid email address" validation message to Zod schemas
  in login, signup, and recover-password routes (fixes 4 tests)
- Wait for success toast before polling mailcatcher in password reset
  tests (fixes 2 tests)
- Fix sign-up with existing email test to wait for first signup and
  use proper toBeVisible() assertion (fixes 1 test)
- Fix signUpNewUser utility to wait for signup completion before
  navigating

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: fitchmultz

frontend/src/routes/login.tsx

@@ -23,7 +23,7 @@ import { PasswordInput } from "@/components/ui/password-input"
 import useAuth, { isLoggedIn } from "@/hooks/useAuth"
 
 const formSchema = z.object({
-  username: z.email(),
+  username: z.email({ message: "Invalid email address" }),
   password: z
     .string()
     .min(1, { message: "Password is required" })

frontend/src/routes/recover-password.tsx

@@ -25,7 +25,7 @@ import useCustomToast from "@/hooks/useCustomToast"
 import { handleError } from "@/utils"
 
 const formSchema = z.object({
-  email: z.email(),
+  email: z.email({ message: "Invalid email address" }),
 })
 
 type FormData = z.infer<typeof formSchema>

frontend/src/routes/signup.tsx

@@ -22,7 +22,7 @@ import useAuth, { isLoggedIn } from "@/hooks/useAuth"
 
 const formSchema = z
   .object({
-    email: z.email(),
+    email: z.email({ message: "Invalid email address" }),
     full_name: z.string().min(1, { message: "Full Name is required" }),
     password: z
       .string()

frontend/tests/reset-password.spec.ts

@@ -43,6 +43,9 @@ test("User can reset password successfully using the link", async ({
   await page.getByTestId("email-input").fill(email)
 
   await page.getByRole("button", { name: "Continue" }).click()
+  await expect(
+    page.getByText("Password recovery email sent successfully"),
+  ).toBeVisible()
 
   const emailData = await findLastEmail({
     request,
@@ -98,6 +101,9 @@ test("Weak new password validation", async ({ page, request }) => {
   await page.goto("/recover-password")
   await page.getByTestId("email-input").fill(email)
   await page.getByRole("button", { name: "Continue" }).click()
+  await expect(
+    page.getByText("Password recovery email sent successfully"),
+  ).toBeVisible()
 
   const emailData = await findLastEmail({
     request,

frontend/tests/sign-up.spec.ts

@@ -77,19 +77,21 @@ test("Sign up with existing email", async ({ page }) => {
 
   // Sign up with an email
   await page.goto("/signup")
-
   await fillForm(page, fullName, email, password, password)
   await page.getByRole("button", { name: "Sign Up" }).click()
 
+  // Wait for first signup to complete (navigates to /login on success)
+  await page.waitForURL("/login")
+
   // Sign up again with the same email
   await page.goto("/signup")
-
   await fillForm(page, fullName, email, password, password)
   await page.getByRole("button", { name: "Sign Up" }).click()
 
-  await page
-    .getByText("The user with this email already exists in the system")
-    .click()
+  // Properly assert error message is visible
+  await expect(
+    page.getByText("The user with this email already exists in the system"),
+  ).toBeVisible()
 })
 
 test("Sign up with weak password", async ({ page }) => {

frontend/tests/utils/user.ts

@@ -13,7 +13,9 @@ export async function signUpNewUser(
   await page.getByTestId("password-input").fill(password)
   await page.getByTestId("confirm-password-input").fill(password)
   await page.getByRole("button", { name: "Sign Up" }).click()
-  await page.goto("/login")
+
+  // Wait for signup to complete (redirects to /login on success)
+  await page.waitForURL("/login")
 }
 
 export async function logInUser(page: Page, email: string, password: string) {