@camunda/e2e-test-suite 0.0.969 → 0.0.971

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

@@ -18,5 +18,6 @@ declare class ConnectorMarketplacePage {
     waitForConnectorSearchResults(): Promise<void>;
     downloadConnectorToProject(): Promise<void>;
     private confirmImport;
+    private waitForImportToSettle;
 }
 export { ConnectorMarketplacePage };

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

@@ -85,6 +85,9 @@ class ConnectorMarketplacePage {
             await this.cancelButton.click({ timeout: 20000 });
             throw new Error('Connector import modal offered no import action ("Add to project", "Replace resource" or "Save as copy"); the import was abandoned.');
         }
+        // Hub imported without asking for confirmation, so the write is in flight
+        // here too.
+        await this.waitForImportToSettle();
     }
     // Hub's import modal labels its primary button after the project state:
     // "Add to project" when the template is new to the project, "Replace
@@ -99,18 +102,53 @@ class ConnectorMarketplacePage {
             this.replaceResourceButton,
             this.saveAsCopyButton,
         ];
-        const deadline = Date.now() + 20000;
+        // Carbon animates the modal in, so a click issued mid-transition is
+        // rejected ("element is not stable") and by the next Playwright retry the
+        // button can already be gone; keep probing the labels until one click
+        // actually lands instead of burning the whole budget on a single button.
+        const deadline = Date.now() + 45000;
         do {
             for (const button of importButtons) {
-                if (await button.isVisible().catch(() => false)) {
-                    await button.click({ timeout: 20000 });
-                    await (0, test_1.expect)(button).toBeHidden({ timeout: 60000 });
-                    return true;
+                if (!(await button.isVisible().catch(() => false))) {
+                    continue;
                 }
+                // Register the waiter before the click: the write can be acknowledged
+                // before the modal finishes unmounting.
+                const templateWritten = this.page
+                    .waitForResponse((response) => response.request().method() === 'POST' &&
+                    response.url().includes('/internal/files'), { timeout: 30000 })
+                    .catch(() => null);
+                const clicked = await button
+                    .click({ timeout: 5000 })
+                    .then(() => true)
+                    .catch(() => false);
+                if (!clicked) {
+                    continue;
+                }
+                await (0, test_1.expect)(button).toBeHidden({ timeout: 60000 });
+                const written = await templateWritten;
+                // A rejected write leaves the project without the template just like
+                // an aborted one, so surface the status here instead of letting it
+                // resurface 60s later as a missing "Authentication" group in
+                // ModelerCreatePage.
+                if (written && !written.ok()) {
+                    throw new Error(`Connector template import failed: writing it into the project returned HTTP ${written.status()}.`);
+                }
+                await this.waitForImportToSettle();
+                return true;
             }
             await (0, sleep_1.sleep)(1000);
         } while (Date.now() < deadline);
         return false;
     }
+    // Hub unmounts the ImportModal as soon as the click is handled, but writes the
+    // template into the project with a POST that is still in flight at that point.
+    // Returning right after the button hides let the caller's `page.reload()`
+    // abort that write ~20ms later, leaving the project's connector-template list
+    // empty and every templated task on the diagram at "Template: Not found" --
+    // the properties panel then never renders the "Authentication" group.
+    async waitForImportToSettle() {
+        await this.page.waitForLoadState('networkidle').catch(() => { });
+    }
 }
 exports.ConnectorMarketplacePage = ConnectorMarketplacePage;

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

@@ -30,6 +30,8 @@ declare class ModelerCreatePage {
     readonly cancelButton: Locator;
     readonly restConnectorOption: Locator;
     readonly changeElementSearchInput: Locator;
+    readonly changeElementTabStrip: Locator;
+    readonly reusableAssetsTab: Locator;
     readonly marketPlaceButton: Locator;
     readonly clientIdTextbox: Locator;
     readonly clientSecretTextbox: Locator;

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

@@ -35,6 +35,8 @@ class ModelerCreatePage {
     cancelButton;
     restConnectorOption;
     changeElementSearchInput;
+    changeElementTabStrip;
+    reusableAssetsTab;
     marketPlaceButton;
     clientIdTextbox;
     clientSecretTextbox;
@@ -156,6 +158,12 @@ class ModelerCreatePage {
             .locator('.djs-popup-results')
             .locator('[data-id="replace.template-io.camunda.connectors.HttpJson.v2"]');
         this.changeElementSearchInput = page.locator('.djs-popup-search input');
+        this.changeElementTabStrip = page.locator('.djs-popup-tabs-container');
+        // The tab renders its label as text content, and the product source
+        // spells it both "Reusable Assets" and "Reusable assets".
+        this.reusableAssetsTab = page.getByRole('tab', {
+            name: /reusable assets/i,
+        });
         // A title query is ambiguous: when the filtered result list has no connector
         // match the popup also renders an empty-state link carrying the same title,
         // so the query resolves to two elements and clicking throws in strict mode.
@@ -965,16 +973,44 @@ class ModelerCreatePage {
             }
         }
     }
-    // Connector templates nest under the change-element popup's "Templates"
-    // category and every entry now renders a description, so filter the popup by
-    // name to surface the entry as a flat search result before clicking it.
+    // Connector templates now live in the popup's "Reusable assets" tab, not
+    // the default "BPMN" one, so they do not render until either that tab is
+    // selected or a search is active (search spans every tab and flattens the
+    // result list).
+    // Use pressSequentially (not fill): the popup syncs its search state from a
+    // keyup listener only, and fill() dispatches just an input event, so the
+    // filter never applies and the unfiltered BPMN tab keeps rendering.
     async filterChangeElementPopup(searchTerm) {
+        // Two independent paths put the template in the DOM, and neither is
+        // reliable alone: selecting the tab renders the template entries
+        // directly, while searching spans every tab and flattens the result
+        // list. Run 32243588212 showed the search registering (the tab strip
+        // hid) with the template still absent from .djs-popup-results, so the
+        // tab click runs first and the search backs it up.
+        const hasTabs = await this.changeElementTabStrip
+            .isVisible({ timeout: 5000 })
+            .catch(() => false);
+        if (hasTabs) {
+            // A failed tab click leaves the search path intact, so do not throw.
+            await this.reusableAssetsTab.click({ timeout: 30000 }).catch(() => { });
+        }
         const hasSearch = await this.changeElementSearchInput
             .isVisible({ timeout: 5000 })
             .catch(() => false);
-        if (hasSearch) {
-            await this.changeElementSearchInput.fill(searchTerm);
+        if (!hasSearch) {
+            return;
         }
+        await this.changeElementSearchInput.click();
+        await this.changeElementSearchInput.fill('');
+        await this.changeElementSearchInput.pressSequentially(searchTerm, {
+            delay: 50,
+        });
+        // The popup hides its tab strip while searching. Wait on that as a
+        // readiness signal, but do not assert it: with the tab already selected
+        // the template is reachable whether or not the search registered.
+        await (0, test_1.expect)(this.changeElementTabStrip)
+            .toBeHidden({ timeout: 30000 })
+            .catch(() => { });
     }
     async clickWebhookMessageStartEventConnectorOption() {
         const maxRetries = 3;
@@ -988,6 +1024,11 @@ class ModelerCreatePage {
                 }
                 else {
                     await this.page.reload();
+                    // A reload drops the selection, so re-select the start event before
+                    // reopening the popup -- the context pad only renders for a selected
+                    // element and "Change element" would otherwise never appear.
+                    await (0, test_1.expect)(this.startEventElement).toBeVisible({ timeout: 60000 });
+                    await this.startEventElement.click({ force: true, timeout: 30000 });
                     await this.changeTypeButton.click({ force: true, timeout: 30000 });
                     await this.filterChangeElementPopup('Webhook Message Start Event Connector');
                     await this.webhookMessageStartEventConnectorOption.click({
@@ -1168,14 +1209,10 @@ class ModelerCreatePage {
         const max = 3;
         for (let i = 0; i < max; i++) {
             try {
-                // Filter the popup to surface the webhook connector. The full connector
-                // list can be long and the specific entry may be below the viewport or
-                // not yet populated when the popup first opens.
-                if (await this.changeElementSearchInput
-                    .isVisible({ timeout: 3000 })
-                    .catch(() => false)) {
-                    await this.changeElementSearchInput.fill('Webhook');
-                }
+                // Filter the popup to surface the webhook connector: it sits in the
+                // "Reusable assets" tab, which the popup only flattens into the result
+                // list while a search is active.
+                await this.filterChangeElementPopup('Webhook');
                 await (0, test_1.expect)(this.intermediateWebhookConnectorOption).toBeVisible({
                     timeout: 30000,
                 });

package/dist/pages/SM-8.8/NavigationPage.d.ts

@@ -30,10 +30,15 @@ declare class NavigationPage {
         username: string;
         password: string;
     }): Promise<void>;
+    private goToWithoutAccess;
     goToOptimizeWithoutAccess({ username, password, }?: {
         username?: string;
         password?: string;
     }): Promise<void>;
+    goToModelerWithoutAccess({ username, password, }?: {
+        username?: string;
+        password?: string;
+    }): Promise<void>;
     goToKeycloak(): Promise<void>;
     goToOCIdentity(sleepTimeout?: number, credentials?: {
         username: string;

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

@@ -5,6 +5,7 @@ const test_1 = require("@playwright/test");
 const LoginPage_1 = require("./LoginPage");
 const sleep_1 = require("../../utils/sleep");
 const constants_1 = require("../../utils/constants");
+const GATEWAY_ERROR_STATUSES = [502, 503, 504];
 class NavigationPage {
     page;
     modelerPageBanner;
@@ -118,18 +119,34 @@ class NavigationPage {
     async goToOptimize(sleepTimeout, credentials) {
         await this.goTo('/optimize', this.optimizePageBanner, sleepTimeout, credentials);
     }
-    // `goTo` ends by asserting the app banner, so it cannot be used to reach
-    // Optimize as a user without the Optimize role: that request ends on a
-    // `text/plain` 403 page carrying neither the banner nor a login form, and
-    // every retry burns a full 60s timeout. Navigate and log in only; the
-    // caller asserts what the denied page shows.
+    // `goTo` ends by asserting the app banner, so it cannot be used to reach a
+    // component the user has no role for: the denied response carries neither
+    // the banner nor a login form, and every retry burns a full 60s timeout.
+    // Navigate and log in only; the caller asserts what the denied page shows.
+    // The ingress answers 5xx while a component is still rolling out (upgrade
+    // scenarios), so retry the request until the component itself responds.
+    async goToWithoutAccess(url, username, password) {
+        await (0, test_1.expect)(async () => {
+            const response = await this.page.goto(url, {
+                timeout: 60000,
+                waitUntil: 'domcontentloaded',
+            });
+            (0, test_1.expect)(GATEWAY_ERROR_STATUSES).not.toContain(response?.status());
+        }).toPass({ timeout: 120000, intervals: [5000, 10000, 15000] });
+        await new LoginPage_1.LoginPage(this.page).login(username, password);
+    }
+    // Optimize denies a user without the Optimize role with a `text/plain` 403
+    // page.
     async goToOptimizeWithoutAccess({ username = 'demo', password = process.env.DISTRO_QA_E2E_TESTS_IDENTITY_FIRSTUSER_PASSWORD ??
         'demo', } = {}) {
-        await this.page.goto('/optimize', {
-            timeout: 60000,
-            waitUntil: 'domcontentloaded',
-        });
-        await new LoginPage_1.LoginPage(this.page).login(username, password);
+        await this.goToWithoutAccess('/optimize', username, password);
+    }
+    // Web Modeler denies a user without the Web Modeler role with a blank page.
+    // It briefly renders its banner before dropping it, so `goTo`'s closing
+    // banner assertion either fails outright or passes by winning that race.
+    async goToModelerWithoutAccess({ username = 'demo', password = process.env.DISTRO_QA_E2E_TESTS_IDENTITY_FIRSTUSER_PASSWORD ??
+        'demo', } = {}) {
+        await this.goToWithoutAccess('/modeler', username, password);
     }
     async goToKeycloak() {
         const keycloakUrl = process.env.KEYCLOAK_URL
@@ -147,18 +164,11 @@ class NavigationPage {
     async goToConsole(sleepTimeout, credentials) {
         await this.goTo('/console', this.consolePageBanner, sleepTimeout, credentials);
     }
-    // `goTo` ends by asserting the app banner, so it cannot be used to reach
-    // Console as a user without the Console role: that request renders
-    // Console's own "Something went wrong" error page, which has neither the
-    // banner nor a login form, and every retry burns a full 60s timeout.
-    // Navigate and log in only; the caller asserts what the denied page shows.
+    // Console denies a user without the Console role with its own "Something
+    // went wrong" error page.
     async goToConsoleWithoutAccess({ username = 'demo', password = process.env.DISTRO_QA_E2E_TESTS_IDENTITY_FIRSTUSER_PASSWORD ??
         'demo', } = {}) {
-        await this.page.goto('/console', {
-            timeout: 60000,
-            waitUntil: 'domcontentloaded',
-        });
-        await new LoginPage_1.LoginPage(this.page).login(username, password);
+        await this.goToWithoutAccess('/console', username, password);
     }
     async goToManagementIdentity(sleepTimeout, credentials) {
         await this.goTo('/identity', this.managementIdentityPageBanner, sleepTimeout, credentials);

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

@@ -74,7 +74,7 @@ SM_8_8_1.test.describe('Identity User Flow Tests', () => {
             });
         });
         await SM_8_8_1.test.step('Ensure Modeler is not Accessible', async () => {
-            await navigationPage.goToModeler(undefined, credentials);
+            await navigationPage.goToModelerWithoutAccess(credentials);
             await (0, test_1.expect)(modelerHomePage.modelerPageBanner).not.toBeVisible({
                 timeout: 60000,
             });

package/package.json

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