@camunda/e2e-test-suite 0.0.982 → 0.0.983

package/dist/pages/SM-8.7/ModelerCreatePage.d.ts

@@ -101,9 +101,9 @@ declare class ModelerCreatePage {
     clickSecondPlacedGateway(): Promise<void>;
     private reloadAndWaitForCanvas;
     /**
-     * Detects if the page has been redirected to the Keycloak login page
-     * (session expiry) or is in the middle of an OIDC redirect, and performs
-     * a re-login if necessary.
+     * Detects if the page has been redirected to a login page (session expiry)
+     * or is in the middle of an OIDC redirect, and performs a re-login if
+     * necessary.
      */
     private handleSessionExpiry;
     clickStartInstanceMainButton(): Promise<void>;

package/dist/pages/SM-8.7/ModelerCreatePage.js

@@ -483,9 +483,9 @@ class ModelerCreatePage {
         await element.waitFor({ state: 'visible', timeout: 120000 });
     }
     /**
-     * Detects if the page has been redirected to the Keycloak login page
-     * (session expiry) or is in the middle of an OIDC redirect, and performs
-     * a re-login if necessary.
+     * Detects if the page has been redirected to a login page (session expiry)
+     * or is in the middle of an OIDC redirect, and performs a re-login if
+     * necessary.
      */
     async handleSessionExpiry() {
         const currentUrl = this.page.url();
@@ -501,12 +501,30 @@ class ModelerCreatePage {
         const loginPage = new LoginPage_1.LoginPage(this.page);
         const keycloakBanner = this.page.locator('body#keycloak-bg[data-page-id="login-login"]');
         const usernameField = this.page.locator('#username');
-        // Check if we landed on a login page (Keycloak form)
-        const isOnLoginPage = await keycloakBanner
+        // `loginPage.usernameInput` covers the Camunda Identity form, which carries
+        // neither the Keycloak body id nor a `#username` input, so neither of the
+        // two locators above matches it.
+        const loginForm = keycloakBanner
             .or(usernameField)
-            .isVisible({ timeout: 5000 })
-            .catch(() => false);
-        if (!isOnLoginPage) {
+            .or(loginPage.usernameInput)
+            .first();
+        // `Run` and the login form are mutually exclusive, so race them rather than
+        // sampling one: an expired session redirects through the OIDC callback
+        // after the reload has already resolved, so a point-in-time visibility
+        // check still sees the Modeler document and misses the login form that
+        // replaces it a moment later. The race also costs nothing on the happy
+        // path — it returns as soon as `Run` renders.
+        const landedOn = await Promise.race([
+            loginForm
+                .waitFor({ state: 'visible', timeout: 15000 })
+                .then(() => 'login')
+                .catch(() => 'unknown'),
+            this.startInstanceMainButton
+                .waitFor({ state: 'visible', timeout: 15000 })
+                .then(() => 'app')
+                .catch(() => 'unknown'),
+        ]);
+        if (landedOn !== 'login') {
             return;
         }
         console.warn('clickStartInstanceMainButton: session expired — performing re-login');

package/dist/pages/SM-8.7/TaskPanelPage.js

@@ -4,6 +4,7 @@ exports.TaskPanelPage = void 0;
 const test_1 = require("@playwright/test");
 const constants_1 = require("../../utils/constants");
 const sleep_1 = require("../../utils/sleep");
+const gatewayErrors_1 = require("../../utils/gatewayErrors");
 class TaskPanelPage {
     page;
     availableTasks;
@@ -39,6 +40,9 @@ class TaskPanelPage {
     }
     async openTask(name, assignButton = true) {
         const maxAttempts = 10;
+        // Shared by every attempt so that a sustained Tasklist outage still fails
+        // this call rather than stretching it by ten times this budget.
+        const gatewayRecoveryDeadline = Date.now() + constants_1._1_SECOND_IN_MS * 60;
         for (let attempt = 1; attempt <= maxAttempts; attempt++) {
             try {
                 await this.page.waitForLoadState('domcontentloaded');
@@ -56,7 +60,7 @@ class TaskPanelPage {
             catch (error) {
                 if (attempt < maxAttempts) {
                     console.warn(`Attempt ${attempt} failed. Reloading page...`);
-                    await this.page.reload();
+                    await (0, gatewayErrors_1.reloadPastGatewayError)(this.page, gatewayRecoveryDeadline);
                     await this.page.waitForLoadState('domcontentloaded');
                 }
                 else {

package/dist/pages/SM-8.7/UtlitiesPage.js

@@ -4,6 +4,8 @@ exports.modelDiagramFromFile = exports.modelAndRunConnectorsTimerEventDiagram =
 const test_1 = require("@playwright/test");
 const fileUpload_1 = require("../../utils/fileUpload");
 const sleep_1 = require("../../utils/sleep");
+const gatewayErrors_1 = require("../../utils/gatewayErrors");
+const constants_1 = require("../../utils/constants");
 // Tasklist renders "No permission for Tasklist" as a terminal page: it reads the
 // permissions off the token minted at login and never re-evaluates them, so a
 // user whose Identity role assignment has not propagated yet stays stuck there.
@@ -61,6 +63,9 @@ async function createAndRunProcess(modelerHomePage, modelerCreatePage, processNa
 }
 exports.createAndRunProcess = createAndRunProcess;
 async function completeTaskWithRetry(page, taskPanelPage, taskDetailsPage, taskName, taskPriority, maxRetries = 5) {
+    // Shared by every attempt so that a sustained Tasklist outage still fails
+    // this call rather than stretching it by five times this budget.
+    const gatewayRecoveryDeadline = Date.now() + constants_1._1_SECOND_IN_MS * 60;
     for (let attempt = 0; attempt < maxRetries; attempt++) {
         try {
             await page.waitForLoadState('domcontentloaded');
@@ -98,7 +103,7 @@ async function completeTaskWithRetry(page, taskPanelPage, taskDetailsPage, taskN
         catch (error) {
             console.warn(`Attempt ${attempt + 1} failed for completing task ${taskName}:`, error instanceof Error ? error.message : error);
             if (attempt < maxRetries - 1) {
-                await page.reload();
+                await (0, gatewayErrors_1.reloadPastGatewayError)(page, gatewayRecoveryDeadline);
                 await page.waitForLoadState('domcontentloaded');
                 await (0, sleep_1.sleep)(1000);
             }

package/dist/tests/SM-8.10/mt-enabled-user-flows.spec.js

@@ -19,7 +19,8 @@ if (process.env.IS_MT === 'true') {
             await (0, _setup_1.captureScreenshot)(page, testInfo);
             await (0, _setup_1.captureFailureVideo)(page, testInfo);
         });
-        (0, SM_8_10_1.test)('Main User Can Create & Access Tenant & User Without Access Cannot - User Task Flow', async ({ page, modelerHomePage, modelerCreatePage, taskPanelPage, operateProcessInstancePage, operateProcessesPage, taskDetailsPage, operateHomePage, navigationPage, ocTenantPage, optimizeHomePage, optimizeDashboardPage, browser, ocIdentityHomePage, managementIdentityTenantPage, managementIdentityPage, keycloakAdminPage, keycloakLoginPage, ocIdentityMappingRulesPage, ocIdentityRolesPage, }) => {
+        // Skipped due to bug #60836: https://github.com/camunda/camunda/issues/60836
+        SM_8_10_1.test.skip('Main User Can Create & Access Tenant & User Without Access Cannot - User Task Flow', async ({ page, modelerHomePage, modelerCreatePage, taskPanelPage, operateProcessInstancePage, operateProcessesPage, taskDetailsPage, operateHomePage, navigationPage, ocTenantPage, optimizeHomePage, optimizeDashboardPage, browser, ocIdentityHomePage, managementIdentityTenantPage, managementIdentityPage, keycloakAdminPage, keycloakLoginPage, ocIdentityMappingRulesPage, ocIdentityRolesPage, }) => {
             SM_8_10_1.test.slow();
             const randomString = await (0, _setup_1.generateRandomStringAsync)(3);
             const processName = 'Main_User_Task_Flow_' + randomString;
@@ -127,7 +128,8 @@ if (process.env.IS_MT === 'true') {
                 }
             });
         });
-        (0, SM_8_10_1.test)('Main User Can Create & Access Tenant - Connectors Flow @tasklistV2', async ({ page, modelerHomePage, modelerCreatePage, operateProcessInstancePage, operateProcessesPage, operateHomePage, navigationPage, managementIdentityTenantPage, connectorSettingsPage, connectorMarketplacePage, optimizeHomePage, optimizeDashboardPage, ocIdentityHomePage, ocTenantPage, }) => {
+        // Skipped due to bug #60836: https://github.com/camunda/camunda/issues/60836
+        SM_8_10_1.test.skip('Main User Can Create & Access Tenant - Connectors Flow @tasklistV2', async ({ page, modelerHomePage, modelerCreatePage, operateProcessInstancePage, operateProcessesPage, operateHomePage, navigationPage, managementIdentityTenantPage, connectorSettingsPage, connectorMarketplacePage, optimizeHomePage, optimizeDashboardPage, ocIdentityHomePage, ocTenantPage, }) => {
             SM_8_10_1.test.slow();
             const randomString = await (0, _setup_1.generateRandomStringAsync)(3);
             const processName = 'Main_User_Connectors_Flow_' + randomString;
@@ -259,7 +261,8 @@ if (process.env.IS_MT === 'true') {
                 });
             }
         });
-        (0, SM_8_10_1.test)('Second User Can Access Tenant Created By Main User - Connectors Flow @tasklistV2', async ({ page, modelerHomePage, modelerCreatePage, operateProcessInstancePage, operateProcessesPage, operateHomePage, navigationPage, managementIdentityTenantPage, connectorSettingsPage, connectorMarketplacePage, optimizeHomePage, optimizeDashboardPage, ocIdentityHomePage, ocTenantPage, managementIdentityPage, keycloakAdminPage, keycloakLoginPage, ocIdentityRolesPage, ocIdentityMappingRulesPage, browser, }) => {
+        // Skipped due to bug #60836: https://github.com/camunda/camunda/issues/60836
+        SM_8_10_1.test.skip('Second User Can Access Tenant Created By Main User - Connectors Flow @tasklistV2', async ({ page, modelerHomePage, modelerCreatePage, operateProcessInstancePage, operateProcessesPage, operateHomePage, navigationPage, managementIdentityTenantPage, connectorSettingsPage, connectorMarketplacePage, optimizeHomePage, optimizeDashboardPage, ocIdentityHomePage, ocTenantPage, managementIdentityPage, keycloakAdminPage, keycloakLoginPage, ocIdentityRolesPage, ocIdentityMappingRulesPage, browser, }) => {
             SM_8_10_1.test.slow();
             const randomString = await (0, _setup_1.generateRandomStringAsync)(3);
             const processName = 'Second_User_Connectors_Flow_' + randomString;
@@ -330,7 +333,8 @@ if (process.env.IS_MT === 'true') {
                 });
             }
         });
-        (0, SM_8_10_1.test)('Default Tenant Is Available And Used When Tenant Not Set In Deployment', async ({ page, modelerHomePage, modelerCreatePage, taskPanelPage, operateProcessInstancePage, operateProcessesPage, taskDetailsPage, operateHomePage, navigationPage, managementIdentityTenantPage, optimizeHomePage, optimizeDashboardPage, ocIdentityHomePage, ocTenantPage, }) => {
+        // Skipped due to bug #60836: https://github.com/camunda/camunda/issues/60836
+        SM_8_10_1.test.skip('Default Tenant Is Available And Used When Tenant Not Set In Deployment', async ({ page, modelerHomePage, modelerCreatePage, taskPanelPage, operateProcessInstancePage, operateProcessesPage, taskDetailsPage, operateHomePage, navigationPage, managementIdentityTenantPage, optimizeHomePage, optimizeDashboardPage, ocIdentityHomePage, ocTenantPage, }) => {
             SM_8_10_1.test.slow();
             const randomString = await (0, _setup_1.generateRandomStringAsync)(3);
             const processName = 'Default_Tenant_Flow_' + randomString;
@@ -466,7 +470,8 @@ if (process.env.IS_MT === 'true') {
                 await operateProcessInstancePage.tenantNameAssertion(tenantName2);
             });
         });
-        (0, SM_8_10_1.test)('Revoking Tenant Access User Flow', async ({ page, modelerHomePage, modelerCreatePage, taskPanelPage, operateProcessInstancePage, operateProcessesPage, taskDetailsPage, operateHomePage, navigationPage, managementIdentityTenantPage, optimizeHomePage, optimizeDashboardPage, ocIdentityHomePage, ocTenantPage, }) => {
+        // Skipped due to bug #60836: https://github.com/camunda/camunda/issues/60836
+        SM_8_10_1.test.skip('Revoking Tenant Access User Flow', async ({ page, modelerHomePage, modelerCreatePage, taskPanelPage, operateProcessInstancePage, operateProcessesPage, taskDetailsPage, operateHomePage, navigationPage, managementIdentityTenantPage, optimizeHomePage, optimizeDashboardPage, ocIdentityHomePage, ocTenantPage, }) => {
             SM_8_10_1.test.slow();
             const randomString = await (0, _setup_1.generateRandomStringAsync)(3);
             const processName1 = 'Revoke_Tenant_Flow_1' + randomString;

package/dist/utils/gatewayErrors.d.ts

@@ -0,0 +1,3 @@
+import { Page } from '@playwright/test';
+export declare function isGatewayErrorPage(page: Page): Promise<boolean>;
+export declare function reloadPastGatewayError(page: Page, deadline: number): Promise<void>;

package/dist/utils/gatewayErrors.js

@@ -0,0 +1,33 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.reloadPastGatewayError = exports.isGatewayErrorPage = void 0;
+const constants_1 = require("./constants");
+const GATEWAY_ERROR_HEADING = /50[234]\s+(Bad Gateway|Service Temporarily Unavailable|Gateway Time-?out)/i;
+async function isGatewayErrorPage(page) {
+    return page
+        .getByRole('heading', { name: GATEWAY_ERROR_HEADING })
+        .first()
+        .isVisible()
+        .catch(() => false);
+}
+exports.isGatewayErrorPage = isGatewayErrorPage;
+// While a component's pod restarts, its ingress answers with an nginx 50x page
+// that carries none of the app. A 5xx does not reject `page.reload`, so a retry
+// loop that reloads after every failed attempt keeps re-rendering that error
+// page and spends its whole attempt budget in seconds, long before the
+// component is back. Reload until the component answers again, bounded by
+// `deadline` so a sustained outage still fails the caller.
+async function reloadPastGatewayError(page, deadline) {
+    for (let attempt = 1;; attempt++) {
+        const response = await page.reload({ waitUntil: 'domcontentloaded' });
+        const status = response?.status();
+        const unavailable = (status !== undefined && status >= 500) ||
+            (await isGatewayErrorPage(page));
+        if (!unavailable || Date.now() >= deadline) {
+            return;
+        }
+        console.warn(`reloadPastGatewayError: ${page.url()} answered ${status} — waiting for the component to come back (attempt ${attempt})`);
+        await page.waitForTimeout(Math.min(constants_1._1_SECOND_IN_MS * 5 * attempt, deadline - Date.now()));
+    }
+}
+exports.reloadPastGatewayError = reloadPastGatewayError;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camunda/e2e-test-suite",
-  "version": "0.0.982",
+  "version": "0.0.983",
   "description": "End-to-end test helpers for Camunda 8",
   "repository": {
     "type": "git",