feat(e2e): replace fixed waits with pollUntil and project timeouts

Centralize condition-based polling for K8s, log-follow, and login flows so tests fail fast instead of sleeping, and move per-spec timeouts into Playwright project config.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: schultzp2020

e2e-tests/playwright.config.ts

@@ -96,6 +96,7 @@ export default defineConfig({
     },
     {
       name: PW_PROJECT.SHOWCASE,
+      timeout: 180 * 1000,
       dependencies: [PW_PROJECT.SMOKE_TEST],
       testIgnore: [
         "**/playwright/seed.spec.ts",
@@ -119,6 +120,7 @@ export default defineConfig({
     },
     {
       name: PW_PROJECT.SHOWCASE_AUTH_PROVIDERS,
+      timeout: 600 * 1000,
       testMatch: ["**/playwright/e2e/auth-providers/*.spec.ts"],
       testIgnore: [
         // temporarily disable github-happy-path
@@ -182,6 +184,7 @@ export default defineConfig({
     },
     {
       name: PW_PROJECT.SHOWCASE_RUNTIME_DB,
+      timeout: 600 * 1000,
       workers: 1,
       testMatch: [
         "**/playwright/e2e/external-database/verify-tls-config-with-external-rds.spec.ts",
@@ -190,6 +193,7 @@ export default defineConfig({
     },
     {
       name: PW_PROJECT.SHOWCASE_RUNTIME,
+      timeout: 900 * 1000,
       workers: 1,
       dependencies: [PW_PROJECT.SHOWCASE_RUNTIME_DB],
       testMatch: [

e2e-tests/playwright/e2e/audit-log/log-utils.ts

@@ -8,6 +8,7 @@ import {
   type EventSeverityLevel,
 } from "./logs";
 import { getBackstageDeploySelector } from "../../utils/helper";
+import { sleep } from "../../utils/poll-until";
 
 function formatError(error: unknown): string {
   if (error instanceof Error) {
@@ -252,9 +253,7 @@ export const LogUtils = {
       attempt++;
       if (attempt <= maxRetries) {
         console.log(`Waiting ${retryDelay / 1000} seconds before retrying...`);
-        await new Promise<void>((resolve) => {
-          setTimeout(resolve, retryDelay);
-        });
+        await sleep(retryDelay);
       }
     }
 

e2e-tests/playwright/e2e/auth-providers/github.spec.ts

@@ -53,7 +53,6 @@ test.describe("Configure Github Provider", async () => {
       description: "authentication",
     });
 
-    test.info().setTimeout(600 * 1000);
     // load default configs from yaml files
     await deployment.loadAllConfigs();
 
@@ -139,7 +138,6 @@ test.describe("Configure Github Provider", async () => {
   });
 
   test.beforeEach(() => {
-    test.info().setTimeout(600 * 1000);
     console.log(
       `Running test case ${test.info().title} - Attempt #${test.info().retry}`,
     );
@@ -285,8 +283,6 @@ test.describe("Configure Github Provider", async () => {
   });
 
   test(`Ingestion of Github users and groups: verify the user entities and groups are created with the correct relationships`, async () => {
-    test.setTimeout(300 * 1000);
-
     await expect
       .poll(
         () =>

e2e-tests/playwright/e2e/auth-providers/gitlab.spec.ts

@@ -55,7 +55,6 @@ test.describe("Configure GitLab Provider", async () => {
       description: "authentication",
     });
 
-    test.info().setTimeout(600 * 1000);
     // load default configs from yaml files
     await deployment.loadAllConfigs();
 
@@ -152,7 +151,6 @@ test.describe("Configure GitLab Provider", async () => {
   });
 
   test.beforeEach(() => {
-    test.info().setTimeout(60 * 1000);
     console.log(
       `Running test case ${test.info().title} - Attempt #${test.info().retry}`,
     );

e2e-tests/playwright/e2e/auth-providers/ldap.spec.ts

@@ -49,7 +49,6 @@ test.describe("Configure LDAP Provider", () => {
       description: "authentication",
     });
 
-    test.info().setTimeout(600 * 1000);
     // load default configs from yaml files
     await deployment.loadAllConfigs();
 
@@ -222,7 +221,6 @@ test.describe("Configure LDAP Provider", () => {
   });
 
   test.beforeEach(() => {
-    test.info().setTimeout(600 * 1000);
     console.log(
       `Running test case ${test.info().title} - Attempt #${test.info().retry}`,
     );

e2e-tests/playwright/e2e/auth-providers/microsoft.spec.ts

@@ -54,7 +54,6 @@ test.describe("Configure Microsoft Provider", async () => {
       description: "authentication",
     });
 
-    test.info().setTimeout(600 * 1000);
     // load default configs from yaml files
     await deployment.loadAllConfigs();
 
@@ -155,7 +154,6 @@ test.describe("Configure Microsoft Provider", async () => {
   });
 
   test.beforeEach(() => {
-    test.info().setTimeout(600 * 1000);
     console.log(
       `Running test case ${test.info().title} - Attempt #${test.info().retry}`,
     );
@@ -317,8 +315,6 @@ test.describe("Configure Microsoft Provider", async () => {
   });
 
   test(`Ingestion of Microsoft users and groups: verify the user entities and groups are created with the correct relationships`, async () => {
-    test.setTimeout(300 * 1000);
-
     await expect
       .poll(
         () =>

e2e-tests/playwright/e2e/auth-providers/oidc.spec.ts

@@ -63,7 +63,6 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
       description: "authentication",
     });
 
-    test.info().setTimeout(600 * 1000);
     // load default configs from yaml files
     await deployment.loadAllConfigs();
     // setup playwright helpers
@@ -152,7 +151,6 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
   });
 
   test.beforeEach(() => {
-    test.info().setTimeout(600 * 1000);
     console.log(
       `Running test case ${test.info().title} - Attempt #${test.info().retry}`,
     );

e2e-tests/playwright/e2e/configuration-test/config-map.spec.ts

@@ -18,8 +18,6 @@ test.describe("Change app-config at e2e test runtime", () => {
   });
 
   test("Verify title change after ConfigMap modification", async ({ page }) => {
-    test.setTimeout(300000);
-
     const configMapName = "app-config-rhdh";
 
     const namespace = process.env.NAME_SPACE_RUNTIME ?? "showcase-runtime";

e2e-tests/playwright/e2e/external-database/verify-tls-config-with-external-azure-db.spec.ts

@@ -71,7 +71,6 @@ test.describe("Verify TLS configuration with Azure Database for PostgreSQL healt
   for (const config of azureConfigurations) {
     test.describe.serial(`Azure DB ${config.name} PostgreSQL version`, () => {
       test.beforeAll(async () => {
-        test.setTimeout(180000);
         test.info().annotations.push({
           type: "database",
           description: config.host?.split(".")[0] || "unknown",
@@ -86,7 +85,6 @@ test.describe("Verify TLS configuration with Azure Database for PostgreSQL healt
 
       test("Configure and restart deployment", async () => {
         const kubeClient = new KubeClient();
-        test.setTimeout(600000);
         await configurePostgresCredentials(kubeClient, namespace, {
           host: config.host,
           user: azureUser,

e2e-tests/playwright/e2e/external-database/verify-tls-config-with-external-rds.spec.ts

@@ -67,7 +67,6 @@ test.describe("Verify TLS configuration with RDS PostgreSQL health check", () =>
   for (const config of rdsConfigurations) {
     test.describe.serial(`RDS ${config.name} PostgreSQL version`, () => {
       test.beforeAll(async () => {
-        test.setTimeout(135000);
         test.info().annotations.push({
           type: "database",
           description: config.host?.split(".")[0] || "unknown",
@@ -82,7 +81,6 @@ test.describe("Verify TLS configuration with RDS PostgreSQL health check", () =>
 
       test("Configure and restart deployment", async () => {
         const kubeClient = new KubeClient();
-        test.setTimeout(600000);
         await configurePostgresCredentials(kubeClient, namespace, {
           host: config.host,
           user: rdsUser,