@camunda/e2e-test-suite 0.0.1137 → 0.0.1138

package/dist/pages/8.7/PlayPage.js

@@ -136,13 +136,67 @@ class PlayPage {
                 if (attempts >= maxRetries - 1)
                     throw error;
                 await this.page.reload();
+                // A reload drops the Modeler back on the Implement tab (the same
+                // trap clickStartInstanceButton's catch block already handles), so
+                // the retry would otherwise wait out its whole 180s budget against
+                // DOM that can never hold the instance details -- which is why the
+                // nightly still failed on "process instance key" despite this loop.
+                // Re-open Play first.
+                if (await this.playTab.isVisible({ timeout: 30000 }).catch(() => false)) {
+                    await this.playTab.click();
+                }
+                // The details panel only renders once an instance exists. The one way
+                // it never rendered is that the start click resolved against a
+                // re-laid-out (briefly detached) overlay and was silently lost --
+                // clickLocatorWithRetry only retries a click that *throws*. Re-issue
+                // the start so the retry has an instance to show; if one is already
+                // running the assertions below simply pass on it.
+                const startTrigger = this.configureTestPanelStartButton
+                    .or(this.startInstanceOverlayButton)
+                    .first();
+                if (await startTrigger.isVisible({ timeout: 30000 }).catch(() => false)) {
+                    await this.clickStartInstanceButton();
+                }
                 attempts++;
             }
         }
     }
     async waitForNextElementToBeActive(historyItem) {
-        await (0, test_1.expect)(this.page.getByText(new RegExp(`^${historyItem}`, 'i'))).toBeVisible({
+        // Same lost-click hazard waitForProcessToBeCompleted documents for the
+        // final task, one step earlier: the complete-job overlay is pinned to the
+        // active element and is re-laid-out (briefly detached) on every instance
+        // update from the cluster, so a click landing in that window resolves
+        // against a stale node without throwing. clickLocatorWithRetry only
+        // retries a click that *throws*, so the job stays pending and the sole
+        // symptom is this locator timing out on the element that consequently
+        // never became active -- exactly the nightly failure on 'service-task*'.
+        //
+        // Once the next element has stayed absent well past any normal Play
+        // advance latency AND a complete-job overlay is still sitting enabled on
+        // the canvas, the previous click demonstrably did not land, so re-issue
+        // it. The grace period matters: it keeps us from completing the job of
+        // the very element we are waiting for during the brief window where its
+        // overlay has rendered but its history entry has not. The assertion is
+        // unchanged -- this still only passes on a visible history entry.
+        const element = this.page.getByText(new RegExp(`^${historyItem}`, 'i'));
+        const recoveryGraceMs = 60000;
+        const startedAt = Date.now();
+        await (0, test_1.expect)(async () => {
+            if (Date.now() - startedAt > recoveryGraceMs) {
+                const isPreviousJobStillPending = (await this.completeJobButton.isVisible().catch(() => false)) &&
+                    (await this.completeJobButton.isEnabled().catch(() => false));
+                // Re-check immediately before clicking: if the element became active
+                // in the meantime, the pending overlay is its own and must be left
+                // alone for the caller's next step.
+                if (isPreviousJobStillPending &&
+                    !(await element.isVisible().catch(() => false))) {
+                    await this.completeJobButton.click({ timeout: 30000 });
+                }
+            }
+            await (0, test_1.expect)(element).toBeVisible({ timeout: 15000 });
+        }).toPass({
             timeout: maxWaitTimeSeconds,
+            intervals: [5000, 10000, 15000],
         });
     }
     async waitForProcessToBeCompleted() {

package/package.json

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