@camunda/e2e-test-suite 0.0.1038 → 0.0.1039

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

@@ -155,6 +155,7 @@ declare class ModelerCreatePage {
     clickDiagramBreadcrumb(): Promise<void>;
     clickEditDiagramNameButton(): Promise<void>;
     enterDiagramName(name: string): Promise<void>;
+    private diagramNameEditorOpen;
     clickVariableInput(): Promise<void>;
     fillVariableInput(variable: string): Promise<void>;
     clickEmbedFormButton(): Promise<void>;

package/dist/pages/8.10/ModelerCreatePage.js

@@ -799,25 +799,42 @@ class ModelerCreatePage {
     async enterDiagramName(name) {
         const maxAttempts = 3;
         for (let attempt = 0; attempt < maxAttempts; attempt++) {
-            if (attempt > 0) {
-                await this.clickDiagramBreadcrumb();
-                await this.clickEditDiagramNameButton();
-            }
-            await this.diagramNameInput.fill(name, { timeout: 90000 });
-            await this.diagramNameInput.press('Enter', { timeout: 90000 });
             try {
+                // Creating the diagram does not reliably leave the name editor open --
+                // hub opens the editor on the breadcrumb, and a slow canvas render
+                // lands on a closed one. The input therefore has to be opened on the
+                // FIRST attempt too, not only on a retry: reaching for it blind burnt
+                // the whole 90s fill timeout and threw straight out of this loop, so
+                // the reopen below (which exists for exactly this) never ran.
+                // clickEditDiagramNameButton() reopens the breadcrumb menu itself when
+                // it is closed, so it covers both the first attempt and a retry.
+                if (!(await this.diagramNameEditorOpen())) {
+                    await this.clickEditDiagramNameButton();
+                }
+                // 20s rather than 90s: the editor is proven open at this point, so the
+                // long budget only delayed the retry that actually recovers -- and at
+                // 90s x 2 x 3 attempts this loop could outlive the 12min test timeout,
+                // losing the artifacts that explain the failure.
+                await this.diagramNameInput.fill(name, { timeout: 20000 });
+                await this.diagramNameInput.press('Enter', { timeout: 20000 });
                 await this.diagramNameBreadcrumb(name).waitFor({
                     state: 'visible',
                     timeout: 30000,
                 });
                 return;
             }
-            catch {
-                console.warn(`Diagram rename to "${name}" did not commit on attempt ${attempt + 1}.`);
+            catch (error) {
+                console.warn(`Diagram rename to "${name}" did not commit on attempt ${attempt + 1}: ${error instanceof Error ? error.message : error}`);
             }
         }
         throw new Error(`Diagram name was not set to "${name}".`);
     }
+    async diagramNameEditorOpen() {
+        return this.diagramNameInput
+            .waitFor({ state: 'visible', timeout: 5000 })
+            .then(() => true)
+            .catch(() => false);
+    }
     async clickVariableInput() {
         await this.variableInput.click({ timeout: 60000 });
     }

package/dist/pages/8.10/ModelerHomePage.js

@@ -477,6 +477,16 @@ class ModelerHomePage {
                 await this.clickMessageBanner();
             },
             postAction: async () => {
+                // The project tile only renders on the Modeler home page. A failure
+                // that leaves the browser inside the project makes the tile unreachable
+                // by construction, so reloading the same URL cannot help and every
+                // remaining attempt just waits out its 60s visibility timeout before
+                // the helper throws. Step back out to home first.
+                const tileVisible = await this.becameVisible(this.crossComponentProjectFolder, 2000);
+                if (!tileVisible &&
+                    (await this.becameVisible(this.homeBreadcrumb, 5000))) {
+                    await this.homeBreadcrumb.click({ timeout: 15000 });
+                }
                 await this.page.reload();
                 await this.page.waitForLoadState('domcontentloaded');
             },

package/dist/pages/8.10/ProcessApplicationPage.js

@@ -86,10 +86,17 @@ class ProcessApplicationPage {
             exact: true,
         });
     }
-    // Either the folder explorer or the breadcrumb proves we are on the process
-    // application page; .first() because both are present once it has rendered.
+    // Only the breadcrumb proves we are on the process application page. The
+    // folder explorer does NOT: hub renders a `region[name="Folder explorer"]` on
+    // the enclosing PROJECT page too (trace snapshots on a
+    // `...--cross-component-test-project` URL carry one), so accepting it let
+    // this resolve on the project page whenever creation had not actually landed.
+    // ModelerHomePage then remembered the project URL as the container URL and
+    // every later navigation went back to the project page, where the
+    // container's `[data-test="diagram-dropdown"]` create menu does not exist --
+    // surfacing 60s later as `element(s) not found` on an unrelated step.
     pageReady() {
-        return this.folderExplorer.or(this.breadcrumb).first();
+        return this.breadcrumb;
     }
     async waitForProcessApplication(timeout = 90000) {
         await (0, test_1.expect)(this.pageReady()).toBeVisible({ timeout });

package/package.json

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