@camunda/e2e-test-suite 0.0.965 → 0.0.967

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

@@ -26,6 +26,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;
@@ -154,6 +157,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.8/ModelerCreatePage.js

@@ -32,6 +32,9 @@ class ModelerCreatePage {
     additionalActionsButton;
     cancelButton;
     restConnectorOption;
+    changeElementSearchInput;
+    changeElementTabStrip;
+    reusableAssetsTab;
     webhookMessageStartEventConnectorOption;
     webhookTab;
     webhookIsActiveButton;
@@ -170,6 +173,12 @@ 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.
+        this.reusableAssetsTab = page.getByRole('tab', { name: /reusable assets/i });
         // Change-element popup entries are `role="option"` under
         // `.djs-popup-results`, and their accessible name now includes the entry
         // description, so no role+name matcher resolves them. Target the stable
@@ -717,10 +726,50 @@ 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)) {
+            await this.reusableAssetsTab.click({ timeout: 30000 });
+            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 this.revealChangeElementTemplate(this.restConnectorOption, 'REST');
         await this.restConnectorOption.click({ timeout: 60000 });
     }
     async clickWebhookStartEventConnectorOption() {
+        await this.revealChangeElementTemplate(this.webhookMessageStartEventConnectorOption, 'Webhook');
         await this.webhookMessageStartEventConnectorOption.click({ timeout: 60000 });
     }
     async clickWebhookTab() {
@@ -992,6 +1041,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/PlayPage.js

@@ -251,16 +251,21 @@ class PlayPage {
         await (0, test_1.expect)(this.scenarioSavedToast).toBeVisible({ timeout: 30000 });
     }
     async assertSavedScenarioNameInPanel(scenarioName) {
-        // After save the detail view stays open on the just-saved test case. It
-        // renders the name as a TestName heading only when the environment is
-        // read-only; otherwise the name lives in the editable Name field (see
-        // TestCaseDetail/index.tsx, gated on testEnvironmentStore.isReadOnly).
-        // Assert the exact name either way.
-        const heading = this.page.getByRole('heading', {
-            name: scenarioName,
-            exact: true,
-        });
-        if (await heading.isVisible({ timeout: 3000 }).catch(() => false)) {
+        // A successful save tears the detail view down: use-test-case-editor.ts
+        // saveTest() calls resetForm() + testCasesStore.resetTestCaseDetail(),
+        // which clears selectedTestCase and stops recording, so the store's
+        // isTestCaseDetailVisible turns false and the panel falls back to the
+        // saved test-case list. The name therefore lands on one of three
+        // surfaces: a list row, the read-only TestName heading, or (pre-reset
+        // hub builds) the still-editable Name field.
+        const savedName = this.getScenarioRow(scenarioName)
+            .or(this.page.getByRole('heading', { name: scenarioName, exact: true }))
+            .first();
+        const onListOrHeading = await savedName
+            .waitFor({ state: 'visible', timeout: 30000 })
+            .then(() => true)
+            .catch(() => false);
+        if (onListOrHeading) {
             return;
         }
         // Updating an existing test case does NOT always leave the detail view

package/dist/tests/8.8/connectors-user-flows.spec.js

@@ -125,6 +125,9 @@ _8_8_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/dist/tests/8.8/hto-user-flows.spec.js

@@ -16,6 +16,7 @@ const resetSession_1 = require("../../utils/resetSession");
 const consoleApiHelpers_1 = require("../../utils/consoleApiHelpers");
 const expectTextWithPagination_1 = require("../../utils/assertionHelpers/expectTextWithPagination");
 const expectTextWithRetry_1 = require("../../utils/assertionHelpers/expectTextWithRetry");
+const expectLocatorWithRetry_1 = require("../../utils/assertionHelpers/expectLocatorWithRetry");
 const users_1 = require("../../utils/users");
 const testUser = (0, users_1.getTestUser)('nineteenthUser');
 _8_8_1.test.describe.configure({ mode: 'parallel' });
@@ -248,8 +249,20 @@ _8_8_1.test.describe('HTO User Flow Tests', () => {
             await (0, test_1.expect)(page.getByText('"testValue"')).not.toBeVisible({
                 timeout: 60000,
             });
-            await (0, test_1.expect)(page.getByText('"updatedValue"')).toBeVisible({
-                timeout: 90000,
+            // Operate's variables panel stops polling after its last search, so a
+            // stale read right after the update pins the old value on screen for
+            // the rest of the wait: the nightly trace shows the PUT succeed, then
+            // `/v2/variables/search` return "updatedValue" and, 100ms later, a
+            // second query return "testValue" again - and that stale response is
+            // the one the panel keeps rendering. Reload between attempts to force
+            // a re-fetch instead of waiting on a poll that never comes.
+            await (0, expectLocatorWithRetry_1.expectLocatorWithRetry)(page, page.getByText('"updatedValue"'), {
+                totalTimeout: 180000,
+                visibilityTimeout: 30000,
+                maxRetries: 5,
+                postAction: async () => {
+                    await page.reload();
+                },
             });
             await appsPage.clickCamundaApps();
             await appsPage.clickTasklist(clusterName);

package/package.json

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