Refactor ClientPage selectors and improve test stability.

Author: Tushar-Jawale

playwright/pages/client.page.ts

@@ -6,7 +6,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
-import { Page, Locator, expect } from '@playwright/test';
+import { Locator, expect } from '@playwright/test';
 import { BasePage } from './BasePage';
 
 /**
@@ -49,9 +49,9 @@ export class ClientPage extends BasePage {
     return this.page.locator('table.bordered-table tbody tr');
   }
 
-  // Get the first client name cell (carries the routerLink).
+  // Get the first client name cell (the displayName column carries the routerLink).
   get firstClientNameCell(): Locator {
-    return this.page.locator('table.bordered-table tbody tr td').first();
+    return this.page.locator('table.bordered-table tbody tr td.mat-column-displayName').first();
   }
 
   // Get the "No client was found" alert message.
@@ -134,6 +134,8 @@ export class ClientPage extends BasePage {
     } catch (e) {
       console.log('Network idle timeout on clients page - proceeding anyway');
     }
+    // Verify the UI is ready by checking for the search input
+    await this.searchInput.waitFor({ state: 'visible', timeout: 15000 });
   }
 
   /**
@@ -149,6 +151,11 @@ export class ClientPage extends BasePage {
     } catch (e) {
       console.log('Network idle timeout after search - proceeding anyway');
     }
+    // Wait for observable results: either rows appear or "no client found" message
+    await Promise.race([
+      this.clientRows.first().waitFor({ state: 'visible', timeout: 15000 }),
+      this.noClientFoundMessage.waitFor({ state: 'visible', timeout: 15000 })
+    ]);
   }
 
   /**

playwright/tests/client.spec.ts

@@ -53,12 +53,11 @@ test.describe('Client Module', () => {
     await expect(page).toHaveURL(/.*clients\/create.*/);
   });
 
-  test('should open a client profile', async ({ page }) => {
+  test('should open a client profile', async () => {
     await clientPage.navigateToClients();
 
     // Trigger search to populate the table
     await clientPage.searchClient('');
-    await page.waitForTimeout(3000);
 
     await expect(clientPage.clientRows.first()).toBeVisible({ timeout: 30000 });
     await clientPage.openClientProfile();