@camunda/e2e-test-suite 0.0.1020 → 0.0.1022

package/dist/pages/8.7/ClusterSecretsPage.d.ts

@@ -16,6 +16,7 @@ declare class ClusterSecretsPage {
     readonly confirmDeleteInput: Locator;
     constructor(page: Page);
     deleteConnectorSecretsIfExist(): Promise<void>;
+    private dismissOpenModal;
     clickCreateNewSecretButton(clusterName: string): Promise<void>;
     clickClusterBanner(): Promise<void>;
     clickCluster(name: string): Promise<void>;

package/dist/pages/8.7/ClusterSecretsPage.js

@@ -38,14 +38,25 @@ class ClusterSecretsPage {
             name: 'Delete',
             exact: true,
         });
-        this.deleteConnectorSecretSubButton = page.getByRole('button', {
-            name: 'danger Delete',
-        });
+        // The confirmation modal's destructive action is a plain "Delete" button
+        // (Console renders Carbon's ModalFooter danger button, whose accessible
+        // name is just its label). The old 'danger Delete' name came from a Carbon
+        // version that prefixed danger buttons with a visually-hidden kind label;
+        // it matches nothing today, so the click timed out and left the modal open
+        // on top of the page. Scoped to the dialog so it cannot collide with the
+        // row-level Delete control behind it.
+        this.deleteConnectorSecretSubButton = page
+            .getByRole('dialog')
+            .getByRole('button', { name: 'Delete', exact: true });
         this.dialog = page.getByRole('dialog');
         this.clusterBanner = this.page
             .getByRole('banner')
             .getByRole('link', { name: 'Clusters' });
-        this.clusterLink = (name) => page.getByRole('link', { name: name });
+        // exact: true because console-user-flows.spec.ts creates a second cluster
+        // called "UI Test Cluster" in the same org -- a substring match on
+        // "Test Cluster" then resolves to both links and fails on strict mode.
+        // ClusterPage's own cluster locators already match exactly.
+        this.clusterLink = (name) => page.getByRole('link', { name: name, exact: true });
         this.clusterSecretTab = this.page.getByRole('tab', {
             name: 'Connector Secrets',
         });
@@ -95,13 +106,39 @@ class ClusterSecretsPage {
                     break;
                 }
             }
-            await (0, test_1.expect)(this.optionsButton).not.toBeVisible({
-                timeout: 30000,
-            });
+            // Console auto-provisions CAMUNDA_PROVIDED_LLM_* secrets whose Delete
+            // menu item is disabled, so the table is never empty once the sweep is
+            // done. A page-wide not.toBeVisible() therefore matched three Options
+            // buttons at once (strict mode violation) while asserting something that
+            // is no longer true. The post-condition this helper actually owns is
+            // that nothing *deletable* is left behind.
+            const deletableSecretRows = this.page
+                .getByRole('row')
+                .filter({ has: this.page.getByRole('cell', { name: 'Options' }) })
+                .filter({ hasNotText: 'CAMUNDA_PROVIDED_' });
+            await (0, test_1.expect)(deletableSecretRows).toHaveCount(0, { timeout: 30000 });
         }
         catch (error) {
             console.error('Error during Connector Secrets deletion: ', error);
+            // A half-completed delete leaves Carbon's confirmation modal on screen,
+            // where its overlay swallows pointer events for every later click -- the
+            // next "Create new secret" click then burns its whole timeout against
+            // the overlay instead of the button. Cleanup must never hand the page
+            // back with a modal still open.
+            await this.dismissOpenModal();
+        }
+    }
+    // Closes whichever Carbon modal is currently on screen, and waits for it to
+    // actually leave the DOM rather than assuming a fixed delay.
+    async dismissOpenModal() {
+        const openModal = this.page.locator('.cds--modal.is-visible');
+        if ((await openModal.count().catch(() => 0)) === 0) {
+            return;
         }
+        await this.page.keyboard.press('Escape').catch(() => { });
+        await (0, test_1.expect)(openModal)
+            .toHaveCount(0, { timeout: 10000 })
+            .catch(() => { });
     }
     async clickCreateNewSecretButton(clusterName) {
         try {
@@ -118,16 +155,7 @@ class ClusterSecretsPage {
         }
     }
     async clickClusterBanner() {
-        try {
-            const blockingModal = this.page.locator('.cds--modal.is-visible');
-            if ((await blockingModal.count()) > 0) {
-                await this.page.keyboard.press('Escape');
-                await (0, sleep_1.sleep)(1000);
-            }
-        }
-        catch (_e) {
-            // no blocking modal to dismiss
-        }
+        await this.dismissOpenModal();
         await (0, test_1.expect)(this.clusterBanner).toBeVisible({ timeout: 30000 });
         await this.clusterBanner.click({ timeout: 60000 });
     }

package/dist/pages/8.7/UtilitiesPage.js

@@ -89,8 +89,14 @@ async function modelRestConnector(modelerCreatePage, connectorSettingsPage, proc
     username: '',
     password: '',
 }) {
+    // 120s to match every other properties-panel wait in the 8.7 suite
+    // (modelJobWorkerDiagram, clickDiagramBreadcrumb and the specs' own waits all
+    // allow 90-200s). 30s was the lone outlier here and the bpmn-js properties
+    // panel regularly needs longer than that to mount on a freshly created
+    // diagram, which failed the REST connector flows before a single element was
+    // modelled.
     await (0, test_1.expect)(modelerCreatePage.generalPanel).toBeVisible({
-        timeout: 30000,
+        timeout: 120000,
     });
     await modelerCreatePage.enterDiagramName(processName);
     await (0, sleep_1.sleep)(10000);

package/dist/pages/SM-8.10/ManagementIdentityPage.d.ts

@@ -27,7 +27,7 @@ declare class ManagementIdentityPage {
     clickUser(username: string): Promise<void>;
     clickAssignedRolesTab(): Promise<void>;
     clickDemoUser(): Promise<void>;
-    clickDeleteAccessButton(accessToDelete: string): Promise<void>;
+    clickDeleteAccessButton(accessToDelete: string | string[]): Promise<void>;
     clickConfirmDeleteButton(): Promise<void>;
     clickAssignRolesButton(): Promise<void>;
     clickAddButton(): Promise<void>;

package/dist/pages/SM-8.10/ManagementIdentityPage.js

@@ -46,9 +46,16 @@ class ManagementIdentityPage {
         this.optimizeCheckbox = page
             .locator('label')
             .filter({ hasText: 'Optimize (Grants full access to Optimize)' });
-        this.modelerCheckbox = page
-            .locator('label')
-            .filter({ hasText: 'Hub (Grants full access to Hub)' });
+        // Mid-rename: camunda/identity#4541 moves these role descriptions from Web
+        // Modeler to Hub, and the role names follow only later. Depending on which
+        // Identity image the deployed chart carries, the same checkbox renders as
+        // "Web Modeler (Grants full access to Web Modeler)", the half-renamed
+        // "Web Modeler (Grants full access to Hub)", or "Hub (Grants full access to
+        // Hub)". Match all three so the suite stays green on either side of the
+        // rollout. The " (" after the name keeps this off the Admin row.
+        this.modelerCheckbox = page.locator('label').filter({
+            hasText: /(Web Modeler|Hub) \(Grants full access to (Web Modeler|Hub)\)/,
+        });
         this.identityCheckbox = page.locator('label').filter({
             hasText: 'ManagementIdentity (Provides full access to Management Identity)',
         });
@@ -60,8 +67,9 @@ class ManagementIdentityPage {
         this.licenseKeyTagNonCommercial = page.getByText('Non-commercial license');
         this.licenseKeyTagProduction = page.getByText('Production license').first();
         this.licenseKeyTagCommercial = page.getByText('Commercial license');
+        // Same Web Modeler → Hub rename straddle as `modelerCheckbox` above.
         this.modelerAdminCheckbox = page.locator('label').filter({
-            hasText: 'Hub Admin (Grants elevated access to Hub)',
+            hasText: /(Web Modeler|Hub) Admin \(Grants elevated access to (Web Modeler|Hub)\)/,
         });
         this.openSidebarButton = page.getByLabel('Open User Sidebar');
         this.logoutButton = page.getByRole('button', { name: 'Log out' });
@@ -100,9 +108,9 @@ class ManagementIdentityPage {
     // and it does not re-render on its own. Rebuild it from a fresh response
     // between attempts before reporting the role as missing.
     async clickDeleteAccessButton(accessToDelete) {
-        const roleRow = this.page
-            .locator(':text("' + accessToDelete + '")')
-            .first();
+        const candidates = typeof accessToDelete === 'string' ? [accessToDelete] : accessToDelete;
+        const rowSelector = candidates.map((c) => `:text("${c}")`).join(', ');
+        const roleRow = this.page.locator(rowSelector).first();
         for (let attempt = 1; attempt <= 2; attempt++) {
             const rowVisible = await roleRow
                 .waitFor({ state: 'visible', timeout: 30000 })
@@ -114,10 +122,12 @@ class ManagementIdentityPage {
             await this.page.reload({ waitUntil: 'domcontentloaded' });
             await this.clickAssignedRolesTab();
         }
-        await (0, test_1.expect)(roleRow, `No assigned role matching "${accessToDelete}" was found. The role may ` +
-            `already be removed, or its label may have changed in this version.`).toBeVisible({ timeout: 60000 });
+        await (0, test_1.expect)(roleRow, `No assigned role matching ${candidates
+            .map((c) => `"${c}"`)
+            .join(' or ')} was found. The role may already be removed, or its ` +
+            `label may have changed in this version.`).toBeVisible({ timeout: 60000 });
         await this.page
-            .locator('td:right-of(:text("' + accessToDelete + '"))')
+            .locator(candidates.map((c) => `td:right-of(:text("${c}"))`).join(', '))
             .first()
             .click({ timeout: 60000 });
     }

package/dist/pages/SM-8.10/NavigationPage.js

@@ -6,13 +6,17 @@ const LoginPage_1 = require("./LoginPage");
 const sleep_1 = require("../../utils/sleep");
 const constants_1 = require("../../utils/constants");
 const env_1 = require("../../utils/env");
-const ORCHESTRATION_CONTEXT_PATH = process.env.ORCHESTRATION_CONTEXT_PATH ?? '/orchestration';
+// Context paths use `||`, not `??`: an env var set to the empty string must fall back to the
+// default. CI wiring that always exports a variable (set to "" when the leg has no override)
+// otherwise defeats the default — `page.goto('')` resolves to Playwright's baseURL, so the app
+// under test is never reached and the failure surfaces as an unrelated navigation timeout.
+const ORCHESTRATION_CONTEXT_PATH = process.env.ORCHESTRATION_CONTEXT_PATH || '/orchestration';
 const NORMALIZED_ORCHESTRATION_CONTEXT_PATH = ORCHESTRATION_CONTEXT_PATH.startsWith('/')
     ? ORCHESTRATION_CONTEXT_PATH
     : `/${ORCHESTRATION_CONTEXT_PATH}`;
-const MODELER_CONTEXT_PATH = process.env.MODELER_CONTEXT_PATH ?? '/modeler';
-const OPTIMIZE_CONTEXT_PATH = process.env.OPTIMIZE_CONTEXT_PATH ?? '/optimize';
-const MANAGEMENT_IDENTITY_CONTEXT_PATH = process.env.MANAGEMENT_IDENTITY_CONTEXT_PATH ?? '/identity';
+const MODELER_CONTEXT_PATH = process.env.MODELER_CONTEXT_PATH || '/modeler';
+const OPTIMIZE_CONTEXT_PATH = process.env.OPTIMIZE_CONTEXT_PATH || '/optimize';
+const MANAGEMENT_IDENTITY_CONTEXT_PATH = process.env.MANAGEMENT_IDENTITY_CONTEXT_PATH || '/identity';
 const IDENTITY_FIRSTUSER_USERNAME = process.env.DISTRO_QA_E2E_TESTS_IDENTITY_FIRSTUSER_USERNAME || 'demo';
 const IDENTITY_FIRSTUSER_PASSWORD = process.env.DISTRO_QA_E2E_TESTS_IDENTITY_FIRSTUSER_PASSWORD || 'demo';
 class NavigationPage {

package/dist/pages/SM-8.10/UtilitiesPage.js

@@ -142,7 +142,7 @@ async function completeTaskWithRetry(page, taskPanelPage, taskDetailsPage, taskN
                 // be re-opened once the blip clears. Progressive backoff gives the
                 // backend time to recover between attempts.
                 await (0, sleep_1.sleep)(2000 * (attempt + 1));
-                const orchestrationContextPath = process.env.ORCHESTRATION_CONTEXT_PATH ?? '/orchestration';
+                const orchestrationContextPath = process.env.ORCHESTRATION_CONTEXT_PATH || '/orchestration';
                 const normalizedOrchestrationPath = orchestrationContextPath.startsWith('/')
                     ? orchestrationContextPath
                     : `/${orchestrationContextPath}`;

package/dist/pages/SM-8.9/NavigationPage.js

@@ -5,14 +5,18 @@ const test_1 = require("@playwright/test");
 const LoginPage_1 = require("./LoginPage");
 const sleep_1 = require("../../utils/sleep");
 const constants_1 = require("../../utils/constants");
-const ORCHESTRATION_CONTEXT_PATH = process.env.ORCHESTRATION_CONTEXT_PATH ?? '/orchestration';
+// Context paths use `||`, not `??`: an env var set to the empty string must fall back to the
+// default. CI wiring that always exports a variable (set to "" when the leg has no override)
+// otherwise defeats the default — `page.goto('')` resolves to Playwright's baseURL, so the app
+// under test is never reached and the failure surfaces as an unrelated navigation timeout.
+const ORCHESTRATION_CONTEXT_PATH = process.env.ORCHESTRATION_CONTEXT_PATH || '/orchestration';
 const NORMALIZED_ORCHESTRATION_CONTEXT_PATH = ORCHESTRATION_CONTEXT_PATH.startsWith('/')
     ? ORCHESTRATION_CONTEXT_PATH
     : `/${ORCHESTRATION_CONTEXT_PATH}`;
-const MODELER_CONTEXT_PATH = process.env.MODELER_CONTEXT_PATH ?? '/modeler';
-const OPTIMIZE_CONTEXT_PATH = process.env.OPTIMIZE_CONTEXT_PATH ?? '/optimize';
-const CONSOLE_CONTEXT_PATH = process.env.CONSOLE_CONTEXT_PATH ?? '/console';
-const MANAGEMENT_IDENTITY_CONTEXT_PATH = process.env.MANAGEMENT_IDENTITY_CONTEXT_PATH ?? '/identity';
+const MODELER_CONTEXT_PATH = process.env.MODELER_CONTEXT_PATH || '/modeler';
+const OPTIMIZE_CONTEXT_PATH = process.env.OPTIMIZE_CONTEXT_PATH || '/optimize';
+const CONSOLE_CONTEXT_PATH = process.env.CONSOLE_CONTEXT_PATH || '/console';
+const MANAGEMENT_IDENTITY_CONTEXT_PATH = process.env.MANAGEMENT_IDENTITY_CONTEXT_PATH || '/identity';
 const IDENTITY_FIRSTUSER_USERNAME = process.env.DISTRO_QA_E2E_TESTS_IDENTITY_FIRSTUSER_USERNAME || 'demo';
 const IDENTITY_FIRSTUSER_PASSWORD = process.env.DISTRO_QA_E2E_TESTS_IDENTITY_FIRSTUSER_PASSWORD || 'demo';
 // Navigation errors raised by Chromium's network stack rather than by the app:

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

@@ -53,7 +53,12 @@ SM_8_10_1.test.describe('Identity User Flow Tests', () => {
             });
         }
         await SM_8_10_1.test.step('Remove Access from Hub', async () => {
-            await managementIdentityPage.clickDeleteAccessButton('Grants full access to Hub');
+            // Either wording, for the same Web Modeler → Hub rename straddle the
+            // `modelerCheckbox` locator documents.
+            await managementIdentityPage.clickDeleteAccessButton([
+                'Grants full access to Hub',
+                'Grants full access to Web Modeler',
+            ]);
             await managementIdentityPage.clickConfirmDeleteButton();
         });
         await SM_8_10_1.test.step('Reset Session', async () => {

package/dist/tests/SM-8.9/connectors-user-flows.spec.js

@@ -117,8 +117,28 @@ SM_8_9_1.test.describe('Connectors User Flow Tests @tasklistV2', () => {
         await SM_8_9_1.test.step('Make Authorization Request', async () => {
             const baseURL = process.env.PLAYWRIGHT_BASE_URL ||
                 `http://gke-${process.env.BASE_URL}.ci.distro.ultrawombat.com`;
-            const response = await request.post(`${baseURL}/connectors/inbound/test-webhook-id${randomString}`);
-            await (0, test_1.expect)(response.status()).toBe(200);
+            const url = `${baseURL}/connectors/inbound/test-webhook-id${randomString}`;
+            // Deploying the process does not mean the gateway is already serving the
+            // inbound endpoint, so the first POST can land on a not-yet-registered
+            // webhook and come back 5xx with nothing actually wrong. The sibling
+            // Intermediate Event step below polls for the same reason.
+            //
+            // Retry ONLY on 5xx. A 5xx means the request was not processed, so
+            // re-posting cannot double-trigger the process; a 2xx or 4xx is a real
+            // answer about a live endpoint and must be asserted as-is rather than
+            // retried until it changes.
+            let response = await request.post(url);
+            const deadline = Date.now() + 180000;
+            let attempt = 0;
+            while (response.status() >= 500 && Date.now() < deadline) {
+                attempt++;
+                console.error(`Webhook readiness attempt ${attempt}: ${url} -> ${response.status()}`);
+                await (0, sleep_1.sleep)(10000);
+                response = await request.post(url);
+            }
+            // Not awaited: expect() on a plain value is synchronous, so an `await`
+            // here would read as a retrying assertion while doing nothing.
+            (0, test_1.expect)(response.status()).toBe(200);
         });
         await SM_8_9_1.test.step('Assert Diagram Has Successfully Completed in Operate', async () => {
             await navigationPage.goToOperate();

package/dist/utils/apiHelpers.js

@@ -190,7 +190,7 @@ function buildZeebeApiUrl(resourcePath, environmentOverride, clusterType) {
     const normalizedPath = resourcePath.startsWith('/')
         ? resourcePath
         : `/${resourcePath}`;
-    const contextPath = process.env.ORCHESTRATION_CONTEXT_PATH ?? '/orchestration';
+    const contextPath = process.env.ORCHESTRATION_CONTEXT_PATH || '/orchestration';
     const pathWithPrefix = environment === 'sm' ? `${contextPath}${normalizedPath}` : normalizedPath;
     return `${baseUrl}${pathWithPrefix}`;
 }

package/package.json

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