@camunda/e2e-test-suite 0.0.899 → 0.0.901

package/dist/pages/8.10/ModelerCreatePage.js

@@ -128,7 +128,13 @@ class ModelerCreatePage {
         // labelled with the diagram name, which makes a role+name match on
         // 'User task' ambiguous as soon as such a diagram exists.
         this.userTaskOption = page.locator('.djs-popup-results [data-id="replace-with-user-task"]');
-        this.priorityInput = page.getByRole('textbox', { name: 'priority' });
+        // The user-task priority field renders as
+        // `<input type="number" id="bio-properties-panel-priorityDefinitionPriority">`,
+        // whose ARIA role is `spinbutton`, not `textbox`. A role=textbox query only
+        // matches it while the entry is flipped into FEEL mode (a CodeMirror
+        // editor), which is not a state this flow should depend on. Target the
+        // number input by its stable id instead.
+        this.priorityInput = page.locator('#bio-properties-panel-priorityDefinitionPriority');
         this.appendEndEventButton = page.getByRole('button', {
             name: 'Append end event',
         });
@@ -507,11 +513,18 @@ class ModelerCreatePage {
         await this.page.waitForTimeout(1000);
     }
     async clickPriorityInput() {
+        // The properties panel remembers which groups are open, so from the second
+        // user task onwards Assignment is already expanded. The previous check used
+        // an invalid descendant selector ('...-group-header empty open') that always
+        // counted 0, so every call clicked the group again — collapsing it or landing
+        // on an arbitrary entry. Detect the header's real 'open' class instead and
+        // toggle via the header rather than the whole group box.
+        const assignmentHeader = this.assignmentSection.locator('.bio-properties-panel-group-header');
         const isExpanded = (await this.assignmentSection
-            .locator('.bio-properties-panel-group-header empty open')
+            .locator('.bio-properties-panel-group-header.open')
             .count()) > 0;
         if (!isExpanded) {
-            await this.assignmentSection.click();
+            await assignmentHeader.click({ timeout: 30000 });
         }
         await this.priorityInput.click({ timeout: 30000 });
     }

package/dist/pages/8.7/ModelerCreatePage.js

@@ -199,7 +199,10 @@ class ModelerCreatePage {
         this.formLinkingTypeDropdown = page.getByLabel('Type');
         this.formLinkTypeOptions = page.locator('#bio-properties-panel-formType');
         this.addFormJsonConfigurationInput = page.getByLabel('Form JSON configuration');
-        this.priorityInput = page.getByRole('textbox', { name: 'priority' });
+        // The user-task priority field renders as `<input type="number">`, whose
+        // ARIA role is `spinbutton`, not `textbox`, so no `textbox` query matches
+        // it. Target it by id instead, as pages/SM-8.10 already does.
+        this.priorityInput = page.locator('#bio-properties-panel-priorityDefinitionPriority');
         this.closeButton = page.getByRole('button', { name: 'Close' });
         this.rbaEnabledDeployedDialog = page.getByRole('dialog', {
             name: 'Process successfully deployed',
@@ -345,11 +348,17 @@ class ModelerCreatePage {
         await (0, sleep_1.sleep)(20000);
     }
     async expandAssignmentSection() {
+        // `assignmentSection` is already the `group-assignmentDefinition` element,
+        // so the header must be matched relative to it — repeating the group
+        // selector looked for a nested group that never exists, which made the
+        // check always report "collapsed" and toggled an open section shut.
         const isExpanded = (await this.assignmentSection
-            .locator('[data-group-id="group-assignmentDefinition"] .bio-properties-panel-group-header.open')
+            .locator('.bio-properties-panel-group-header.open')
             .count()) > 0;
         if (!isExpanded) {
-            await this.assignmentSection.click({ timeout: 60000 });
+            await this.assignmentSection
+                .locator('.bio-properties-panel-group-header')
+                .click({ timeout: 60000 });
         }
     }
     async clickAssigneeInput() {
@@ -965,12 +974,7 @@ class ModelerCreatePage {
         await this.page.waitForTimeout(1000);
     }
     async clickPriorityInput() {
-        const isExpanded = (await this.assignmentSection
-            .locator('.bio-properties-panel-group-header empty open')
-            .count()) > 0;
-        if (!isExpanded) {
-            await this.assignmentSection.click();
-        }
+        await this.expandAssignmentSection();
         await this.priorityInput.click({ timeout: 30000 });
     }
     async setPriority(taskName, priority, numberOfTasks) {

package/dist/pages/8.9/ModelerCreatePage.js

@@ -127,7 +127,13 @@ class ModelerCreatePage {
         // name, so a role+name match on 'User task' is ambiguous as soon as a
         // diagram is called e.g. "Optimize Job Worker User Task Diagram".
         this.userTaskOption = page.locator('.djs-popup-results [data-id="replace-with-user-task"]');
-        this.priorityInput = page.getByRole('textbox', { name: 'priority' });
+        // The user-task priority field renders as
+        // `<input type="number" id="bio-properties-panel-priorityDefinitionPriority">`,
+        // whose ARIA role is `spinbutton`, not `textbox`. A role=textbox query only
+        // matches it while the entry is flipped into FEEL mode (a CodeMirror
+        // editor), which is not a state this flow should depend on. Target the
+        // number input by its stable id instead.
+        this.priorityInput = page.locator('#bio-properties-panel-priorityDefinitionPriority');
         this.appendEndEventButton = page.getByRole('button', {
             name: 'Append end event',
         });
@@ -505,11 +511,19 @@ class ModelerCreatePage {
         await this.page.waitForTimeout(1000);
     }
     async clickPriorityInput() {
+        // `.bio-properties-panel-group-header empty open` is a descendant query for
+        // <empty><open> elements, so it always counted 0 and every call clicked the
+        // group container. Only the header toggles the group, so a container click
+        // lands on whichever entry sits at its centre once expanded — including an
+        // entry's `fx` FEEL toggle. Probe the header's own `open` class and click
+        // the header, so expanding is correct and idempotent across the loop.
         const isExpanded = (await this.assignmentSection
-            .locator('.bio-properties-panel-group-header empty open')
+            .locator('.bio-properties-panel-group-header.open')
             .count()) > 0;
         if (!isExpanded) {
-            await this.assignmentSection.click();
+            await this.assignmentSection
+                .locator('.bio-properties-panel-group-header')
+                .click({ timeout: 30000 });
         }
         await this.priorityInput.click({ timeout: 30000 });
     }

package/package.json

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