fix(e2e): address CTO/QA review — remove mock-incompatible test, fix smoke test, fix a11y

Co-Authored-By: Paperclip <noreply@paperclip.ing>

e2e/smoke.spec.ts

@@ -1,8 +1,8 @@
 import { test, expect } from './fixtures';
 
-test("app loads", async ({ page }) => {
-  await page.goto("/");
-  await expect(page).toHaveTitle(/CartSnitch/);
-  // Unauthenticated users are redirected to login
+test('app loads', async ({ page }) => {
+  await page.goto('/');
+  // Unauthenticated users are redirected to /login
   await expect(page).toHaveURL(/\/login/);
+  await expect(page.getByRole('heading', { name: /sign in|log in/i })).toBeVisible();
 });

src/pages/Login.tsx

@@ -46,7 +46,7 @@ export function Login() {
   }
 
   return (
-    <div className="flex min-h-screen flex-col items-center justify-center px-4">
+    <main className="flex min-h-screen flex-col items-center justify-center px-4">
       <h1 className="mb-2 text-3xl font-bold text-gray-900">CartSnitch</h1>
       <p className="mb-8 text-sm text-gray-500">Track prices. Save money.</p>
 
@@ -92,6 +92,6 @@ export function Login() {
           Sign up
         </Link>
       </p>
-    </div>
+    </main>
   )
 }