@camunda/e2e-test-suite 0.0.1278 → 0.0.1279

package/dist/pages/8.7/AppsPage.js

@@ -83,8 +83,14 @@ class AppsPage {
     // (see clickHubClusterApp and isHubClusterAppPresent below) gets at most
     // one match, scoped to this cluster.
     async expandHubClusterGroup(clusterName) {
+        // Anchored at the start: an unanchored substring match on "Test Cluster"
+        // also matches "UI Test Cluster" (created by console-user-flows in the
+        // same org), and .first() then opened that cluster's popover instead --
+        // a nightly trace shows the "UI Test Cluster" group open while the
+        // "Tasklist"/"Operate" lookup for "Test Cluster" timed out.
+        const clusterNamePattern = new RegExp(`^${escapeRegExp(clusterName)}\\b`);
         const toggle = this.hubSidebar
-            .getByRole('button', { name: clusterName })
+            .getByRole('button', { name: clusterNamePattern })
             .first();
         // The diagram editor (and other full-screen views reached from it, e.g.
         // right after runProcessInstance/deployDiagram) has no "Main navigation"
@@ -165,14 +171,16 @@ class AppsPage {
         // of its own, in which case hasNotText would fail to exclude the toggle
         // we just expanded and this loop would immediately collapse it again.
         // Match by accessible name (same algorithm getByRole's own name option
-        // uses) instead, excluding any name CONTAINING clusterName -- not just
-        // one exactly equal to it, since (per the toggle locator above) the
-        // current cluster's own toggle name is not exactly clusterName either,
-        // and an exact-equality exclusion would wrongly treat it as "other" and
-        // immediately re-collapse the group this method just expanded.
+        // uses) instead, excluding any name STARTING WITH clusterName (the same
+        // anchoring as the toggle locator, so "UI Test Cluster" still counts as
+        // another group) -- not just one exactly equal to it, since (per the
+        // toggle locator above) the current cluster's own toggle name is not
+        // exactly clusterName either, and an exact-equality exclusion would
+        // wrongly treat it as "other" and immediately re-collapse the group this
+        // method just expanded.
         const otherExpandedToggles = this.hubSidebar.getByRole('button', {
             expanded: true,
-            name: new RegExp(`^(?!.*${escapeRegExp(clusterName)}).+$`),
+            name: new RegExp(`^(?!${escapeRegExp(clusterName)}\\b).+$`),
         });
         for (let i = 0; i < 5 && (await otherExpandedToggles.count()) > 0; i++) {
             // Let a failed collapse throw rather than swallowing it: the app link

package/dist/pages/8.7/ClusterDetailsPage.d.ts

@@ -62,6 +62,7 @@ declare class ClusterDetailsPage {
     clickCreateButton(): Promise<void>;
     clickCloseModalButton(): Promise<void>;
     clickSettingsTab(): Promise<void>;
+    private isRBAToggleInState;
     enableRBA(): Promise<void>;
     disableRBA(): Promise<void>;
     searchAndClickClientCredentialsLink(name: string): Promise<ClientCredentialsDetailsPage>;

package/dist/pages/8.7/ClusterDetailsPage.js

@@ -234,9 +234,21 @@ class ClusterDetailsPage {
     async clickSettingsTab() {
         await this.settingsTab.click({ timeout: 60000 });
     }
+    // The switch reports its state through aria-checked. Clicking it
+    // unconditionally opened the opposite confirmation dialog whenever the
+    // cluster was already in the requested state (e.g. "Ensure RBA is
+    // Disabled" on a cluster that never had it on), so the Enable/Disable
+    // dialog wait below timed out. Returns false when the state is unknown.
+    async isRBAToggleInState(enabled) {
+        const checked = await this.rbaToggle.getAttribute('aria-checked');
+        return checked !== null && (checked === 'true') === enabled;
+    }
     async enableRBA() {
         await (0, test_1.expect)(this.rbaHeading).toBeVisible({ timeout: 60000 });
         await (0, test_1.expect)(this.rbaToggle).toBeVisible({ timeout: 30000 });
+        if (await this.isRBAToggleInState(true)) {
+            return;
+        }
         await this.rbaToggle.click();
         await (0, test_1.expect)(this.enableRBADialog).toBeVisible({ timeout: 60000 });
         await this.rbaEnableButton.click();
@@ -247,6 +259,9 @@ class ClusterDetailsPage {
     async disableRBA() {
         await (0, test_1.expect)(this.rbaHeading).toBeVisible({ timeout: 60000 });
         await (0, test_1.expect)(this.rbaToggle).toBeVisible({ timeout: 30000 });
+        if (await this.isRBAToggleInState(false)) {
+            return;
+        }
         await this.rbaToggle.click();
         await (0, test_1.expect)(this.disableRBADialog).toBeVisible({ timeout: 60000 });
         await this.rbaDisableButton.click();

package/dist/pages/8.7/ClusterSecretsPage.js

@@ -31,9 +31,16 @@ class ClusterSecretsPage {
         this.connectorSecretsTable = page.getByRole('table', {
             name: 'Connector secrets',
         });
+        // Legacy Carbon table: an "Options" cell holding the kebab. Hub's
+        // design-system data table instead labels each row's kebab
+        // "Row actions" (data-slot="data-table-actions-trigger"), with no
+        // "Options" cell -- so the old locator found nothing, the pre-clean
+        // returned early, and re-creating an existing secret failed with
+        // "Secret names must be unique".
         this.optionsButton = page
             .getByRole('cell', { name: 'Options' })
-            .getByRole('button');
+            .getByRole('button')
+            .or(page.getByRole('button', { name: 'Row actions', exact: true }));
         this.deleteConnectorSecretButton = page.getByRole('menuitem', {
             name: 'Delete',
             exact: true,
@@ -124,7 +131,7 @@ class ClusterSecretsPage {
             // that nothing *deletable* is left behind.
             const deletableSecretRows = this.page
                 .getByRole('row')
-                .filter({ has: this.page.getByRole('cell', { name: 'Options' }) })
+                .filter({ has: this.optionsButton })
                 .filter({ hasNotText: 'CAMUNDA_PROVIDED_' });
             await (0, test_1.expect)(deletableSecretRows).toHaveCount(0, { timeout: 30000 });
         }

package/dist/pages/8.7/ConnectorMarketplacePage.d.ts

@@ -7,6 +7,7 @@ declare class ConnectorMarketplacePage {
     readonly replaceResourceButton: Locator;
     readonly addToProjectButton: Locator;
     readonly cancelButton: Locator;
+    readonly tryAgainButton: Locator;
     readonly closeButton: Locator;
     readonly browseAllMarketplaceDetailsButton: Locator;
     constructor(page: Page);

package/dist/pages/8.7/ConnectorMarketplacePage.js

@@ -10,6 +10,7 @@ class ConnectorMarketplacePage {
     replaceResourceButton;
     addToProjectButton;
     cancelButton;
+    tryAgainButton;
     closeButton;
     browseAllMarketplaceDetailsButton;
     constructor(page) {
@@ -38,6 +39,7 @@ class ConnectorMarketplacePage {
             name: 'Add to project',
         });
         this.cancelButton = page.getByRole('button', { name: 'Cancel' });
+        this.tryAgainButton = page.getByRole('button', { name: 'Try again' });
         this.closeButton = page.getByRole('button', { name: 'Close' });
     }
     async clickSearchForConnectorTextbox() {
@@ -72,11 +74,33 @@ class ConnectorMarketplacePage {
         // genuine timeout here throw rather than silently falling through to the
         // immediate isVisible() probes below, which would just repeat the same
         // race this wait exists to close.
-        await this.addToProjectButton
+        //
+        // The Browse all review step first "checks this project" (camunda-hub
+        // marketplace-connector-detail-state.ts), which waits on the project's
+        // element-template metadata request -- nightly and live traces show that
+        // request still pending 30s in. When the check fails the footer offers
+        // "Try again" (RETRY_INSPECTION) instead of "Add to project", so give the
+        // check longer and re-run it on failure rather than timing out.
+        const outcome = this.addToProjectButton
             .or(this.replaceResourceButton)
             .or(this.cancelButton)
-            .first()
-            .waitFor({ state: 'visible', timeout: 30000 });
+            .first();
+        for (let attempt = 0; attempt < 3; attempt++) {
+            await outcome
+                .or(this.tryAgainButton)
+                .first()
+                .waitFor({ state: 'visible', timeout: 120000 });
+            if (await outcome.isVisible()) {
+                break;
+            }
+            // The retrying check keeps "Try again" on screen (as a loading button),
+            // so wait for its result here instead of re-polling the union above.
+            await this.tryAgainButton.click({ timeout: 10000 });
+            await outcome
+                .waitFor({ state: 'visible', timeout: 120000 })
+                .catch(() => { });
+        }
+        await outcome.waitFor({ state: 'visible', timeout: 10000 });
         let imported = false;
         // A fresh download shows "Add to project"; an already-present resource
         // shows "Replace resource". Pick whichever the ImportModal rendered —

package/dist/pages/8.7/LoginPage.js

@@ -77,8 +77,20 @@ class LoginPage {
         // before) must clear it too, or getUserToken() can return the previous
         // user's cached token.
         (0, consoleApiHelpers_1.resetHubBearerToken)(this.page);
-        // Navigate to app root to start the auth flow.
-        await this.page.goto('/');
+        // Navigate to app root to start the auth flow. Callers usually arrive
+        // here straight from a "Log out" click, whose own redirect chain (Hub's
+        // /login?returnUrl=/logout -> Auth0 logout) can still be in flight and
+        // interrupt this goto. Let that navigation land, then start over once.
+        try {
+            await this.page.goto('/');
+        }
+        catch (error) {
+            if (!String(error).includes('is interrupted by another navigation')) {
+                throw error;
+            }
+            await this.page.waitForLoadState('domcontentloaded', { timeout: 60000 });
+            await this.page.goto('/');
+        }
         await this.page
             .waitForURL((url) => !url.hostname.includes('console'), { timeout: 60000 })
             .catch(() => { });
@@ -86,8 +98,18 @@ class LoginPage {
         // logout/session-reset, keeping the page on Console instead of redirecting
         // to the Auth0 login form. Detect Console (settings button visible) and
         // force logout until the Auth0 email input appears.
-        const settingsBtn = this.page.getByLabel('Open Settings');
-        const logoutBtn = this.page.getByRole('button', { name: 'Log out' });
+        // Hub's design-system shell has no "Open Settings" button: its user menu
+        // is a "User menu, <full name>" button whose "Log out" is a menuitem (see
+        // SettingsPage). Without these the loop never noticed it had been
+        // silently re-authenticated on Hub and waited out the username input.
+        const settingsBtn = this.page
+            .getByLabel('Open Settings')
+            .or(this.page.getByRole('button', { name: /^User menu,/ }))
+            .first();
+        const logoutBtn = this.page
+            .getByRole('button', { name: 'Log out' })
+            .or(this.page.getByRole('menuitem', { name: 'Log out' }))
+            .first();
         for (let i = 0; i < 3; i++) {
             const onAuth0 = await this.usernameInput
                 .waitFor({ state: 'visible', timeout: 15000 })

package/dist/pages/8.7/UtilitiesPage.js

@@ -8,6 +8,7 @@ const sleep_1 = require("../../utils/sleep");
 const fileUpload_1 = require("../../utils/fileUpload");
 const mailSlurpClient_1 = require("../../utils/mailSlurpClient");
 const consoleApiHelpers_1 = require("../../utils/consoleApiHelpers");
+const sessionRecovery_1 = require("../../utils/sessionRecovery");
 // Tracks which worker-derived `timeout` values have already paid the
 // pre-flight stagger below, so a worker only pays it once per process
 // lifetime rather than on every call -- see the comment at the call sites.
@@ -601,7 +602,11 @@ async function modelAndRunConnectorsDocHandlingDiagram(modelerCreatePage, modele
     });
     await connectorMarketplacePage.downloadConnectorToProject();
     await (0, sleep_1.sleep)(5000);
-    await page.reload();
+    // A live run showed this reload bounce through Hub's /login?returnUrl=...
+    // and never reach "load" -- the stuck-session case recoverStuckModelerLogin
+    // exists for. Wait for the document only, then recover if it is stuck.
+    await page.reload({ waitUntil: 'domcontentloaded' });
+    await (0, sessionRecovery_1.recoverStuckModelerLogin)(page);
     await (0, sleep_1.sleep)(5000);
     await modelerCreatePage.fillStartEventVariablesForDocHandling(zeebeUrl, zeebeClientId, zeebeClientSecret);
     await modelerCreatePage.runProcessInstance(clusterName);

package/dist/tests/8.7/connectors-user-flows.spec.js

@@ -182,7 +182,9 @@ _8_7_1.test.describe('Connectors User Flow Tests', () => {
                 timeout: 20000,
             });
             await appsPage.clickCamundaApps();
-            await (0, test_1.expect)(appsPage.modelerLink).toBeVisible({ timeout: 20000 });
+            // No separate modelerLink check: Hub has no app-switcher "Modeler" link
+            // (clickModeler() goes through its home link instead), and on the
+            // legacy shell clickModeler() asserts modelerLink itself.
             await appsPage.clickModeler();
             await (0, test_1.expect)(modelerHomePage.modelerPageBanner).toBeVisible({
                 timeout: 120000,

package/package.json

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