@camunda/e2e-test-suite 0.0.1621 → 0.0.1622

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

@@ -55,7 +55,32 @@ declare class ClusterPage {
     deleteCluster(name: string): Promise<void>;
     clickClusterTab(): Promise<void>;
     clickCreateNewClusterButton(): Promise<void>;
-    clickClusterNameInput(): Promise<void>;
+    /**
+     * `recoverViaClusterList` decides what happens after a failed click:
+     * clickLocatorWithRetry runs it as the recovery between attempts, and it
+     * navigates to the cluster list.
+     *
+     * On the duplication form that is fatal. The form is reached from a
+     * cluster's own overflow menu, so navigating away abandons it: the name
+     * input no longer exists, and attempts 2 and 3 cannot find it however long
+     * they wait. One recoverable slow render therefore became "Failed to click
+     * locator getByPlaceholder('Enter a name for the cluster') after 3
+     * attempts" — which is why DUP-01 failed on a run where the same form
+     * worked for DUP-04 minutes earlier.
+     *
+     * The default is true to preserve existing behaviour, NOT because the
+     * recovery works on the create-cluster form. It does not: createCluster()
+     * opens that form once before awaiting this helper, and the retry loop
+     * never reopens it, so returning to the list strands those attempts the
+     * same way. It is simply rarely reached there — the create form renders
+     * synchronously after its button click, where the duplication form waits
+     * on a menu and a navigation. Turning it off for every caller is the real
+     * fix and belongs in its own change, with a run behind it: createCluster()
+     * is on the critical path of every spec in this suite.
+     */
+    clickClusterNameInput(options?: {
+        recoverViaClusterList?: boolean;
+    }): Promise<void>;
     fillClusterNameInput(name: string): Promise<void>;
     private extractClusterIdFromUrl;
     private logClusterCreationInfo;

package/dist/pages/8.10/ClusterPage.js

@@ -363,11 +363,39 @@ class ClusterPage {
             },
         });
     }
-    async clickClusterNameInput() {
+    /**
+     * `recoverViaClusterList` decides what happens after a failed click:
+     * clickLocatorWithRetry runs it as the recovery between attempts, and it
+     * navigates to the cluster list.
+     *
+     * On the duplication form that is fatal. The form is reached from a
+     * cluster's own overflow menu, so navigating away abandons it: the name
+     * input no longer exists, and attempts 2 and 3 cannot find it however long
+     * they wait. One recoverable slow render therefore became "Failed to click
+     * locator getByPlaceholder('Enter a name for the cluster') after 3
+     * attempts" — which is why DUP-01 failed on a run where the same form
+     * worked for DUP-04 minutes earlier.
+     *
+     * The default is true to preserve existing behaviour, NOT because the
+     * recovery works on the create-cluster form. It does not: createCluster()
+     * opens that form once before awaiting this helper, and the retry loop
+     * never reopens it, so returning to the list strands those attempts the
+     * same way. It is simply rarely reached there — the create form renders
+     * synchronously after its button click, where the duplication form waits
+     * on a menu and a navigation. Turning it off for every caller is the real
+     * fix and belongs in its own change, with a run behind it: createCluster()
+     * is on the critical path of every spec in this suite.
+     */
+    async clickClusterNameInput(options = {}) {
+        const { recoverViaClusterList = true } = options;
         await (0, clickLocatorWithRetry_1.clickLocatorWithRetry)(this.page, this.clusterNameInput, {
-            postAction: async () => {
-                await this.clickClusterBanner();
-            },
+            ...(recoverViaClusterList
+                ? {
+                    postAction: async () => {
+                        await this.clickClusterBanner();
+                    },
+                }
+                : {}),
         });
     }
     async fillClusterNameInput(name) {

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

@@ -124,7 +124,11 @@ _8_10_1.test.describe('Console cluster duplication', () => {
             await duplicateAction.click();
             // The pre-fill itself is covered a layer down (DUP-03); here we only rename
             // the duplicate so it is uniquely identifiable and deletable.
-            await clusterPage.clickClusterNameInput();
+            // No cluster-list recovery here: this form is reached from the
+            // cluster's overflow menu, so navigating away abandons it.
+            await clusterPage.clickClusterNameInput({
+                recoverViaClusterList: false,
+            });
             await clusterPage.fillClusterNameInput(duplicateClusterName);
             createdClusterNames.push(duplicateClusterName);
             // Duplicate opens a "Duplicate a cluster" page (not a dialog) that submits
@@ -205,7 +209,11 @@ _8_10_1.test.describe('Console cluster duplication', () => {
             await duplicateAction.click();
             // Every editable field of the duplication form, changed away from the
             // value the form carried over from the source cluster.
-            await clusterPage.clickClusterNameInput();
+            // No cluster-list recovery here: this form is reached from the
+            // cluster's overflow menu, so navigating away abandons it.
+            await clusterPage.clickClusterNameInput({
+                recoverViaClusterList: false,
+            });
             await clusterPage.fillClusterNameInput(duplicateClusterName);
             // clickGCPRegion only acts when REGION_SELECTION is set, so the tile
             // click below is what actually moves the region either way.
@@ -256,6 +264,125 @@ _8_10_1.test.describe('Console cluster duplication', () => {
             await clusterDetailsPage.assertComponentsHealth(undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
         });
     });
+    // DUP-04 - the case the epic exists for: configuration drift. Everything is
+    // asserted on the cluster that was really created, never on the duplication
+    // form's pre-fill (DUP-03 owns that layer).
+    //
+    // Scope. The epic lists plan type, region, channel and generation, labels,
+    // IP allowlist rules, encryption settings, connector secrets, alert
+    // subscriptions, BYOM credentials, the BYOIDP secret and the backup
+    // schedule. This case asserts the two the suite's page objects can both
+    // populate on the source and read back off the duplicate: region and
+    // connector secrets. The rest have no page object that can set them on the
+    // source, and a source cluster where a setting was never populated makes
+    // the matching assertion on the duplicate pass by copying nothing - exactly
+    // the false green the preconditions warn about. Covering them needs
+    // page-object work first, not an invented selector here.
+    //
+    // Alert subscriptions belong to that second group despite createAlert()
+    // existing. It takes no recipient and no name: it selects the Email
+    // delivery method and submits. So an alert on the duplicate cannot be told
+    // apart from any other alert, and asserting one would pass whether or not
+    // duplication copied anything. assertAlertText() reads the detail row, not
+    // the recipient - every other caller in this suite passes an alert name
+    // such as 'Amount triggered1'. Asserting carry-over needs createAlert() to
+    // accept something that distinguishes the subscription.
+    (0, _8_10_1.test)('DUP-04: the carried-over settings are present on the created duplicate', async ({ page, homePage, clusterPage, clusterDetailsPage, clusterSecretsPage, }) => {
+        // Two cluster creations plus health polling - this case is long by nature.
+        _8_10_1.test.setTimeout(60 * 60 * 1000);
+        const randomSuffix = await (0, _setup_1.generateRandomStringAsync)(6);
+        const sourceClusterName = `dup-src-cfg-${randomSuffix}`;
+        const duplicateClusterName = `dup-cfg-${randomSuffix}`;
+        const secretKey = `DUP_SECRET_${randomSuffix}`;
+        const secretValue = `dup-secret-value-${randomSuffix}`;
+        await _8_10_1.test.step('given a healthy source cluster in a known region, with a connector secret', async () => {
+            await clusterPage.clickClusterTab();
+            createdClusterNames.push(sourceClusterName);
+            // The region is pinned rather than left to selectSubRegion()'s random
+            // pick: a known source region is the only way the duplicate's region can
+            // be asserted as "the source's one" instead of "some region".
+            await clusterPage.createCluster(sourceClusterName, 'GCP', undefined, SOURCE_SUB_REGION);
+            await clusterPage.assertClusterHealthyStatusWithRetry(sourceClusterName, undefined, undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
+            await clusterSecretsPage.createNewSecret(secretKey, secretValue, sourceClusterName);
+            // The source really holds each setting before anything is duplicated.
+            // Without this, a missing setting on the duplicate below would be
+            // indistinguishable from a setting that was never created at all.
+            await homePage.clickClusters();
+            await (0, UtilitiesPage_1.waitForLoadingToFinish)(page);
+            await (0, test_1.expect)(clusterPage.clusterRegion(sourceClusterName, SOURCE_REGION_LABEL)).toBeVisible({ timeout: 60000 });
+        });
+        await _8_10_1.test.step('when I duplicate that cluster, changing only its name', async () => {
+            await homePage.clickClusters();
+            await clusterPage.clickClusterLink(sourceClusterName);
+            await clusterDetailsPage.dismissOopsIfPresent();
+            await (0, UtilitiesPage_1.waitForLoadingToFinish)(page);
+            await page.getByRole('button', { name: 'cluster operations' }).click();
+            const duplicateAction = page
+                .getByRole('menuitem', { name: /duplicate/i })
+                .or(page.getByRole('button', { name: /duplicate/i }));
+            await (0, test_1.expect)(duplicateAction).toBeVisible({ timeout: 30000 });
+            await duplicateAction.click();
+            // Only the name changes: every other field keeps whatever the form
+            // carried over, which is what this case is here to verify landed.
+            await clusterPage.clickClusterNameInput();
+            await clusterPage.fillClusterNameInput(duplicateClusterName);
+            createdClusterNames.push(duplicateClusterName);
+            const submitDuplicate = page.getByRole('button', {
+                name: 'Duplicate cluster',
+                exact: true,
+            });
+            await (0, test_1.expect)(submitDuplicate).toBeEnabled({ timeout: 30000 });
+            // Same as DUP-01/DUP-02: hold the page until the duplicate POST has
+            // answered, otherwise navigating away aborts the in-flight request.
+            const duplicateResponse = page.waitForResponse((response) => response.request().method() === 'POST' &&
+                /\/clusters\/[^/]+\/duplicate$/.test(new URL(response.url()).pathname), { timeout: 60000 });
+            await submitDuplicate.click();
+            const duplicate = await duplicateResponse;
+            (0, test_1.expect)(duplicate.ok()).toBe(true);
+            const { clusterId } = await duplicate.json();
+            await page.waitForURL(new RegExp(`/cluster/${clusterId}`), {
+                timeout: 60000,
+            });
+        });
+        await _8_10_1.test.step('then the duplicate comes up healthy', async () => {
+            await homePage.clickClusters();
+            await (0, UtilitiesPage_1.waitForLoadingToFinish)(page);
+            await (0, test_1.expect)(clusterPage.clusterLink(duplicateClusterName)).toBeVisible({
+                timeout: 60000,
+            });
+            // Completion rule: the settings below are only meaningful on a cluster
+            // that actually runs, so health is asserted before them.
+            await clusterPage.assertClusterHealthyStatusWithRetry(duplicateClusterName, undefined, undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
+            await clusterPage.clickClusterLink(duplicateClusterName);
+            await clusterDetailsPage.dismissOopsIfPresent();
+            await clusterDetailsPage.assertComponentsHealth(undefined, (0, UtilitiesPage_1.consoleSessionRecovery)(page, page.url()));
+        });
+        await _8_10_1.test.step('and the duplicate carries the region of the source cluster', async () => {
+            await homePage.clickClusters();
+            await (0, UtilitiesPage_1.waitForLoadingToFinish)(page);
+            // Scoped to the duplicate's own row: the source is still listed on this
+            // page with the same label, so a page-wide text read would pass on the
+            // source alone.
+            await (0, test_1.expect)(clusterPage.clusterRegion(duplicateClusterName, SOURCE_REGION_LABEL)).toBeVisible({ timeout: 60000 });
+            // And it did not silently land in the other region the org can create
+            // in - drift in either direction is the failure this case guards.
+            await (0, test_1.expect)(clusterPage.clusterRegion(duplicateClusterName, OVERRIDE_REGION_LABEL)).toHaveCount(0);
+        });
+        await _8_10_1.test.step('and the connector secret of the source is present on the duplicate', async () => {
+            await homePage.clickClusters();
+            await clusterPage.clickClusterLink(duplicateClusterName);
+            await clusterDetailsPage.dismissOopsIfPresent();
+            await clusterPage.clickConnectorSecretsTab(duplicateClusterName);
+            await (0, UtilitiesPage_1.waitForLoadingToFinish)(page);
+            // Scoped to the duplicate's own secrets view. Only the key is asserted:
+            // Hub never renders a secret's value back, by design.
+            const duplicateSecrets = page.getByRole('main');
+            const carriedOverSecret = duplicateSecrets
+                .getByRole('cell', { name: secretKey, exact: true })
+                .or(duplicateSecrets.getByText(secretKey, { exact: true }));
+            await (0, test_1.expect)(carriedOverSecret.first()).toBeVisible({ timeout: 60000 });
+        });
+    });
     // DUP-05. The epic's claim for this slice is a claim about the form: the
     // admin learns *before submitting* that the process data of the source does
     // not come across and the duplicate starts empty. So the assertion is the
@@ -314,7 +441,11 @@ _8_10_1.test.describe('Console cluster duplication', () => {
             await (0, test_1.expect)(submitDuplicate).toBeVisible({ timeout: 30000 });
         });
         await _8_10_1.test.step('and submitting that form creates a duplicate that reaches a healthy running state', async () => {
-            await clusterPage.clickClusterNameInput();
+            // No cluster-list recovery here: this form is reached from the
+            // cluster's overflow menu, so navigating away abandons it.
+            await clusterPage.clickClusterNameInput({
+                recoverViaClusterList: false,
+            });
             await clusterPage.fillClusterNameInput(duplicateClusterName);
             createdClusterNames.push(duplicateClusterName);
             const submitDuplicate = page.getByRole('button', {

package/dist/tests/8.10/test-setup.spec.js

@@ -21,6 +21,11 @@ _8_10_1.test.describe('Cluster Setup Tests', () => {
     (0, _8_10_1.test)('Enable Alpha Features', async ({ page, loginPage, consoleOrganizationsPage, homePage, }, testInfo) => {
         _8_10_1.test.skip(process.env.IS_PROD === 'true', 'Skipping test because not required on PROD test org');
         _8_10_1.test.skip(process.env.IS_AG === 'true', 'Skipping test because not required when IS_AG is true');
+        // A scoped run judges one case against a throwaway organisation that is
+        // not entitled to alpha features, so the AI-powered-features row never
+        // renders and this test cannot pass there. The nightlies, which run
+        // against an entitled org, are unaffected: IS_SCOPED_RUN is unset there.
+        _8_10_1.test.skip(process.env.IS_SCOPED_RUN === 'true', 'Skipping test because a scoped run org has no alpha-feature entitlement');
         await (0, UtilitiesPage_1.loginWithRetry)(page, loginPage, users_1.testUsers.mainUser, (testInfo.workerIndex + 1) * 1000);
         await homePage.clickOrganization();
         await (0, UtilitiesPage_1.assertLocatorVisibleWithRetry)(page, consoleOrganizationsPage.settingsTab, 'settings tab', 60000, false, 2);
@@ -72,16 +77,27 @@ _8_10_1.test.describe('Cluster Setup Tests', () => {
             // a bounded, non-blocking loss -- not worth risking the whole run for.
             if (process.env.IS_AG !== 'true') {
                 await clusterPage.clickConnectorSecretsTab(clusterName);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.awsSecretsValues);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.openAiSecretsValues);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.gcpSecretsValues);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.azureSecretsValues);
+                // Every set below but connectorSecretsValues reads
+                // credentials from the environment. A scoped run imports
+                // none of them, so each value is undefined and the create
+                // fallback throws. connectorSecretsValues is three
+                // literals, so it is seeded there too -- the connector
+                // secret user flows need it and it cannot be undefined.
+                const envBackedSecrets = process.env.IS_SCOPED_RUN !== 'true';
+                if (envBackedSecrets) {
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.awsSecretsValues);
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.openAiSecretsValues);
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.gcpSecretsValues);
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.azureSecretsValues);
+                }
                 await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.connectorSecretsValues);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.bedrockKbSecretsValues);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.agentcoreSecretsValues);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.agentcoreLtmSecretsValues);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.bedrockCodeInterpreterSecretsValues);
-                await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.anthropicBedrockMantleSecretsValues);
+                if (envBackedSecrets) {
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.bedrockKbSecretsValues);
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.agentcoreSecretsValues);
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.agentcoreLtmSecretsValues);
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.bedrockCodeInterpreterSecretsValues);
+                    await clusterSecretsPage.createSetOfSecrets(clusterName, connectorSecrets_1.anthropicBedrockMantleSecretsValues);
+                }
             }
         }
     });

package/package.json

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