@camunda/e2e-test-suite 0.0.1145 → 0.0.1147

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

@@ -35,6 +35,7 @@ declare class ModelerCreatePage {
     readonly cancelButton: Locator;
     readonly restConnectorOption: Locator;
     readonly changeElementSearchInput: Locator;
+    readonly changeElementResults: Locator;
     readonly changeElementTabStrip: Locator;
     readonly reusableAssetsTab: Locator;
     readonly marketPlaceButton: Locator;
@@ -176,6 +177,8 @@ declare class ModelerCreatePage {
     expandAssignmentSection(): Promise<void>;
     clickAppendElementButton(): Promise<void>;
     clickParallelGatewayOption(): Promise<void>;
+    private openReplacePopup;
+    private revealReplaceEntry;
     clickConnectToOtherElementButton(): Promise<void>;
     clickPriorityInput(): Promise<void>;
     fillPriorityInput(priority: string): Promise<void>;

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

@@ -42,6 +42,7 @@ class ModelerCreatePage {
     cancelButton;
     restConnectorOption;
     changeElementSearchInput;
+    changeElementResults;
     changeElementTabStrip;
     reusableAssetsTab;
     marketPlaceButton;
@@ -207,6 +208,7 @@ 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.changeElementResults = page.locator('.djs-popup-results');
         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".
@@ -1390,8 +1392,77 @@ class ModelerCreatePage {
         // multiplied the worst-case wait.
         await this.hoverOnLocator(this.appendPreButton);
     }
+    // Owns the whole replace step, popup included: the append leaves the new
+    // (exclusive) gateway selected while the canvas is still panning it into
+    // view, and a forced "Change element" click issued mid-pan lands on empty
+    // canvas -- that clears the selection, closes the context pad, and the
+    // replace popup never opens, so a bare wait on the entry burns its full
+    // timeout with nothing on screen. Pin the shape by id up front so the popup
+    // can be reopened against the same element instead of a positional one.
     async clickParallelGatewayOption() {
-        await this.parallelGatewayOption.click({ timeout: 90000 });
+        const selectedId = await this.page
+            .locator('.djs-element.selected')
+            .first()
+            .getAttribute('data-element-id', { timeout: 10000 })
+            .catch(() => null);
+        const element = selectedId
+            ? this.page
+                .locator(`.djs-element[data-element-id="${selectedId}"]`)
+                .first()
+            : this.page.locator('.djs-element.selected').first();
+        const maxRetries = 3;
+        let lastError;
+        for (let attempt = 1; attempt <= maxRetries; attempt++) {
+            try {
+                if (attempt > 1) {
+                    // Drop a popup left open on a stale search filter (or on the wrong
+                    // element) so the retry reopens a fresh one for `element`.
+                    await this.page.keyboard.press('Escape');
+                }
+                await this.openReplacePopup(element, attempt > 1);
+                await this.revealReplaceEntry(this.parallelGatewayOption, 'Parallel gateway');
+                await this.parallelGatewayOption.click({ timeout: 30000 });
+                // bpmn-js closes the popup once an entry is applied, so a popup still
+                // on screen means the click was swallowed and the element was not
+                // replaced.
+                await (0, test_1.expect)(this.changeElementResults).toBeHidden({ timeout: 30000 });
+                return;
+            }
+            catch (error) {
+                lastError = error;
+                console.error(`Parallel gateway attempt ${attempt} failed: ${error}`);
+                await (0, sleep_1.sleep)(5000);
+            }
+        }
+        throw new Error(`Failed to replace the element with a parallel gateway after ${maxRetries} attempts: ${lastError}`);
+    }
+    async openReplacePopup(element, force) {
+        if (await this.isVisibleWithin(this.changeElementResults, 2000)) {
+            return;
+        }
+        // The context pad only renders for a selected element, so "Change element"
+        // would otherwise never appear.
+        if (!(await this.isVisibleWithin(this.changeTypeButton, 5000))) {
+            await element.click({ force: true, timeout: 30000 });
+        }
+        await this.changeTypeButton.click({ timeout: 60000, force });
+        await (0, test_1.expect)(this.changeElementResults).toBeVisible({ timeout: 30000 });
+    }
+    async revealReplaceEntry(option, searchTerm) {
+        if (await this.isVisibleWithin(option, 5000)) {
+            return;
+        }
+        // The popup lists the project's deployed resources alongside the built-in
+        // replace entries, so the built-in entry can sit below the fold of the
+        // scrolling results list; the popup's own search filters it back into view.
+        if (await this.isVisibleWithin(this.changeElementSearchInput, 5000)) {
+            await this.changeElementSearchInput.click();
+            await this.changeElementSearchInput.fill('');
+            await this.changeElementSearchInput.pressSequentially(searchTerm, {
+                delay: 50,
+            });
+        }
+        await (0, test_1.expect)(option).toBeVisible({ timeout: 30000 });
     }
     async clickConnectToOtherElementButton() {
         await (0, test_1.expect)(this.connectToOtherElementButton).toBeVisible({
@@ -1430,7 +1501,6 @@ class ModelerCreatePage {
             await this.hoverOnLocator(this.appendElementButton);
         }
         await this.clickAppendGatewayButton();
-        await this.clickChangeTypeButton();
         await this.clickParallelGatewayOption();
         await this.clickAppendElementButton();
         await this.clickAppendTaskButton();
@@ -1444,7 +1514,6 @@ class ModelerCreatePage {
         await this.assertImplementationOption('zeebeUserTask');
         await this.clickAppendElementButton();
         await this.clickAppendGatewayButton();
-        await this.clickChangeTypeButton();
         await this.clickParallelGatewayOption();
         for (let i = 2; i <= numberOfTasks; i++) {
             await this.clickFirstPlacedGateway();

package/dist/test-setup.js

@@ -16,7 +16,64 @@ async function captureScreenshot(page, testInfo) {
     if (testInfo.status === 'passed' || testInfo.status === 'skipped') {
         return;
     }
-    const screenshotPath = `test-results/screenshots/screenshot-${(0, crypto_1.randomUUID)()}.png`;
+    // Capture every open tab, not just the one handed in. Flows that reach
+    // Operate/Tasklist through a link (openAppTabWithRetry) assert against a
+    // second Page, so capturing only `page` attaches a picture of the tab the
+    // test had already left: a doc-handling failure asserting the Operate
+    // completed-icon attached the Modeler canvas instead, which is unusable for
+    // triage.
+    let targets = [page];
+    try {
+        targets = [
+            page,
+            ...page
+                .context()
+                .pages()
+                .filter((candidate) => candidate !== page && !candidate.isClosed()),
+        ];
+    }
+    catch (contextError) {
+        console.warn(`captureScreenshot: could not enumerate context pages (${contextError}). ` +
+            'Capturing the handed-in page only.');
+    }
+    let extraTabIndex = 1;
+    for (const target of targets) {
+        const screenshotPath = `test-results/screenshots/screenshot-${(0, crypto_1.randomUUID)()}.png`;
+        const isPrimaryTab = target === page;
+        let captured = false;
+        if (isPrimaryTab) {
+            captured = await screenshotWithViewportFallback(target, screenshotPath);
+        }
+        else {
+            try {
+                await target.screenshot({ path: screenshotPath, timeout: 30000 });
+                captured = true;
+            }
+            catch (error) {
+                console.warn(`captureScreenshot: extra-tab viewport capture failed (${error}). ` +
+                    'Continuing without a screenshot; the test verdict is unaffected.');
+            }
+        }
+        if (!captured) {
+            continue;
+        }
+        const attachmentName = isPrimaryTab
+            ? 'screenshot'
+            : `screenshot-tab-${extraTabIndex++}`;
+        try {
+            await testInfo.attach(attachmentName, {
+                path: screenshotPath,
+                contentType: 'image/png',
+            });
+        }
+        catch (error) {
+            console.warn(`captureScreenshot: failed to attach ${attachmentName} (${error}). ` +
+                'Continuing without the attachment; the test verdict is unaffected.');
+        }
+    }
+}
+exports.captureScreenshot = captureScreenshot;
+async function screenshotWithViewportFallback(page, screenshotPath) {
     try {
         await page.screenshot({
             path: screenshotPath,
@@ -33,15 +90,11 @@ async function captureScreenshot(page, testInfo) {
         catch (viewportError) {
             console.warn(`captureScreenshot: viewport capture also failed (${viewportError}). ` +
                 'Continuing without a screenshot; the test verdict is unaffected.');
-            return;
+            return false;
         }
     }
-    await testInfo.attach('screenshot', {
-        path: screenshotPath,
-        contentType: 'image/png',
-    });
+    return true;
 }
-exports.captureScreenshot = captureScreenshot;
 async function captureFailureVideo(page, testInfo) {
     // Same policy as captureScreenshot above: skip on passed/skipped rather
     // than requiring literally 'failed', so timed-out/interrupted tests keep

package/package.json

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