All tests passing

Author: VictorioBerra

tests/main.spec.js

@@ -2,8 +2,6 @@
 import { test, expect } from '@playwright/test';
 const mailhog = require('mailhog')();
 
-const host = 'http://localhost:8090';
-
 const TEST_EMAIL = 'testuser@example.com';
 const TEST_PASSWORD = 'test_password1234';
 
@@ -13,14 +11,7 @@ let NEW_EMAIL = 'changeduser@example.com';
 
 // Helper to extract the body from a MailHog message
 function getMailBody(result) {
-  if (!result) return '';
-  // Try common MailHog API shapes
-  if (result.Content && result.Content.Body) return result.Content.Body;
-  if (result.Body) return result.Body;
-  if (result.MIME && result.MIME.Body) return result.MIME.Body;
-  if (result.html && typeof result.html === 'string') return result.html;
-  if (result.text && typeof result.text === 'string') return result.text;
-  return '';
+  return result.Content.Body;
 }
 
 function extractVerifyLinkFromBody(body) {
@@ -30,8 +21,20 @@ function extractVerifyLinkFromBody(body) {
   return match ? match[1] : null;
 }
 
+function extractConfirmEmailLinkFromBody(body) {
+  const cleanedBody = body.replace(/=\r?\n/g, '');
+  const match = cleanedBody.match(/\[Confirm new email\]\(([^)]+)\)/i);
+  return match ? match[1] : null;
+}
+
+function extractResetPasswordLinkFromBody(body) {
+  const cleanedBody = body.replace(/=\r?\n/g, '');
+  const match = cleanedBody.match(/\[Reset password\]\(([^)]+)\)/i);
+  return match ? match[1] : null;
+}
+
 async function login(page, email, password = LATEST_PASSWORD) {
-  await page.goto(`${host}/auth/login`);
+  await page.goto(`/auth/login`);
   await page.fill('input[name="identity"]', email);
   await page.fill('input[name="password"]', password);
   await page.click('button[type="submit"]');
@@ -43,7 +46,7 @@ async function expectLoggedIn(page) {
 }
 
 test('register an account (registration only)', async ({ page }) => {
-  await page.goto(`${host}/auth/register`);
+  await page.goto(`/auth/register`);
 
   // Fill in the registration form
   await page.fill('input[name="identity"]', TEST_EMAIL);
@@ -89,7 +92,7 @@ test('login with verified account', async ({ page, context }) => {
 test('Change password', async ({ page, context }) => {
   await login(page, TEST_EMAIL);
 
-  await page.goto(`${host}/account/change-password`);
+  await page.goto(`/account/change-password`);
   await page.fill('input[name="currentPassword"]', LATEST_PASSWORD);
   const newPassword = 'new_password_' + Date.now();
   await page.fill('input[name="newPassword"]', newPassword);
@@ -109,7 +112,7 @@ test('Login with changed password', async ({ page, context }) => {
 
 test('Change email', async ({ page, context }) => {
   await login(page, TEST_EMAIL, LATEST_PASSWORD);
-  await page.goto(`${host}/account/change-email`);
+  await page.goto(`/account/change-email`);
   await page.fill('input[name="identity"]', NEW_EMAIL);
   await page.click('button[type="submit"]');
   const flashContainer = await page.getByTestId('flashContainer');
@@ -122,16 +125,13 @@ test('Confirm Change email', async ({ page, context }) => {
   expect(result).not.toBeNull();
 
   const body = getMailBody(result);
-  const match = body.match(/\[Confirm new email\]\(([^)]+)\)/i);
-  expect(match).not.toBeNull();
+  const confirmLink = extractConfirmEmailLinkFromBody(body);
+  expect(confirmLink).not.toBeNull();
 
-  const confirmLink = match[1];
   await page.goto(confirmLink);
   await page.fill('input[name="password"]', LATEST_PASSWORD);
   await page.click('button[type="submit"]');
 
-  await page.screenshot({ path: 'screenshot.png' });
-
   const flashContainer = await page.getByTestId('flashContainer');
   await expect(flashContainer).toHaveText("Your email has been changed. You will need to login again.");
 });
@@ -142,17 +142,18 @@ test('Login with changed email', async ({ page, context }) => {
 });
 
 test('Logout', async ({ page, context }) => {
-  await page.goto(`${host}/account/logout`);
+  await page.goto(`/account/logout`);
 
   const cookies = await context.cookies();
   const pbAuthCookie = cookies.find(c => c.name === 'pb_auth');
   expect(pbAuthCookie).toBeUndefined();
 });
 
 test('Send Forgot Password Reset', async ({ page, context }) => {
-  await page.goto(`${host}/auth/forgot`);
+  await page.goto(`/auth/forgot`);
   await page.fill('input[name="identity"]', NEW_EMAIL);
   await page.click('button[type="submit"]');
+
   const flashContainer = await page.getByTestId('flashContainer');
   await expect(flashContainer).toHaveText("Password reset email sent, please check your inbox.");
 });
@@ -162,14 +163,17 @@ test('Confirm Forgot Password Reset', async ({ page, context }) => {
   const RESET_PASSWORD = 'reset_password_9999';
   const result = await mailhog.latestTo(NEW_EMAIL);
   expect(result).not.toBeNull();
+
   const body = getMailBody(result);
-  const match = body.match(/\[Reset Password\]\(([^)]+)\)/i);
-  expect(match).not.toBeNull();
-  const resetLink = match[1];
-  await page.goto(resetLink);
-  await page.fill('input[name="newPassword"]', RESET_PASSWORD);
+  const resetPasswordLink = extractResetPasswordLinkFromBody(body);
+  expect(resetPasswordLink).not.toBeNull();
+
+  await page.goto(resetPasswordLink);
+
+  await page.fill('input[name="password"]', RESET_PASSWORD);
   await page.fill('input[name="confirmPassword"]', RESET_PASSWORD);
   await page.click('button[type="submit"]');
+
   LATEST_PASSWORD = RESET_PASSWORD;
   const flashContainer = await page.getByTestId('flashContainer');
   await expect(flashContainer).toHaveText("Your password has been reset, you can now login.");