@camunda/e2e-test-suite 0.0.1234 → 0.0.1236

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

@@ -12,6 +12,7 @@ declare class OCIdentityHomePage {
     readonly licenseKeyTagCommercial: Locator;
     readonly noPermissionsMessage: Locator;
     constructor(page: Page);
+    private openTab;
     clickAuthorizationsTab(): Promise<void>;
     clickRolesTab(): Promise<void>;
     clickGroupsTab(): Promise<void>;

package/dist/pages/8.10/OCIdentityHomePage.js

@@ -73,21 +73,35 @@ class OCIdentityHomePage {
         this.licenseKeyTagCommercial = page.getByText('Commercial license');
         this.noPermissionsMessage = page.getByText('You don’t have access to this component');
     }
+    // Every caller arrives here from somewhere else in Admin -- a role detail
+    // page that was just reloaded, a freshly dismissed modal -- and the default
+    // retry gives the sidebar three 10s looks with no recovery between them.
+    // That is the whole budget a hand-off has if the SPA is still re-rendering
+    // its shell, and when it runs out the run reports "Failed to assert
+    // visibility locator ... 'Authorizations' ... after 3 attempts" for a nav
+    // that is permanently part of the page. Look longer, and reload between
+    // attempts so a shell that never finished painting gets another chance
+    // instead of being polled in the same broken state three times.
+    async openTab(tab) {
+        await (0, expectLocatorWithRetry_1.expectLocatorWithRetry)(this.page, tab, {
+            visibilityTimeout: 20000,
+            postAction: async () => {
+                await this.page.reload();
+            },
+        });
+        await tab.click({ timeout: 30000 });
+    }
     async clickAuthorizationsTab() {
-        await (0, expectLocatorWithRetry_1.expectLocatorWithRetry)(this.page, this.authorizationsTab);
-        await this.authorizationsTab.click({ timeout: 30000 });
+        await this.openTab(this.authorizationsTab);
     }
     async clickRolesTab() {
-        await (0, expectLocatorWithRetry_1.expectLocatorWithRetry)(this.page, this.rolesTab);
-        await this.rolesTab.click({ timeout: 30000 });
+        await this.openTab(this.rolesTab);
     }
     async clickGroupsTab() {
-        await (0, expectLocatorWithRetry_1.expectLocatorWithRetry)(this.page, this.groupsTab);
-        await this.groupsTab.click({ timeout: 30000 });
+        await this.openTab(this.groupsTab);
     }
     async clickClusterVariablesTab() {
-        await (0, expectLocatorWithRetry_1.expectLocatorWithRetry)(this.page, this.clusterVariablesTab);
-        await this.clusterVariablesTab.click({ timeout: 30000 });
+        await this.openTab(this.clusterVariablesTab);
     }
 }
 exports.OCIdentityHomePage = OCIdentityHomePage;

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

@@ -25,7 +25,9 @@ declare class OCIdentityRolesPage {
     readonly assignUserModal: Locator;
     readonly assignUserModalInputField: Locator;
     readonly assignUserModalAssignButton: Locator;
+    readonly roleSearchBox: Locator;
     constructor(page: Page);
+    private searchForRole;
     clickMappingRulesTab(): Promise<void>;
     clickRole(roleID: string): Promise<void>;
     assignMappingRuleToRole(roleId: string, mappingId: string): Promise<void>;

package/dist/pages/8.10/OCIdentityRolesPage.js

@@ -30,6 +30,7 @@ class OCIdentityRolesPage {
     assignUserModal;
     assignUserModalInputField;
     assignUserModalAssignButton;
+    roleSearchBox;
     constructor(page) {
         this.page = page;
         this.rolesList = page.getByRole('table');
@@ -105,6 +106,25 @@ class OCIdentityRolesPage {
         this.assignUserModalAssignButton = this.assignUserModal.getByRole('button', {
             name: /assign user/i,
         });
+        // Both role-list shapes render this filter with the same placeholder:
+        // camunda/camunda identity/client/src/pages/roles/List.tsx (Carbon
+        // TableToolbarSearch) and ListV2.tsx (design-system SearchInput) each pass
+        // `searchPlaceholder={t("searchByRoleId")}`. Only V2 mirrors it into an
+        // aria-label, so match on the placeholder to cover both.
+        this.roleSearchBox = page.getByPlaceholder('Search by role ID');
+    }
+    // The roles list is server-side paginated at 10 rows per page, and these
+    // long-lived RBA clusters accumulate a role per test run -- the org that
+    // failed was already at "Page 6 of 6". A freshly created role sorts to the
+    // end, so `findLocatorWithPagination` has to walk every page one request at
+    // a time and exhausts its 30s budget before it arrives, reporting a role
+    // that is plainly on screen as missing. Both list shapes pass
+    // `searchKey="roleId"` with the default `eq` operator, so filtering queries
+    // the backend for that exact role and puts it on page 1.
+    async searchForRole(roleId) {
+        await (0, test_1.expect)(this.roleSearchBox).toBeVisible({ timeout: 60000 });
+        await this.roleSearchBox.fill(roleId, { timeout: 30000 });
+        await (0, test_1.expect)(this.roleSearchBox).toHaveValue(roleId, { timeout: 30000 });
     }
     async clickMappingRulesTab() {
         await (0, test_1.expect)(this.selectResourceTypeTab('Mapping Rules')).toBeVisible({
@@ -113,6 +133,7 @@ class OCIdentityRolesPage {
         await this.selectResourceTypeTab('Mapping Rules').click();
     }
     async clickRole(roleID) {
+        await this.searchForRole(roleID);
         await (0, test_1.expect)(this.roleCell(roleID).first()).toBeVisible({ timeout: 60000 });
         await this.roleCell(roleID).first().click();
     }
@@ -149,11 +170,21 @@ class OCIdentityRolesPage {
         await this.createNameField.fill(role.name);
         await this.createRoleModalCreateButton.click();
         await (0, test_1.expect)(this.createRoleModal).not.toBeVisible({ timeout: 30000 });
-        await (0, expectLocatorWithPagination_1.expectLocatorWithPagination)(this.page, this.rolesList
+        await this.searchForRole(role.id);
+        await (0, expectLocatorWithRetry_1.expectLocatorWithRetry)(this.page, this.rolesList
             .getByRole('cell', {
             name: role.name,
         })
-            .first());
+            .first(), {
+            visibilityTimeout: 20000,
+            postAction: async () => {
+                // The filter is component state, not a URL param, so a reload drops
+                // it -- re-apply it or the retry falls back to the page walk this
+                // whole approach exists to avoid.
+                await this.page.reload();
+                await this.searchForRole(role.id);
+            },
+        });
     }
     async assignMappingToRole(role, mappingRuleId) {
         try {
@@ -169,8 +200,12 @@ class OCIdentityRolesPage {
         await (0, test_1.expect)(this.rowValue(mappingRuleId)).toContainText(`${mappingRuleId}`);
     }
     async assignUserToRole(role, email) {
+        // Filter before probing for the role: unfiltered, this only ever sees the
+        // ten rows of page 1, so an existing role further down the list reads as
+        // missing and we try to create it a second time.
+        await this.searchForRole(role);
         try {
-            await (0, test_1.expect)(this.page.getByText(role).first()).toBeVisible({
+            await (0, test_1.expect)(this.roleCell(role).first()).toBeVisible({
                 timeout: 10000,
             });
         }
@@ -186,6 +221,7 @@ class OCIdentityRolesPage {
         });
     }
     async deleteRole(name) {
+        await this.searchForRole(name);
         const locator = this.deleteRoleButton(name);
         await (0, expectLocatorWithPagination_1.expectLocatorWithPagination)(this.page, locator);
         await locator.click();

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

@@ -17,7 +17,7 @@ declare class TaskPanelPage {
     private openFilterControls;
     private collapseFilterPanel;
     private static appliedFilter;
-    taskCount(name: string): Promise<number>;
+    taskCount(name: string, expectedCount?: number): Promise<number>;
     clickTasksTab(): Promise<void>;
     scrollToLastTask(name: string): Promise<void>;
     scrollToFirstTask(name: string): Promise<void>;

package/dist/pages/8.10/TaskPanelPage.js

@@ -190,9 +190,37 @@ class TaskPanelPage {
         const param = filterParam[option] ?? null;
         return (url) => url.searchParams.get('filter') === param;
     }
-    async taskCount(name) {
-        await (0, sleep_1.sleep)(20000);
-        return this.availableTasks.getByText(name, { exact: true }).count();
+    // `expectedCount` turns the single fixed-delay sample into a poll. Tasklist
+    // only lists a task once the exporter has indexed the instance, and on these
+    // long-lived SaaS RBA clusters that routinely lags past the 20s wait -- the
+    // count then comes back one short and the caller's `toBe(before + 1)` fails
+    // on a task that does arrive moments later. Callers that have no expectation
+    // yet (the baseline read taken before starting a process) still get the
+    // fixed wait. The count returned is always the real one, so a task that
+    // genuinely never appears still fails the caller's assertion.
+    async taskCount(name, expectedCount) {
+        const tasks = this.availableTasks.getByText(name, { exact: true });
+        if (expectedCount === undefined) {
+            await (0, sleep_1.sleep)(20000);
+            return tasks.count();
+        }
+        // Re-reading the same rendered list only finds the new task if the panel
+        // refetches while we wait, and the sample that failed sat on 1 of an
+        // expected 2 for the whole budget -- the instance was started, the
+        // exporter published its task afterwards, and the view never asked again.
+        // Reload between samples so each one is an actual query rather than a
+        // re-count of the DOM we already had. The count returned is still the
+        // real one, so a task that genuinely never arrives still fails the
+        // caller's assertion.
+        const deadline = Date.now() + 180000;
+        let count = await tasks.count();
+        while (count !== expectedCount && Date.now() < deadline) {
+            await (0, sleep_1.sleep)(5000);
+            await this.page.reload();
+            await (0, test_1.expect)(this.availableTasks).toBeVisible({ timeout: 60000 });
+            count = await tasks.count();
+        }
+        return count;
     }
     async clickTasksTab() {
         await (0, test_1.expect)(this.tasksTab).toBeVisible({ timeout: 60000 });

package/dist/pages/c8Run-8.10/OperateHomePage.d.ts

@@ -6,6 +6,8 @@ declare class OperateHomePage {
     readonly informationDialog: Locator;
     readonly editVariableButton: Locator;
     readonly variableValueInput: Locator;
+    readonly variableValueEditor: Locator;
+    readonly variableValueContent: Locator;
     readonly saveVariableButton: Locator;
     readonly editVariableSpinner: Locator;
     constructor(page: Page);

package/dist/pages/c8Run-8.10/OperateHomePage.js

@@ -9,6 +9,8 @@ class OperateHomePage {
     informationDialog;
     editVariableButton;
     variableValueInput;
+    variableValueEditor;
+    variableValueContent;
     saveVariableButton;
     editVariableSpinner;
     constructor(page) {
@@ -28,6 +30,21 @@ class OperateHomePage {
         });
         this.editVariableButton = page.getByTestId('edit-variable-button');
         this.variableValueInput = page.getByTestId('edit-variable-value');
+        // "The row is in write mode." InlineJsonEditor renders a CodeMirror
+        // instance carrying this test id only on its editable branch; the
+        // read-only branch renders ReadOnlyEditor instead, with no CodeMirror
+        // at all. The product's own suite resolves the editor the same way --
+        // see camunda/camunda
+        // qa/c8-orchestration-cluster-e2e-test-suite/pages/OperateHomePage.ts.
+        this.variableValueEditor = page.getByTestId('code-mirror-editor');
+        // The editable node *inside* that container. CodeMirror puts its
+        // contenteditable on `.cm-content`, so that -- not the container -- is
+        // what takes focus and what holds the text. Asserting focus on the
+        // container reports `inactive` even when the caret is correctly placed,
+        // because clicking the container focuses its contenteditable child.
+        this.variableValueContent = this.variableValueEditor
+            .locator('.cm-content')
+            .first();
         // Carbon icon-only buttons expose their accessible name via aria-labelledby
         // (tooltip text "Save") which overrides aria-label for computed accessible name.
         // Use a CSS attribute selector to match the raw aria-label attribute directly.
@@ -56,8 +73,34 @@ class OperateHomePage {
         throw new Error(`Failed to click the processes tab after ${maxRetries} attempts.`);
     }
     async clickEditVariableButton(variableName) {
-        const editVariableButton = 'variable-' + variableName;
-        await this.page.getByTestId(editVariableButton).getByLabel('Edit').click();
+        const editVariableButton = this.page
+            .getByTestId('variable-' + variableName)
+            .getByLabel('Edit');
+        await (0, test_1.expect)(editVariableButton).toBeVisible({ timeout: 30000 });
+        // The caller reaches this straight out of activeIconAssertion, which
+        // reloads the page while it polls for the status icon, so the variables
+        // panel is usually still mounting when the click lands. A click during
+        // that render resolves without raising and without effect: the row stays
+        // read-only, InlineJsonEditor keeps rendering ReadOnlyEditor, and the
+        // editor the next step needs is never created. Retry until write mode is
+        // actually active rather than asserting on the editor afterwards and
+        // reporting the dropped click as a missing element.
+        //
+        // Only click while the row is still read-only. Entering write mode
+        // replaces the Edit control, so once the click has taken, clicking again
+        // just waits out its timeout against an element that is gone or still
+        // being replaced -- which is what the retry did on the slower macOS and
+        // Windows runners, where the editor needs longer to mount than one
+        // iteration allowed: it re-clicked, exhausted the budget, and raised the
+        // click timeout instead of the state the row was actually in. Probing
+        // first keeps the retry for a genuinely dropped click and turns the slow
+        // case into what it should be, a wait.
+        await (0, test_1.expect)(async () => {
+            if (await editVariableButton.isVisible()) {
+                await editVariableButton.click({ timeout: 10000 });
+            }
+            await (0, test_1.expect)(this.variableValueEditor).toBeVisible({ timeout: 15000 });
+        }).toPass({ timeout: 90000 });
     }
     async clickVariableValueInput() {
         await this.variableValueInput.click();
@@ -69,15 +112,18 @@ class OperateHomePage {
         // Operate migrated this editor from Monaco to CodeMirror
         // (@uiw/react-codemirror; see camunda/camunda#62782) -- the editable
         // region is now a div.cm-content, not .monaco-editor, and it no longer
-        // exposes a Monaco-style .cursor element. clickEditVariableButton still
-        // triggers autoFocus, so wait for the content element to actually have
-        // focus (:focus) before sending events, same intent as the old cursor
-        // check. Scoped to the variable row to avoid matching other editors.
-        const variableRow = this.page.getByTestId('variable-testVariable');
-        await variableRow
-            .locator('.cm-content:focus')
-            .first()
-            .waitFor({ state: 'visible', timeout: 30000 });
+        // exposes a Monaco-style .cursor element.
+        //
+        // clickEditVariableButton has already established that the editor exists
+        // -- it retries the Edit click until it does -- so this only has to put
+        // the caret in it. Do not wait on focus arriving by itself: autoFocus
+        // fires when the editor mounts, which is before this runs, so a wait for
+        // `:focus` can only ever observe focus that has already been placed
+        // elsewhere. Click to focus, then confirm. Both act on the
+        // contenteditable rather than its container, which cannot hold focus.
+        await (0, test_1.expect)(this.variableValueContent).toBeVisible({ timeout: 30000 });
+        await this.variableValueContent.click();
+        await (0, test_1.expect)(this.variableValueContent).toBeFocused({ timeout: 10000 });
         // Insert the COMPLETE JSON value, both quotes included. Monaco used to
         // auto-close a lone opening quote, so the old recipe inserted only
         // `"value`; CodeMirror's closeBrackets extension ignores insertions
@@ -97,7 +143,9 @@ class OperateHomePage {
         await this.page.keyboard.press('Backspace');
         const jsonValue = JSON.stringify(newValue);
         await this.page.keyboard.insertText(jsonValue);
-        await (0, test_1.expect)(variableRow.locator('.cm-content').first()).toContainText(jsonValue, { timeout: 10000 });
+        await (0, test_1.expect)(this.variableValueContent).toContainText(jsonValue, {
+            timeout: 10000,
+        });
     }
     async clickSaveVariableButton() {
         await this.saveVariableButton.click({ timeout: 30000 });

package/dist/tests/8.10/rba-enabled-v2-user-flows.spec.js

@@ -39,10 +39,7 @@ _8_10_1.test.describe.parallel('RBA Enabled User Flows Test @tasklistV2', () =>
         const process1 = processName + '1';
         const process2 = processName + '2';
         const role = `rbaNoPermission${randomString}`;
-        const testUser = {
-            username: process.env.C8_USERNAME_TEST_4,
-            password: process.env.C8_PASSWORD_TEST_4,
-        };
+        const testUser = (0, users_1.requireTestUser)('C8_USERNAME_TEST_4');
         await _8_10_1.test.step('Navigate to OC Identity and Assign User To A Role', async () => {
             await appsPage.clickCamundaApps();
             await appsPage.clickAdmin(clusterName);
@@ -149,10 +146,7 @@ _8_10_1.test.describe.parallel('RBA Enabled User Flows Test @tasklistV2', () =>
         const process2 = processName + '2';
         const role1 = `rbaOnePerm${randomString}1`;
         const role2 = `rbaOnePerm${randomString}2`;
-        const testUser = {
-            username: process.env.C8_USERNAME_TEST_6,
-            password: process.env.C8_PASSWORD_TEST_6,
-        };
+        const testUser = (0, users_1.requireTestUser)('C8_USERNAME_TEST_6');
         await _8_10_1.test.step('Navigate to OC Identity and Assign User To Two Roles', async () => {
             await appsPage.clickCamundaApps();
             await appsPage.clickAdmin(clusterName);
@@ -365,7 +359,7 @@ _8_10_1.test.describe.parallel('RBA Enabled User Flows Test @tasklistV2', () =>
             const firstProcessTaskCount = await taskPanelPage.taskCount(process1);
             await taskPanelPage.clickProcessesTab();
             await taskProcessesPage.startProcess(process1);
-            const updatedFirstProcessTaskCount = await taskPanelPage.taskCount(process1);
+            const updatedFirstProcessTaskCount = await taskPanelPage.taskCount(process1, firstProcessTaskCount + 1);
             (0, test_1.expect)(updatedFirstProcessTaskCount).toBe(firstProcessTaskCount + 1);
         });
         await _8_10_1.test.step('Assert Second Process Can Be Started From Tasklist', async () => {
@@ -373,7 +367,7 @@ _8_10_1.test.describe.parallel('RBA Enabled User Flows Test @tasklistV2', () =>
             const secondProcessTaskCount = await taskPanelPage.taskCount(process2);
             await taskPanelPage.clickProcessesTab();
             await taskProcessesPage.startProcess(process2);
-            const updatedSecondProcessTaskCount = await taskPanelPage.taskCount(process2);
+            const updatedSecondProcessTaskCount = await taskPanelPage.taskCount(process2, secondProcessTaskCount + 1);
             (0, test_1.expect)(updatedSecondProcessTaskCount).toBe(secondProcessTaskCount + 1);
         });
         await _8_10_1.test.step('Navigate to Operate and Assert Both Processes Are Accessible', async () => {
@@ -395,10 +389,7 @@ _8_10_1.test.describe.parallel('RBA Enabled User Flows Test @tasklistV2', () =>
         const role = `rbaSelectedProcessesPermission${randomString}`;
         const process1 = processName + '1';
         const process2 = processName + '2';
-        const testUser = {
-            username: process.env.C8_USERNAME_TEST_5,
-            password: process.env.C8_PASSWORD_TEST_5,
-        };
+        const testUser = (0, users_1.requireTestUser)('C8_USERNAME_TEST_5');
         await _8_10_1.test.step('Navigate to OC Identity and Assign User To A Role', async () => {
             await appsPage.clickCamundaApps();
             await appsPage.clickAdmin(clusterName);
@@ -470,7 +461,7 @@ _8_10_1.test.describe.parallel('RBA Enabled User Flows Test @tasklistV2', () =>
             const firstProcessTaskCount = await taskPanelPage.taskCount(process1);
             await taskPanelPage.clickProcessesTab();
             await taskProcessesPage.startProcess(process1);
-            const updatedFirstProcessTaskCount = await taskPanelPage.taskCount(process1);
+            const updatedFirstProcessTaskCount = await taskPanelPage.taskCount(process1, firstProcessTaskCount + 1);
             (0, test_1.expect)(updatedFirstProcessTaskCount).toBe(firstProcessTaskCount + 1);
         });
         await _8_10_1.test.step('Assert Second Process Can Be Started From Tasklist', async () => {
@@ -478,7 +469,7 @@ _8_10_1.test.describe.parallel('RBA Enabled User Flows Test @tasklistV2', () =>
             const secondProcessTaskCount = await taskPanelPage.taskCount(process2);
             await taskPanelPage.clickProcessesTab();
             await taskProcessesPage.startProcess(process2);
-            const updatedSecondProcessTaskCount = await taskPanelPage.taskCount(process2);
+            const updatedSecondProcessTaskCount = await taskPanelPage.taskCount(process2, secondProcessTaskCount + 1);
             (0, test_1.expect)(updatedSecondProcessTaskCount).toBe(secondProcessTaskCount + 1);
         });
         await _8_10_1.test.step('Navigate to Operate and Assert Both Processes Are Accessible', async () => {

package/dist/utils/users.d.ts

@@ -100,3 +100,7 @@ export declare function getTestUsers(): ({
     username: string;
     password: string;
 })[];
+export declare function requireTestUser(usernameVar: string): {
+    username: string;
+    password: string;
+};

package/dist/utils/users.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getTestUsers = exports.getTestUser = exports.smokeTestUsers = exports.testUsers = void 0;
+exports.requireTestUser = exports.getTestUsers = exports.getTestUser = exports.smokeTestUsers = exports.testUsers = void 0;
 exports.testUsers = {
     mainUser: {
         username: process.env.C8_USERNAME,
@@ -115,3 +115,37 @@ function getTestUsers() {
     return [exports.testUsers.mainUser];
 }
 exports.getTestUsers = getTestUsers;
+// Specs that exercise a *restricted* user reach for a numbered credential
+// slot directly rather than through getTestUser. The SaaS organisation a run
+// gets is sized by grepping the spec for the slots it names
+// (scripts/flakiness-saas-create-org.sh), so when a slot is missed the
+// variable is simply absent -- and `process.env.C8_USERNAME_TEST_4!` then
+// hands `undefined` to whatever consumes it next. Run 35494241466 spent 150
+// of its 152 repeats that way: roughly twenty minutes each to reach the
+// assign-user modal and die on "locator.fill: value: expected string, got
+// undefined", an error that names neither the slot nor the org that lacks it.
+//
+// Resolve the pair here instead, so a slot the org was never given is
+// reported in one second and by name. Callers pass the username variable as
+// a literal so the org-sizing grep above still sees it in the spec.
+//
+// There is deliberately no fallback to mainUser of the kind getTestUser has:
+// mainUser is the organisation owner, and an RBA flow that runs its
+// "restricted" user as the owner would assert nothing.
+function requireTestUser(usernameVar) {
+    const passwordVar = usernameVar.replace('USERNAME', 'PASSWORD');
+    const username = process.env[usernameVar];
+    const password = process.env[passwordVar];
+    const missing = [
+        username ? null : usernameVar,
+        password ? null : passwordVar,
+    ].filter(Boolean);
+    if (missing.length > 0) {
+        throw new Error(`Test user credentials are not set: ${missing.join(', ')}. The ` +
+            'organisation for this run was created without that user slot -- ' +
+            'see scripts/flakiness-saas-create-org.sh, which sizes the org from ' +
+            'the slots the spec names.');
+    }
+    return { username: username, password: password };
+}
+exports.requireTestUser = requireTestUser;

package/package.json

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