[WIP] Update timeouts and add validation for dashboard loading (#14)

Copilot · bg-playground · web-flow · commit d4ce7b08aa67 · 2026-02-12T06:49:06.000-05:00
* Initial plan

* Fix Playwright timeout issues with 60s timeouts and load validation

Co-authored-by: bg-playground &lt;259109604+bg-playground@users.noreply.github.com&gt;

---------

Co-authored-by: copilot-swe-agent[bot] &lt;198982749+Copilot@users.noreply.github.com&gt;
Co-authored-by: bg-playground &lt;259109604+bg-playground@users.noreply.github.com&gt;
diff --git a/automated-testing/pages/AdminPage.js b/automated-testing/pages/AdminPage.js
@@ -28,6 +28,7 @@ export class AdminPage {
   }
 
   async getRecordsCount() {
+    await this.recordsFound.waitFor({ state: 'visible', timeout: 60000 });
     return await this.recordsFound.textContent() || '';
   }
 
diff --git a/automated-testing/pages/DashboardPage.js b/automated-testing/pages/DashboardPage.js
@@ -24,27 +24,32 @@ export class DashboardPage {
   }
 
   async navigateToAdmin() {
-    await this.adminMenu.waitFor({ state: 'visible' });
+    await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });
+    await this.adminMenu.waitFor({ state: 'visible', timeout: 60000 });
     await this.adminMenu.click();
   }
 
   async navigateToPIM() {
-    await this.pimMenu.waitFor({ state: 'visible' });
+    await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });
+    await this.pimMenu.waitFor({ state: 'visible', timeout: 60000 });
     await this.pimMenu.click();
   }
 
   async navigateToLeave() {
-    await this.leaveMenu.waitFor({ state: 'visible' });
+    await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });
+    await this.leaveMenu.waitFor({ state: 'visible', timeout: 60000 });
     await this.leaveMenu.click();
   }
 
   async navigateToTime() {
-    await this.timeMenu.waitFor({ state: 'visible' });
+    await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });
+    await this.timeMenu.waitFor({ state: 'visible', timeout: 60000 });
     await this.timeMenu.click();
   }
 
   async navigateToRecruitment() {
-    await this.recruitmentMenu.waitFor({ state: 'visible' });
+    await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });
+    await this.recruitmentMenu.waitFor({ state: 'visible', timeout: 60000 });
     await this.recruitmentMenu.click();
   }
 
diff --git a/automated-testing/pages/LoginPage.js b/automated-testing/pages/LoginPage.js
@@ -17,6 +17,8 @@ export class LoginPage {
     await this.usernameInput.fill(username);
     await this.passwordInput.fill(password);
     await this.loginButton.click();
+    // Wait for navigation to complete after login
+    await this.page.waitForLoadState('networkidle', { timeout: 60000 });
   }
 
   async getErrorMessage() {

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ export class AdminPage {`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`async getRecordsCount() {`
	`31`	`+ await this.recordsFound.waitFor({ state: 'visible', timeout: 60000 });`
`31`	`32`	`return await this.recordsFound.textContent() \|\| '';`
`32`	`33`	`}`
`33`	`34`
Original file line number	Diff line number	Diff line change
`@@ -24,27 +24,32 @@ export class DashboardPage {`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`async navigateToAdmin() {`
`27`		`- await this.adminMenu.waitFor({ state: 'visible' });`
	`27`	`+ await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });`
	`28`	`+ await this.adminMenu.waitFor({ state: 'visible', timeout: 60000 });`
`28`	`29`	`await this.adminMenu.click();`
`29`	`30`	`}`
`30`	`31`
`31`	`32`	`async navigateToPIM() {`
`32`		`- await this.pimMenu.waitFor({ state: 'visible' });`
	`33`	`+ await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });`
	`34`	`+ await this.pimMenu.waitFor({ state: 'visible', timeout: 60000 });`
`33`	`35`	`await this.pimMenu.click();`
`34`	`36`	`}`
`35`	`37`
`36`	`38`	`async navigateToLeave() {`
`37`		`- await this.leaveMenu.waitFor({ state: 'visible' });`
	`39`	`+ await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });`
	`40`	`+ await this.leaveMenu.waitFor({ state: 'visible', timeout: 60000 });`
`38`	`41`	`await this.leaveMenu.click();`
`39`	`42`	`}`
`40`	`43`
`41`	`44`	`async navigateToTime() {`
`42`		`- await this.timeMenu.waitFor({ state: 'visible' });`
	`45`	`+ await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });`
	`46`	`+ await this.timeMenu.waitFor({ state: 'visible', timeout: 60000 });`
`43`	`47`	`await this.timeMenu.click();`
`44`	`48`	`}`
`45`	`49`
`46`	`50`	`async navigateToRecruitment() {`
`47`		`- await this.recruitmentMenu.waitFor({ state: 'visible' });`
	`51`	`+ await this.dashboardTitle.waitFor({ state: 'visible', timeout: 60000 });`
	`52`	`+ await this.recruitmentMenu.waitFor({ state: 'visible', timeout: 60000 });`
`48`	`53`	`await this.recruitmentMenu.click();`
`49`	`54`	`}`
`50`	`55`
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,8 @@ export class LoginPage {`
`17`	`17`	`await this.usernameInput.fill(username);`
`18`	`18`	`await this.passwordInput.fill(password);`
`19`	`19`	`await this.loginButton.click();`
	`20`	`+ // Wait for navigation to complete after login`
	`21`	`+ await this.page.waitForLoadState('networkidle', { timeout: 60000 });`
`20`	`22`	`}`
`21`	`23`
`22`	`24`	`async getErrorMessage() {`