Fix pre-existing test bugs causing CI failures in auth and PIM tests

Co-authored-by: bg-playground <259109604+bg-playground@users.noreply.github.com>

Author: Copilot

automated-testing/pages/LoginPage.js

@@ -17,8 +17,10 @@ export class LoginPage {
     await this.usernameInput.fill(username);
     await this.passwordInput.fill(password);
     await this.loginButton.click();
-    
-    // Wait for dashboard to be fully loaded - single, reliable wait
+  }
+
+  async loginAndWaitForDashboard(username, password) {
+    await this.login(username, password);
     await this.page.waitForURL(/.*dashboard/, { timeout: 90000 });
     await this.page.getByRole('heading', { name: 'Dashboard' }).waitFor({ state: 'visible', timeout: 90000 });
   }

automated-testing/tests/ui/admin.spec.js

@@ -15,7 +15,7 @@ test.describe('Admin Module Tests', () => {
 
     // Login before each test
     await loginPage.goto();
-    await loginPage.login('Admin', 'admin123');
+    await loginPage.loginAndWaitForDashboard('Admin', 'admin123');
     await expect(page).toHaveURL(/.*dashboard/);
     await dashboardPage.navigateToAdmin();
   });

automated-testing/tests/ui/auth.spec.js

@@ -14,7 +14,7 @@ test.describe('Authentication Tests', () => {
 
   test('TC-AUTH-001: Valid Login', async ({ page }) => {
     await test.step('Enter valid credentials and login', async () => {
-      await loginPage.login('Admin', 'admin123');
+      await loginPage.loginAndWaitForDashboard('Admin', 'admin123');
     });
 
     await test.step('Verify dashboard is displayed', async () => {
@@ -27,6 +27,7 @@ test.describe('Authentication Tests', () => {
   test('TC-AUTH-002: Invalid Username', async ({ page }) => {
     await test.step('Enter invalid username', async () => {
       await loginPage.login('InvalidUser', 'admin123');
+      await loginPage.errorMessage.waitFor({ state: 'visible', timeout: 10000 });
     });
 
     await test.step('Verify error message is displayed', async () => {
@@ -38,6 +39,7 @@ test.describe('Authentication Tests', () => {
   test('TC-AUTH-003: Invalid Password', async ({ page }) => {
     await test.step('Enter invalid password', async () => {
       await loginPage.login('Admin', 'WrongPassword');
+      await loginPage.errorMessage.waitFor({ state: 'visible', timeout: 10000 });
     });
 
     await test.step('Verify error message is displayed', async () => {
@@ -70,7 +72,7 @@ test.describe('Authentication Tests', () => {
 
   test('TC-AUTH-006: Logout Functionality', async ({ page }) => {
     await test.step('Login successfully', async () => {
-      await loginPage.login('Admin', 'admin123');
+      await loginPage.loginAndWaitForDashboard('Admin', 'admin123');
       await expect(page).toHaveURL(/.*dashboard/);
     });
 
@@ -88,6 +90,7 @@ test.describe('Authentication Tests', () => {
   test('TC-AUTH-008: SQL Injection Prevention', async ({ page }) => {
     await test.step('Attempt SQL injection in username', async () => {
       await loginPage.login("Admin' OR '1'='1", 'anything');
+      await loginPage.errorMessage.waitFor({ state: 'visible', timeout: 10000 });
     });
 
     await test.step('Verify login fails securely', async () => {
@@ -99,6 +102,7 @@ test.describe('Authentication Tests', () => {
   test('TC-AUTH-009: XSS Prevention', async ({ page }) => {
     await test.step('Attempt XSS in username field', async () => {
       await loginPage.login("<script>alert('XSS')</script>", 'admin123');
+      await loginPage.errorMessage.waitFor({ state: 'visible', timeout: 10000 });
     });
 
     await test.step('Verify no script execution', async () => {

automated-testing/tests/ui/pim.spec.js

@@ -17,7 +17,7 @@ test.describe('PIM Module Tests', () => {
 
     // Login before each test
     await loginPage.goto();
-    await loginPage.login('Admin', 'admin123');
+    await loginPage.loginAndWaitForDashboard('Admin', 'admin123');
     await expect(page).toHaveURL(/.*dashboard/);
     await dashboardPage.navigateToPIM();
   });
@@ -53,9 +53,7 @@ test.describe('PIM Module Tests', () => {
     });
 
     await test.step('Verify employee is saved', async () => {
-      // Wait for save to complete and page redirect
-      await page.waitForTimeout(3000);
-      await expect(page.getByText('Personal Details')).toBeVisible();
+      await expect(page.getByText('Personal Details')).toBeVisible({ timeout: 30000 });
     });
   });