@camunda/e2e-test-suite 0.0.1425 → 0.0.1426

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

@@ -20,6 +20,7 @@ declare class ClusterPage {
     readonly dangerDeleteButton: Locator;
     readonly testClusterLink: Locator;
     readonly connectorSecretsTab: Locator;
+    readonly connectorSecretsTabSelected: Locator;
     readonly createAClusterButton: Locator;
     readonly clusterVersionOption: Locator;
     readonly generationSearchInput: Locator;

package/dist/pages/8.10/ClusterPage.js

@@ -25,6 +25,7 @@ class ClusterPage {
     dangerDeleteButton;
     testClusterLink;
     connectorSecretsTab;
+    connectorSecretsTabSelected;
     createAClusterButton;
     clusterVersionOption;
     generationSearchInput;
@@ -80,6 +81,13 @@ class ClusterPage {
             .or(page.getByRole('tab', {
             name: 'Connector Secrets',
         }));
+        // Same tab, but only when it is the active one. Hub renders the cluster
+        // tabs as Radix triggers whose active tab is controlled React state, so a
+        // visually successful click is not proof the panel switched -- see
+        // clickConnectorSecretsTab.
+        this.connectorSecretsTabSelected = page
+            .getByRole('tab', { name: 'Cluster secrets', selected: true })
+            .or(page.getByRole('tab', { name: 'Connector Secrets', selected: true }));
         // Camunda Hub's create-cluster form replaced the Carbon tiles (a
         // <label> per option) and tabs with design-system choice cards and toggle
         // groups, all exposed as role="radio" named by their title. Each locator
@@ -441,12 +449,41 @@ class ClusterPage {
         });
     }
     async clickConnectorSecretsTab(clusterName) {
-        await (0, clickLocatorWithRetry_1.clickLocatorWithRetry)(this.page, this.connectorSecretsTab, {
-            postAction: async () => {
-                await this.clickClusterBanner();
-                await this.clickClusterLink(clusterName);
-            },
+        // Hub's cluster tabs are Radix triggers whose active tab is controlled
+        // React state. A background refetch landing right after the click can
+        // reset that state back to the default Overview tab, so a single
+        // successful click sometimes leaves the panel on Overview and the secrets
+        // Import button never mounts (AlwaysGreen run 38259225730: the tab was
+        // clicked on a visible/enabled/stable element yet Overview stayed
+        // selected for the full 30s Import-button wait). Click, confirm the
+        // secrets tab actually became selected, and re-click until it does.
+        const maxAttempts = 3;
+        const importButton = this.page.getByRole('button', {
+            name: 'Import',
+            exact: true,
+        });
+        for (let attempt = 1; attempt <= maxAttempts; attempt++) {
+            await (0, clickLocatorWithRetry_1.clickLocatorWithRetry)(this.page, this.connectorSecretsTab, {
+                postAction: async () => {
+                    await this.clickClusterBanner();
+                    await this.clickClusterLink(clusterName);
+                },
+            });
+            const activated = await importButton
+                .waitFor({ state: 'visible', timeout: 10000 })
+                .then(() => this.connectorSecretsTabSelected.isVisible())
+                .catch(() => false);
+            if (activated) {
+                return;
+            }
+        }
+        // Still on Overview after re-clicking: fail loudly rather than proceeding
+        // to a secrets panel that never mounted -- this keeps a genuinely broken
+        // panel red instead of masking it.
+        await (0, test_1.expect)(this.connectorSecretsTabSelected).toBeVisible({
+            timeout: 10000,
         });
+        await (0, test_1.expect)(importButton).toBeVisible({ timeout: 10000 });
     }
     async clickLatestStableForMinorVersion() {
         const minorVersion = process.env.MINOR_VERSION;

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

@@ -19,6 +19,7 @@ declare class ClusterSecretsPage {
     readonly clusterBanner: Locator;
     readonly clusterLink: (name: string) => Locator;
     readonly clusterSecretTab: Locator;
+    readonly secretsFetchErrorReloadButton: Locator;
     readonly confirmDeleteInput: Locator;
     constructor(page: Page);
     deleteConnectorSecretsIfExist(): Promise<void>;
@@ -44,5 +45,6 @@ declare class ClusterSecretsPage {
         name: string;
         value: string;
     }[]): Promise<void>;
+    private restoreClusterPageIfSignedOut;
 }
 export { ClusterSecretsPage };

package/dist/pages/8.10/ClusterSecretsPage.js

@@ -4,6 +4,7 @@ exports.ClusterSecretsPage = void 0;
 const test_1 = require("@playwright/test");
 const sleep_1 = require("../../utils/sleep");
 const expectLocatorWithRetry_1 = require("../../utils/assertionHelpers/expectLocatorWithRetry");
+const sessionRecovery_1 = require("../../utils/sessionRecovery");
 class ClusterSecretsPage {
     page;
     createNewSecretButton;
@@ -24,6 +25,7 @@ class ClusterSecretsPage {
     clusterBanner;
     clusterLink;
     clusterSecretTab;
+    secretsFetchErrorReloadButton;
     confirmDeleteInput;
     constructor(page) {
         this.page = page;
@@ -112,6 +114,11 @@ class ClusterSecretsPage {
         this.clusterSecretTab = this.page
             .getByRole('tab', { name: 'Cluster secrets' })
             .or(this.page.getByRole('tab', { name: 'Connector Secrets' }));
+        // c4's secrets tab replaces the list with an "Oops ... something went
+        // wrong." card when its GET of the secrets fails; this button refetches.
+        this.secretsFetchErrorReloadButton = page
+            .getByRole('tabpanel', { name: /^(Cluster secrets|Connector Secrets)$/ })
+            .getByRole('button', { name: 'Reload', exact: true });
         this.confirmDeleteInput = page.getByRole('textbox', {
             name: 'Type the word DELETE to confirm',
         });
@@ -390,6 +397,7 @@ class ClusterSecretsPage {
         await (0, test_1.expect)(this.createNewSecretButton).toBeVisible({
             timeout: 60000,
         });
+        const clusterUrl = this.page.url();
         await this.page.reload();
         // Hub keeps the selected cluster tab out of the URL, so the reload lands
         // on Overview, where assertSecretsOrCreate finds no search box and tries
@@ -397,10 +405,32 @@ class ClusterSecretsPage {
         // The cluster page can still be hydrating after the reload and snap back
         // to Overview after the click lands (CI trace: click succeeded, the
         // screenshot still showed Overview), so re-click until the panel renders.
+        // Re-clicking the already selected tab does not refetch a failed secrets
+        // list, so use the error card's own Reload button for that case.
         await (0, test_1.expect)(async () => {
-            await this.clusterSecretTab.first().click({ timeout: 15000 });
+            await this.restoreClusterPageIfSignedOut(clusterUrl);
+            if (await this.secretsFetchErrorReloadButton.isVisible()) {
+                await this.secretsFetchErrorReloadButton.click({ timeout: 15000 });
+            }
+            else {
+                await this.clusterSecretTab.first().click({ timeout: 15000 });
+            }
             await (0, test_1.expect)(this.createNewSecretButton).toBeVisible({ timeout: 10000 });
-        }).toPass({ timeout: 90000 });
+        }).toPass({ timeout: 180000 });
+    }
+    // A reload is a full Hub navigation through Auth0 and /login-callback. When
+    // Hub's session hand-off fails it calls Auth0 logout, which parks the tab on
+    // the identity provider's login form (PR run 38354316061). Sign back in as
+    // the same user, then open the cluster with a fresh goto: the callback code
+    // is single use, so that URL cannot be reloaded.
+    async restoreClusterPageIfSignedOut(clusterUrl) {
+        if (!(0, sessionRecovery_1.isAtIdentityProvider)(this.page) ||
+            !(await (0, sessionRecovery_1.restoreSessionIfLoggedOut)(this.page))) {
+            return;
+        }
+        await this.page.goto(clusterUrl, { waitUntil: 'domcontentloaded' });
+        await this.page.waitForURL((url) => url.host === new URL(clusterUrl).host &&
+            !/^\/login(-callback)?\/?$/.test(url.pathname), { timeout: 120000 });
     }
 }
 exports.ClusterSecretsPage = ClusterSecretsPage;

package/dist/tests/SM-8.10/smoke-tests.spec.js

@@ -10,7 +10,7 @@ const resetSession_1 = require("../../utils/resetSession");
 const KeycloakUtils_1 = require("../../pages/SM-8.10/KeycloakUtils");
 const constants_1 = require("../../utils/constants");
 SM_8_10_1.test.describe.parallel('Smoke Tests', () => {
-    SM_8_10_1.test.beforeEach(async ({ navigationPage, managementIdentityPage, managementIdentityTenantPage, keycloakLoginPage, keycloakAdminPage, page, browser, loginPage, ocIdentityHomePage, ocIdentityRolesPage, }, testInfo) => {
+    SM_8_10_1.test.beforeEach(async ({ navigationPage, managementIdentityPage, managementIdentityTenantPage, keycloakLoginPage, keycloakAdminPage, page, browser, ocIdentityHomePage, ocIdentityRolesPage, }, testInfo) => {
         if (process.env.IS_MIGRATION === 'true') {
             await navigationPage.goToOCAdmin((testInfo.workerIndex + 1) * 1000);
         }
@@ -20,7 +20,14 @@ SM_8_10_1.test.describe.parallel('Smoke Tests', () => {
             const { testUsername, testPassword } = await (0, KeycloakUtils_1.setupKeycloakUser)(navigationPage, managementIdentityPage, keycloakLoginPage, keycloakAdminPage, ocIdentityHomePage, ocIdentityRolesPage, process.env.IS_MT === 'true', managementIdentityTenantPage);
             console.log(`BeforeEach end: timeout = ${testInfo.timeout}`);
             await (0, resetSession_1.resetSession)(browser, page);
-            await loginPage.login(testUsername, testPassword);
+            // Not loginPage.login() on the page resetSession leaves us on: where
+            // "/" serves Console it never redirects to Keycloak, so login()
+            // returned false unnoticed and later goTo() calls fell back to the
+            // first user (an org admin) instead of this test user.
+            await navigationPage.goToManagementIdentity(undefined, {
+                username: testUsername,
+                password: testPassword,
+            });
         }
     });
     SM_8_10_1.test.afterEach(async ({ page }, testInfo) => {
@@ -64,7 +71,7 @@ SM_8_10_1.test.describe.parallel('Smoke Tests', () => {
         }
         console.log(`[${testInfo.title}] Test end: timeout = ${testInfo.timeout}`);
     });
-    (0, SM_8_10_1.test)('Most Common Flow User Flow With All Apps', async ({ page, modelerHomePage, navigationPage, modelerCreatePage, operateHomePage, operateProcessesPage, operateProcessInstancePage, optimizeHomePage, optimizeCollectionsPage, optimizeReportPage, taskDetailsPage, taskPanelPage, }, testInfo) => {
+    (0, SM_8_10_1.test)('Most Common Flow User Flow With All Apps', async ({ page, browser, modelerHomePage, navigationPage, modelerCreatePage, operateHomePage, operateProcessesPage, operateProcessInstancePage, optimizeHomePage, optimizeCollectionsPage, optimizeReportPage, taskDetailsPage, taskPanelPage, }, testInfo) => {
         console.log(`[${testInfo.title}] Test start: timeout = ${testInfo.timeout}`);
         const randomName = await (0, _setup_1.generateRandomStringAsync)(3);
         const processName = 'Zeebe_User_Task_Process' + randomName;
@@ -72,6 +79,7 @@ SM_8_10_1.test.describe.parallel('Smoke Tests', () => {
         await SM_8_10_1.test.step('Open Cross Component Test Project and Create a BPMN Diagram Template', async () => {
             await navigationPage.goToModeler();
             await modelerHomePage.createCrossComponentProjectFolder();
+            await modelerHomePage.assignWorkspaceEnvironmentAsOrgAdmin(browser);
             await modelerHomePage.clickDiagramTypeDropdown();
             await modelerHomePage.clickBpmnTemplateOption();
         });

package/package.json

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