@camunda/e2e-test-suite 0.0.1413 → 0.0.1414

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

@@ -69,6 +69,7 @@ declare class ClusterDetailsPage {
     clickSettingsTab(): Promise<void>;
     private waitForAuthorizationsSection;
     private setAuthorizationsViaSwitch;
+    private submitAuthorizationsConfirm;
     enableAuthorizations(): Promise<void>;
     disableAuthorizations(): Promise<void>;
     searchAndClickClientCredentialsLink(name: string): Promise<ClientCredentialsDetailsPage>;

package/dist/pages/8.10/ClusterDetailsPage.js

@@ -369,13 +369,39 @@ class ClusterDetailsPage {
             .then(() => true)
             .catch(() => false);
         if (confirmOpened) {
-            const confirm = enabled ? this.rbaEnableButton : this.rbaDisableButton;
-            await confirm.click({ timeout: 30000 });
-            await (0, test_1.expect)(this.page.getByText(enabled ? 'Enabling...' : 'Disabling...')).not.toBeVisible({ timeout: 60000 });
+            await this.submitAuthorizationsConfirm(enabled);
         }
         await (0, test_1.expect)(this.authorizationsSwitch).toHaveAttribute('aria-checked', String(enabled), { timeout: 120000 });
         return true;
     }
+    // The confirm dialog sends one PATCH to console-backend. When it fails, the
+    // dialog stays open with the error in an inline alert and does not retry by
+    // itself (`submitAuthorizationsConfirmation` in camunda-cloud-management-apps'
+    // c4 settings tab), so the switch behind the modal never becomes reachable.
+    // On INT console-backend intermittently answers without CORS headers, which
+    // the browser reports as "fetch error". Submit again, as a user would: the
+    // body names the target state, so a PATCH that did land is not applied twice.
+    async submitAuthorizationsConfirm(enabled) {
+        const confirm = enabled ? this.rbaEnableButton : this.rbaDisableButton;
+        const submitError = this.dialog.getByRole('alert');
+        await (0, test_1.expect)(async () => {
+            // A submit that was still in flight when the previous attempt gave up
+            // may have closed the dialog since.
+            if (await this.dialog.isHidden()) {
+                return;
+            }
+            if (await submitError.isVisible()) {
+                console.warn(`Console rejected the authorizations change ` +
+                    `("${await submitError.textContent()}"); submitting it again.`);
+            }
+            await confirm.click({ timeout: 30000 });
+            // The submit settles either way: the dialog closes, or the alert shows.
+            await test_1.expect
+                .poll(async () => (await this.dialog.isHidden()) || (await submitError.isVisible()), { timeout: 60000 })
+                .toBe(true);
+            await (0, test_1.expect)(this.dialog, 'Console did not accept the authorizations change').toBeHidden({ timeout: 1000 });
+        }).toPass({ timeout: 300000, intervals: [5000, 10000, 20000] });
+    }
     async enableAuthorizations() {
         await (0, sleep_1.sleep)(5000);
         await this.waitForAuthorizationsSection();

package/dist/pages/8.10/ClusterPage.js

@@ -275,7 +275,13 @@ class ClusterPage {
                 timeout: 30000,
             });
             if (await deleteMenuItem.isVisible()) {
-                await deleteMenuItem.click({ timeout: 30000 });
+                // Radix renders the menu at `translate(0, -200%)` until it has
+                // positioned it against the trigger (@radix-ui/react-popper). When
+                // that never happens the item is visible but outside the viewport, and
+                // a pointer click retried for the full 30s without landing. Enter on
+                // the item is the same selection: Radix's MenuItem turns it into a
+                // click.
+                await deleteMenuItem.press('Enter', { timeout: 30000 });
             }
             await (0, test_1.expect)(this.dialog).toBeVisible({ timeout: 60000 });
             await (0, test_1.expect)(confirmInput.first()).toBeVisible({ timeout: 60000 });

package/dist/pages/8.10/TaskProcessesPage.js

@@ -19,7 +19,13 @@ class TaskProcessesPage {
         this.popupContinueButton = page.getByRole('button', { name: 'Continue' });
         this.popup = page.getByLabel('Start your process on demand');
         this.processes = page.locator('[data-testid="process-tile"]');
-        this.availableTasks = page.getByTitle('Available tasks').first();
+        // Same dual-shell anchor as TaskPanelPage: the redesigned Tasklist shell
+        // dropped the `title="Available tasks"` wrapper, and the rollout is per
+        // app, so match the `Tasks side panel` region as well as the old title.
+        this.availableTasks = page
+            .getByRole('region', { name: 'Tasks side panel' })
+            .or(page.getByTitle('Available tasks'))
+            .first();
         this.waitingForTasksText = this.page.getByText('Waiting for tasks...');
         this.searchProcessesSearchbox = page.getByRole('searchbox', {
             name: 'Search processes',
@@ -79,7 +85,7 @@ class TaskProcessesPage {
     // `queryUserTasks({processInstanceKey})` every second under a hard
     // `TASKS_TIMEOUT: 15000`; when that elapses the machine goes straight to
     // `Succeeded` and performs no navigation at all, leaving the browser on the
-    // Processes page -- which never renders the "Available tasks" panel. On
+    // Processes page -- which never renders the tasks side panel. On
     // these long-lived SaaS RBA clusters the exporter routinely lags past 15s,
     // so waiting 90s for a redirect the product gave up on after 15 can only
     // time out. Open the Tasks view ourselves when the redirect did not happen;

package/dist/tests/8.10/console-cluster-duplication.spec.js

@@ -40,11 +40,18 @@ _8_10_1.test.describe('Console cluster duplication', () => {
         await (0, UtilitiesPage_1.loginWithRetry)(page, loginPage, testUser, (testInfo.workerIndex + 1) * 1000);
         await homePage.clickClusters();
     });
-    _8_10_1.test.afterEach(async ({ page, homePage, clusterPage }, testInfo) => {
+    _8_10_1.test.afterEach(async ({ page, loginPage, homePage, clusterPage }, testInfo) => {
         await (0, _setup_1.captureScreenshot)(page, testInfo);
         // Postcondition: no cluster left behind, source or duplicate.
         const namesToDelete = [...createdClusterNames].reverse();
         createdClusterNames.length = 0;
+        // A body that failed because the Hub session dropped ends on the identity
+        // provider's login form, where clickClusters() finds no navigation and
+        // every cluster below would leak.
+        if (namesToDelete.length > 0 &&
+            /^(weblogin|login|auth)\./.test(new URL(page.url()).host)) {
+            await (0, UtilitiesPage_1.loginWithRetry)(page, loginPage, testUser, 0, 3);
+        }
         const undeleted = [];
         for (const clusterName of namesToDelete) {
             try {
@@ -55,6 +62,10 @@ _8_10_1.test.describe('Console cluster duplication', () => {
                 // Collect rather than throw: one cluster that will not delete must
                 // not strand the ones after it.
                 undeleted.push(`${clusterName} (${String(error)})`);
+                // A delete that failed half-way can leave the row-actions menu or
+                // the confirm dialog open, which blocks pointer events on the rest of
+                // the page and failed every delete after it.
+                await page.keyboard.press('Escape');
             }
         }
         // Soft, so the video capture below still runs, but still a failure: a
@@ -75,7 +86,11 @@ _8_10_1.test.describe('Console cluster duplication', () => {
             await clusterPage.clickClusterTab();
             createdClusterNames.push(sourceClusterName);
             await clusterPage.createCluster(sourceClusterName);
-            await clusterPage.assertClusterHealthyStatusWithRetry(sourceClusterName);
+            // Every health poll in this spec gets session recovery: the Hub session
+            // can drop to the identity provider's login form during these
+            // multi-minute polls, and reloading only re-renders that form (a run's
+            // retry spent its whole 500s budget on it).
+            await clusterPage.assertClusterHealthyStatusWithRetry(sourceClusterName, undefined, undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
             // A connector secret, an API client and an alert, so the duplication has
             // real settings to carry over instead of copying nothing.
             await clusterSecretsPage.createNewSecret(`DUP_SECRET_${randomSuffix}`, `dup-secret-value-${randomSuffix}`, sourceClusterName);
@@ -140,10 +155,10 @@ _8_10_1.test.describe('Console cluster duplication', () => {
             await (0, test_1.expect)(clusterPage.clusterLink(duplicateClusterName)).toBeVisible({
                 timeout: 60000,
             });
-            await clusterPage.assertClusterHealthyStatusWithRetry(duplicateClusterName);
+            await clusterPage.assertClusterHealthyStatusWithRetry(duplicateClusterName, undefined, undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
             await clusterPage.clickClusterLink(duplicateClusterName);
             await clusterDetailsPage.dismissOopsIfPresent();
-            await clusterDetailsPage.assertComponentsHealth();
+            await clusterDetailsPage.assertComponentsHealth(undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
         });
     });
     // Scoped to name and region. The epic's case also lists plan type, channel
@@ -168,7 +183,7 @@ _8_10_1.test.describe('Console cluster duplication', () => {
             // over". The second argument is the cloud provider, the fourth the
             // sub-region -- passing a region label as the provider matches no tab.
             await clusterPage.createCluster(sourceClusterName, 'GCP', undefined, SOURCE_SUB_REGION);
-            await clusterPage.assertClusterHealthyStatusWithRetry(sourceClusterName);
+            await clusterPage.assertClusterHealthyStatusWithRetry(sourceClusterName, undefined, undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
             // A populated source - against an empty one this case would pass by
             // copying nothing at all.
             await clusterSecretsPage.createNewSecret(`DUP_SECRET_${randomSuffix}`, `dup-secret-value-${randomSuffix}`, sourceClusterName);
@@ -231,10 +246,10 @@ _8_10_1.test.describe('Console cluster duplication', () => {
             await (0, test_1.expect)(clusterPage.clusterRegion(sourceClusterName, SOURCE_REGION_LABEL)).toBeVisible();
             // Completion rule: a duplicate that is created but never healthy is the
             // failure this epic exists to prevent.
-            await clusterPage.assertClusterHealthyStatusWithRetry(duplicateClusterName);
+            await clusterPage.assertClusterHealthyStatusWithRetry(duplicateClusterName, undefined, undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
             await clusterPage.clickClusterLink(duplicateClusterName);
             await clusterDetailsPage.dismissOopsIfPresent();
-            await clusterDetailsPage.assertComponentsHealth();
+            await clusterDetailsPage.assertComponentsHealth(undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
         });
     });
 });

package/package.json

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