@camunda/e2e-test-suite 0.0.1003 → 0.0.1005

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

@@ -42,6 +42,7 @@ declare class ModelerCreatePage {
     readonly firstPlacedElement: Locator;
     readonly secondPlacedElement: Locator;
     readonly appendElementButton: Locator;
+    readonly appendIndicator: Locator;
     readonly payloadInput: Locator;
     readonly marketPlaceButton: Locator;
     readonly publicHolidayConnectorOption: Locator;
@@ -176,6 +177,7 @@ declare class ModelerCreatePage {
     clickFirstPlacedElement(): Promise<void>;
     clickSecondPlacedElement(): Promise<void>;
     clickSecondElement(): Promise<void>;
+    openAppendCreatePad(): Promise<void>;
     clickAppendElementButton(): Promise<void>;
     clickMarketPlaceButton(): Promise<void>;
     clickPublicHolidayConnectorOption(): Promise<void>;

package/dist/pages/8.8/ModelerCreatePage.js

@@ -48,6 +48,7 @@ class ModelerCreatePage {
     firstPlacedElement;
     secondPlacedElement;
     appendElementButton;
+    appendIndicator;
     payloadInput;
     marketPlaceButton;
     publicHolidayConnectorOption;
@@ -120,6 +121,10 @@ class ModelerCreatePage {
         this.appendElementButton = page
             .locator('[class="djs-create-pad open djs-append-create-pad"]')
             .first();
+        // Every element keeps an indicator overlay in the DOM and hides the ones
+        // that do not apply, so match on the visible call-to-action indicator —
+        // the one belonging to the currently selected element.
+        this.appendIndicator = page.locator('.djs-append-indicator.selected:visible');
         this.appendTaskButton = page.getByRole('button', { name: 'Append task' });
         this.changeTypeButton = page.getByRole('button', {
             name: 'Change element',
@@ -510,6 +515,7 @@ class ModelerCreatePage {
             else if (parentElement == 'parallelGateway') {
                 await this.clickSecondPlacedGateway();
             }
+            await this.openAppendCreatePad();
             await this.appendElementButton.hover();
             await this.appendEndEventButton.click({ timeout: 120000 });
         }
@@ -883,6 +889,13 @@ class ModelerCreatePage {
         await this.endEventCanvas.click({ timeout: 60000 });
     }
     async clickConnectToOtherElementButton() {
+        // 'Connect to other element' is a create-pad entry
+        // (`data-entry-id="connect"`), so the pad has to be open before the entry
+        // exists at all. The element this is called on already has an outgoing
+        // sequence flow, which is exactly the case that no longer auto-opens the
+        // pad since improved-canvas 1.11.0 — go through the append indicator
+        // first, which is what clickAppendElementButton does (with its retries).
+        await this.clickAppendElementButton();
         await (0, test_1.expect)(this.connectToOtherElementButton).toBeVisible({
             timeout: 60000,
         });
@@ -899,11 +912,35 @@ class ModelerCreatePage {
         await (0, test_1.expect)(this.secondElement).toBeVisible({ timeout: 60000 });
         await this.secondElement.click({ timeout: 60000 });
     }
+    // Selecting an element does not open the append create pad on its own. The
+    // improved canvas gives the selected element a "+" overlay
+    // (.djs-append-indicator) instead, and only a mouseenter on that indicator
+    // opens the pad. The pad still opens by itself right after an append, and
+    // the indicator is hidden while the pad is open, so both lookups are
+    // best-effort and this is a no-op in that case.
+    async openAppendCreatePad() {
+        const padAlreadyOpen = await this.appendElementButton
+            .isVisible()
+            .catch(() => false);
+        if (padAlreadyOpen) {
+            return;
+        }
+        const indicator = this.appendIndicator.first();
+        const indicatorShown = await indicator
+            .waitFor({ state: 'visible', timeout: 10000 })
+            .then(() => true)
+            .catch(() => false);
+        if (!indicatorShown) {
+            return;
+        }
+        await indicator.hover({ force: true, timeout: 30000 });
+    }
     async clickAppendElementButton() {
         const maxRetries = 3;
         for (let retries = 0; retries < maxRetries; retries++) {
             try {
                 await this.page.waitForTimeout(1000);
+                await this.openAppendCreatePad();
                 await (0, test_1.expect)(this.appendElementButton).toBeVisible({ timeout: 10000 });
                 await this.appendElementButton.hover({ force: true });
                 return;
@@ -1117,6 +1154,7 @@ class ModelerCreatePage {
     async clickFirstPlacedGateway() {
         try {
             await this.firstPlacedGateway.click({ timeout: 60000 });
+            await this.openAppendCreatePad();
             await (0, test_1.expect)(this.appendElementButton).toBeVisible({ timeout: 60000 });
         }
         catch (error) {
@@ -1153,6 +1191,7 @@ class ModelerCreatePage {
             await this.page.reload();
             await (0, sleep_1.sleep)(20000);
             await this.clickFirstPlacedElement();
+            await this.openAppendCreatePad();
             await this.appendElementButton.hover();
         }
         await this.clickAppendGatewayButton();

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

@@ -40,6 +40,7 @@ declare class ModelerCreatePage {
     readonly firstPlacedElement: Locator;
     readonly secondPlacedElement: Locator;
     readonly appendElementButton: Locator;
+    readonly appendIndicator: Locator;
     readonly payloadInput: Locator;
     readonly marketPlaceButton: Locator;
     readonly publicHolidayConnectorOption: Locator;
@@ -176,6 +177,7 @@ declare class ModelerCreatePage {
     clickFirstPlacedElement(): Promise<void>;
     clickSecondPlacedElement(): Promise<void>;
     clickSecondElement(): Promise<void>;
+    openAppendCreatePad(): Promise<void>;
     clickAppendElementButton(): Promise<void>;
     clickMarketPlaceButton(): Promise<void>;
     clickPublicHolidayConnectorOption(): Promise<void>;

package/dist/pages/8.9/ModelerCreatePage.js

@@ -46,6 +46,7 @@ class ModelerCreatePage {
     firstPlacedElement;
     secondPlacedElement;
     appendElementButton;
+    appendIndicator;
     payloadInput;
     marketPlaceButton;
     publicHolidayConnectorOption;
@@ -120,6 +121,7 @@ class ModelerCreatePage {
         this.appendElementButton = page
             .locator('[class="djs-create-pad open djs-append-create-pad"]')
             .first();
+        this.appendIndicator = page.locator('.djs-append-indicator.selected');
         this.appendTaskButton = page.getByRole('button', { name: 'Append task' });
         this.changeTypeButton = page.getByRole('button', {
             name: 'Change element',
@@ -513,6 +515,7 @@ class ModelerCreatePage {
             else if (parentElement == 'parallelGateway') {
                 await this.clickSecondPlacedGateway();
             }
+            await this.openAppendCreatePad();
             await this.appendElementButton.hover();
             await this.appendEndEventButton.click({ timeout: 120000 });
         }
@@ -835,6 +838,13 @@ class ModelerCreatePage {
         await this.endEventCanvas.click({ timeout: 60000 });
     }
     async clickConnectToOtherElementButton() {
+        // 'Connect to other element' is a create-pad entry
+        // (`data-entry-id="connect"`), so the pad has to be open before the entry
+        // exists at all. The element this is called on already has an outgoing
+        // sequence flow, which is exactly the case that no longer auto-opens the
+        // pad since improved-canvas 1.11.0 — go through the append indicator
+        // first, which is what clickAppendElementButton does (with its retries).
+        await this.clickAppendElementButton();
         await (0, test_1.expect)(this.connectToOtherElementButton).toBeVisible({
             timeout: 60000,
         });
@@ -851,11 +861,36 @@ class ModelerCreatePage {
         await (0, test_1.expect)(this.secondElement).toBeVisible({ timeout: 60000 });
         await this.secondElement.click({ timeout: 60000 });
     }
+    // Selecting an element no longer opens the append create pad on its own.
+    // @camunda/improved-canvas gives the selected element a "+" overlay
+    // (.djs-append-indicator) instead, and only a mouseenter on that indicator
+    // opens the pad (AppendIndicator#_openMenu) — camunda-hub's own e2e suite
+    // does exactly this in its `openAppend` cypress command. The pad still opens
+    // by itself right after an append, and the indicator hides once the pad is
+    // open, so both lookups are best-effort and this is a no-op in that case.
+    async openAppendCreatePad() {
+        const padAlreadyOpen = await this.appendElementButton
+            .isVisible()
+            .catch(() => false);
+        if (padAlreadyOpen) {
+            return;
+        }
+        const indicator = this.appendIndicator.first();
+        const indicatorShown = await indicator
+            .waitFor({ state: 'visible', timeout: 10000 })
+            .then(() => true)
+            .catch(() => false);
+        if (!indicatorShown) {
+            return;
+        }
+        await indicator.hover({ force: true, timeout: 30000 });
+    }
     async clickAppendElementButton() {
         const maxRetries = 3;
         for (let retries = 0; retries < maxRetries; retries++) {
             try {
                 await this.page.waitForTimeout(1000);
+                await this.openAppendCreatePad();
                 await (0, test_1.expect)(this.appendElementButton).toBeVisible({ timeout: 10000 });
                 await this.appendElementButton.hover({ force: true });
                 return;
@@ -1065,6 +1100,7 @@ class ModelerCreatePage {
     async clickFirstPlacedGateway() {
         try {
             await this.firstPlacedGateway.click({ timeout: 60000 });
+            await this.openAppendCreatePad();
             await (0, test_1.expect)(this.appendElementButton).toBeVisible({ timeout: 60000 });
         }
         catch (error) {
@@ -1101,6 +1137,7 @@ class ModelerCreatePage {
             await this.page.reload();
             await (0, sleep_1.sleep)(20000);
             await this.clickFirstPlacedElement();
+            await this.openAppendCreatePad();
             await this.appendElementButton.hover();
         }
         await this.clickAppendGatewayButton();

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

@@ -311,10 +311,7 @@ _8_8_1.test.describe('HTO User Flow Tests', () => {
             await (0, expectTextWithPagination_1.expectTextWithPagination)(page, processName + '2');
         });
     });
-    // Temporarily skipped pending fix PR #3144:
-    // https://github.com/camunda/c8-cross-component-e2e-tests/pull/3144
-    // Re-enable in the same change that lands #3144.
-    _8_8_1.test.skip('Zeebe User Task With Priority', async ({ page, homePage, modelerHomePage, appsPage, modelerCreatePage, taskPanelPage, taskDetailsPage, operateHomePage, operateProcessesPage, operateProcessInstancePage, }) => {
+    (0, _8_8_1.test)('Zeebe User Task With Priority', async ({ page, homePage, modelerHomePage, appsPage, modelerCreatePage, taskPanelPage, taskDetailsPage, operateHomePage, operateProcessesPage, operateProcessInstancePage, }) => {
         const randomName = await (0, _setup_1.generateRandomStringAsync)(3);
         const processName = 'Zeebe_User_Task_Process_With_Priority' + randomName;
         const taskName = 'priorityTest' + randomName;

package/dist/tests/8.8/smoke-tests.spec.js

@@ -37,10 +37,7 @@ _8_8_1.test.describe('Smoke Tests', () => {
         await (0, _setup_1.captureScreenshot)(page, testInfo);
         await (0, _setup_1.captureFailureVideo)(page, testInfo);
     });
-    // Temporarily skipped pending fix PR #3144:
-    // https://github.com/camunda/c8-cross-component-e2e-tests/pull/3144
-    // Re-enable in the same change that lands #3144.
-    _8_8_1.test.skip('Most Common Flow User Flow With All Apps', async ({ page, homePage, modelerHomePage, appsPage, modelerCreatePage, formJsPage, }) => {
+    (0, _8_8_1.test)('Most Common Flow User Flow With All Apps', async ({ page, homePage, modelerHomePage, appsPage, modelerCreatePage, formJsPage, }) => {
         _8_8_1.test.slow();
         const randomString = useFixedNames
             ? 'UpgradeTest'

package/dist/tests/8.9/hto-user-flows.spec.js

@@ -301,10 +301,7 @@ _8_9_1.test.describe('HTO User Flow Tests', () => {
             await (0, expectTextWithPagination_1.expectTextWithPagination)(page, processName + '2');
         });
     });
-    // Temporarily skipped pending fix PR #3144:
-    // https://github.com/camunda/c8-cross-component-e2e-tests/pull/3144
-    // Re-enable in the same change that lands #3144.
-    _8_9_1.test.skip('Zeebe User Task With Priority', async ({ page, homePage, modelerHomePage, appsPage, modelerCreatePage, taskPanelPage, taskDetailsPage, operateHomePage, operateProcessesPage, operateProcessInstancePage, }) => {
+    (0, _8_9_1.test)('Zeebe User Task With Priority', async ({ page, homePage, modelerHomePage, appsPage, modelerCreatePage, taskPanelPage, taskDetailsPage, operateHomePage, operateProcessesPage, operateProcessInstancePage, }) => {
         const randomName = await (0, _setup_1.generateRandomStringAsync)(3);
         const processName = 'Zeebe_User_Task_Process_With_Priority' + randomName;
         const taskName = 'priorityTest' + randomName;

package/dist/tests/8.9/smoke-tests.spec.js

@@ -36,10 +36,7 @@ _8_9_1.test.describe('Smoke Tests', () => {
         await (0, _setup_1.captureScreenshot)(page, testInfo);
         await (0, _setup_1.captureFailureVideo)(page, testInfo);
     });
-    // Temporarily skipped pending fix PR #3144:
-    // https://github.com/camunda/c8-cross-component-e2e-tests/pull/3144
-    // Re-enable in the same change that lands #3144.
-    _8_9_1.test.skip('Most Common Flow User Flow With All Apps', async ({ page, homePage, modelerHomePage, appsPage, modelerCreatePage, formJsPage, }) => {
+    (0, _8_9_1.test)('Most Common Flow User Flow With All Apps', async ({ page, homePage, modelerHomePage, appsPage, modelerCreatePage, formJsPage, }) => {
         _8_9_1.test.slow();
         const randomString = useFixedNames
             ? 'UpgradeTest'

package/package.json

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