@camunda/e2e-test-suite 0.0.970 → 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/package.json

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