@camunda/e2e-test-suite 0.0.970 → 0.0.972

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

@@ -25,6 +25,9 @@ declare class ModelerCreatePage {
     readonly additionalActionsButton: Locator;
     readonly cancelButton: Locator;
     readonly restConnectorOption: Locator;
+    readonly changeElementSearchInput: Locator;
+    readonly changeElementTabStrip: Locator;
+    readonly reusableAssetsTab: Locator;
     readonly webhookMessageStartEventConnectorOption: Locator;
     readonly webhookTab: Locator;
     readonly webhookIsActiveButton: Locator;
@@ -156,6 +159,8 @@ declare class ModelerCreatePage {
     clickNewForm(): Promise<void>;
     clickDeploySubButton(): Promise<void>;
     clickCancelButton(): Promise<void>;
+    private waitForVisible;
+    revealChangeElementTemplate(option: Locator, searchTerm: string): Promise<void>;
     clickRestConnectorOption(): Promise<void>;
     clickWebhookStartEventConnectorOption(): Promise<void>;
     clickWebhookTab(): Promise<void>;

package/dist/pages/8.10/ModelerCreatePage.js

@@ -31,6 +31,9 @@ class ModelerCreatePage {
     additionalActionsButton;
     cancelButton;
     restConnectorOption;
+    changeElementSearchInput;
+    changeElementTabStrip;
+    reusableAssetsTab;
     webhookMessageStartEventConnectorOption;
     webhookTab;
     webhookIsActiveButton;
@@ -170,6 +173,17 @@ class ModelerCreatePage {
         this.deploySubButton = page.locator('[data-test="deploy-action"]');
         this.cancelButton = page.getByRole('button', { name: 'Cancel' });
         this.restConnectorOption = page.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, which wins over its `title`
+        // attribute in the accessible-name computation. Match case-insensitively:
+        // the label is capitalised differently across bpmn-js releases. Scope the
+        // lookup to the popup's own tab strip -- the properties panel renders its
+        // own tablist (see webhookTab below), so an unscoped tab query is
+        // ambiguous.
+        this.reusableAssetsTab = this.changeElementTabStrip.getByRole('tab', {
+            name: /reusable assets/i,
+        });
         // bpmn-js popup entries are `li[role="option"]` inside a `role="listbox"`,
         // so getByRole('listitem') no longer matches. Their accessible name also
         // includes the entry description, and three entries share the label
@@ -734,11 +748,53 @@ class ModelerCreatePage {
     async clickCancelButton() {
         await this.cancelButton.click({ timeout: 60000 });
     }
+    // `waitFor` actually blocks up to the timeout; `isVisible()` would return
+    // immediately and report false while the popup is still rendering.
+    async waitForVisible(locator, timeout) {
+        return locator
+            .first()
+            .waitFor({ state: 'visible', timeout })
+            .then(() => true)
+            .catch(() => false);
+    }
+    // The change-element popup is now tabbed: every `replace.template-*` entry
+    // is assigned to the "Reusable Assets" tab, so connector templates are not
+    // in the DOM at all while the default "BPMN" tab is selected - which is why
+    // the template locators time out with `<element(s) not found>`.
+    // Opening that tab is version-tolerant: on an untabbed popup the strip is
+    // absent and this is a no-op.
+    // Search is the fallback, because it spans the full entry set across every
+    // tab. It must be typed with `pressSequentially`: the popup's search input
+    // is uncontrolled and its state is written by a `keyup` handler only, so
+    // `fill()` (which dispatches `input` alone) never applies the filter.
+    async revealChangeElementTemplate(option, searchTerm) {
+        if (await this.waitForVisible(option, 5000)) {
+            return;
+        }
+        if (await this.waitForVisible(this.changeElementTabStrip, 5000)) {
+            // Do not throw: a failed tab click leaves the search fallback below
+            // intact, and search alone is enough on an untabbed popup.
+            await this.reusableAssetsTab.click({ timeout: 30000 }).catch(() => { });
+            if (await this.waitForVisible(option, 15000)) {
+                return;
+            }
+        }
+        if (!(await this.waitForVisible(this.changeElementSearchInput, 5000))) {
+            return;
+        }
+        await this.changeElementSearchInput.click();
+        await this.changeElementSearchInput.fill('');
+        await this.changeElementSearchInput.pressSequentially(searchTerm, {
+            delay: 50,
+        });
+    }
     async clickRestConnectorOption() {
-        await (0, test_1.expect)(this.restConnectorOption).toBeVisible({ timeout: 15000 });
+        await this.revealChangeElementTemplate(this.restConnectorOption, 'REST');
+        await (0, test_1.expect)(this.restConnectorOption).toBeVisible({ timeout: 90000 });
         await this.restConnectorOption.click();
     }
     async clickWebhookStartEventConnectorOption() {
+        await this.revealChangeElementTemplate(this.webhookMessageStartEventConnectorOption, 'Webhook');
         await this.webhookMessageStartEventConnectorOption.click({ timeout: 60000 });
     }
     async clickWebhookTab() {
@@ -991,6 +1047,7 @@ class ModelerCreatePage {
         await this.intermediateBoundaryEvent.click({ timeout: 60000 });
     }
     async clickIntermediateWebhookConnectorOption() {
+        await this.revealChangeElementTemplate(this.intermediateWebhookConnectorOption, 'Webhook');
         await this.intermediateWebhookConnectorOption.click({ timeout: 60000 });
     }
     async clickCorrelationKeyProcessInput() {

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/tests/8.10/connectors-user-flows.spec.js

@@ -132,6 +132,9 @@ _8_10_1.test.describe('Connectors User Flow Tests @tasklistV2', () => {
             await modelerCreatePage.enterDiagramName(processName);
             await (0, sleep_1.sleep)(10000);
             await modelerCreatePage.clickChangeTypeButton();
+            // Connector templates now sit behind the change-element popup's
+            // "Reusable Assets" tab, so surface the entry before asserting on it.
+            await modelerCreatePage.revealChangeElementTemplate(modelerCreatePage.webhookMessageStartEventConnectorOption, 'Webhook');
             await (0, test_1.expect)(modelerCreatePage.webhookMessageStartEventConnectorOption).toBeVisible({
                 timeout: 30000,
             });

package/package.json

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