@camunda/e2e-test-suite 0.0.1043 → 0.0.1045

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

@@ -23,6 +23,7 @@ declare class ModelerHomePage {
     readonly crossComponentProjectFolder: Locator;
     readonly homeBreadcrumb: Locator;
     readonly formNameInput: Locator;
+    readonly formEditorCanvas: Locator;
     readonly rows: Locator;
     readonly licenseKeyTagNonProduction: Locator;
     readonly licenseKeyTagNonCommercial: Locator;

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

@@ -39,6 +39,7 @@ class ModelerHomePage {
     crossComponentProjectFolder;
     homeBreadcrumb;
     formNameInput;
+    formEditorCanvas;
     rows;
     licenseKeyTagNonProduction;
     licenseKeyTagNonCommercial;
@@ -129,6 +130,11 @@ class ModelerHomePage {
             .or(page.getByRole('button', { name: 'My organisation' }))
             .first();
         this.formNameInput = page.locator('[data-test="editable-input"]');
+        // form-js labels its editor region "Form Definition" on both shells (see
+        // FormJsPage.formEditor). Unlike the app header, it mounts only once the
+        // editor itself has rendered, which makes it the ready signal enterFormName
+        // gates on.
+        this.formEditorCanvas = page.getByLabel('Form Definition').first();
         this.rows = page.getByRole('row');
         this.licenseKeyTagNonProduction = page
             .getByText('Non-production license')
@@ -343,9 +349,15 @@ class ModelerHomePage {
         // renamed the PROJECT to the form's name instead of the form. The Carbon
         // shell could not hit this: editable-input existed only inside the editor,
         // so the gate implicitly waited for navigation.
+        //
+        // waitForURL is not enough on its own: the URL flips to /forms/ before the
+        // editor mounts, and the app header -- kebab included -- is already painted
+        // by then, still showing the process application as the deepest crumb. The
+        // canvas mounts together with the entity crumb, so gate on it rather than
+        // on the header, which is present either way.
         await this.page.waitForURL(/\/forms\//, { timeout: 60000 });
         await this.formNameInput
-            .or(this.moreActionsButton)
+            .or(this.formEditorCanvas)
             .first()
             .waitFor({ state: 'visible', timeout: 30000 });
         if (await this.formNameInput.isVisible()) {

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

@@ -23,6 +23,7 @@ declare class OCTenantPage {
     readonly assignRoleButton: Locator;
     readonly roleIdSearchBox: Locator;
     readonly assignRoleSubButton: Locator;
+    readonly assignRoleModal: Locator;
     readonly cancelButton: Locator;
     readonly gotItButton: Locator;
     readonly filterTableSearchbox: Locator;
@@ -64,6 +65,7 @@ declare class OCTenantPage {
     clickRoleIdSearchBox(): Promise<void>;
     fillRoleIdSearchBox(roleId: string, roleIdentifier: string): Promise<void>;
     clickAssignRoleSubButton(): Promise<void>;
+    private isRoleAssigned;
     assignRoleToTenant(roleId: string, roleIdentifier: string): Promise<void>;
 }
 export { OCTenantPage };

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

@@ -27,6 +27,7 @@ class OCTenantPage {
     assignRoleButton;
     roleIdSearchBox;
     assignRoleSubButton;
+    assignRoleModal;
     cancelButton;
     gotItButton;
     filterTableSearchbox;
@@ -72,9 +73,10 @@ class OCTenantPage {
         this.roleIdSearchBox = page.getByRole('searchbox', {
             name: 'Search by Role ID',
         });
-        this.assignRoleSubButton = page
-            .getByLabel('Assign role')
-            .getByRole('button', { name: 'Assign role' });
+        this.assignRoleModal = page.getByLabel('Assign role');
+        this.assignRoleSubButton = this.assignRoleModal.getByRole('button', {
+            name: 'Assign role',
+        });
         this.cancelButton = page
             .getByRole('dialog')
             .getByRole('button', { name: 'Cancel' });
@@ -263,7 +265,10 @@ class OCTenantPage {
         await this.roleIdSearchBox.click();
     }
     async fillRoleIdSearchBox(roleId, roleIdentifier) {
-        const result = this.page.getByText(roleIdentifier).first();
+        // Scoped to the modal: the tenant's own roles table sits behind it and
+        // carries the same role text, so an unscoped match resolves to that row
+        // and its click is intercepted by the modal overlay.
+        const result = this.assignRoleModal.getByText(roleIdentifier).first();
         // The dropdown matches against the roles fetched with an empty search
         // filter, so clearing the box re-triggers that fetch before retyping.
         await (0, test_1.expect)(async () => {
@@ -277,10 +282,21 @@ class OCTenantPage {
     async clickAssignRoleSubButton() {
         await this.assignRoleSubButton.click();
     }
+    async isRoleAssigned(roleId) {
+        return this.row(roleId)
+            .waitFor({ state: 'visible', timeout: 15000 })
+            .then(() => true)
+            .catch(() => false);
+    }
     async assignRoleToTenant(roleId, roleIdentifier) {
         await this.clickRolesTab();
         await this.page.waitForLoadState('networkidle');
-        if (await this.row(roleId).isVisible()) {
+        // The roles table renders from its own fetch, which `networkidle` can
+        // resolve ahead of, so an immediate `isVisible()` reads the table as empty
+        // and reopens the modal for a role that is already assigned. The assign
+        // dropdown never offers an already-assigned role, so that attempt can
+        // never converge.
+        if (await this.isRoleAssigned(roleId)) {
             console.log(`Role ${roleId} is already assigned to tenant`);
             return;
         }

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

@@ -47,14 +47,14 @@ class OperateProcessInstancePage {
                 return; // Exit the function if the expectation is met
             }
             catch (error) {
-                // If the active icon isn't found, reload the page and try again
+                // If the completed icon isn't found, reload the page and try again
                 retryCount++;
                 console.log(`Attempt ${retryCount} failed. Retrying...`);
                 await this.page.reload();
                 await this.page.waitForLoadState('domcontentloaded');
             }
         }
-        throw new Error(`Active icon not visible after ${maxRetries} attempts.`);
+        throw new Error(`Completed icon not visible after ${maxRetries} attempts.`);
     }
     async activeIconAssertion() {
         let retryCount = 0;

package/dist/tests/SM-8.10/mt-enabled-user-flows.spec.js

@@ -449,19 +449,28 @@ if (process.env.IS_MT === 'true') {
                 await modelerCreatePage.runProcessInstance('', tenantName2);
             });
             await SM_8_10_1.test.step('View Process Instances in Operate, and Assert Process Complete in Operate with Correct Tenant', async () => {
-                await navigationPage.goToOperate();
-                await operateHomePage.clickProcessesTab();
-                await operateProcessesPage.clickProcessInstanceLink(processName1, 'completed');
-                await operateProcessInstancePage.completedIconAssertion();
+                // Re-entered through `goToOperate` on every attempt: this flow reaches
+                // Operate ~10 minutes in, and when the session's token refresh fails
+                // the instance route answers with an error document instead of the app
+                // shell. The reload loop inside `completedIconAssertion` only
+                // re-requests that same document, so the session has to be
+                // re-established before the icon can be asserted at all.
+                const openCompletedInstance = async (processName) => {
+                    await (0, test_1.expect)(async () => {
+                        await navigationPage.goToOperate();
+                        await operateHomePage.clickProcessesTab();
+                        await operateProcessesPage.clickProcessInstanceLink(processName, 'completed');
+                        await operateProcessInstancePage.completedIconAssertion();
+                    }).toPass({ timeout: 600000, intervals: [5000] });
+                };
+                await openCompletedInstance(processName1);
                 await (0, expectTextWithRetry_1.expectTextWithRetry)(page, '"Awesome!"', {
                     postAction: async () => {
                         await page.reload();
                     },
                 });
                 await operateProcessInstancePage.tenantNameAssertion(tenantName1);
-                await operateHomePage.clickProcessesTab();
-                await operateProcessesPage.clickProcessInstanceLink(processName2, 'completed');
-                await operateProcessInstancePage.completedIconAssertion();
+                await openCompletedInstance(processName2);
                 await (0, expectTextWithRetry_1.expectTextWithRetry)(page, '"Awesome!"', {
                     postAction: async () => {
                         await page.reload();

package/package.json

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