@camunda/e2e-test-suite 0.0.615 → 0.0.617

package/dist/pages/SM-8.10/LoginPage.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.LoginPage = void 0;
-const test_1 = require("@playwright/test");
 class LoginPage {
     page;
     usernameInput;
@@ -17,18 +16,31 @@ class LoginPage {
         this.page = page;
         this.usernameInput = page.getByLabel('Username or email');
         this.passwordInput = page.getByLabel('Password');
-        this.loginButton = page.getByRole('button', { name: 'Log in' });
+        this.loginButton = page.getByRole('button', { name: /log in|sign in/i });
         this.coreUsernameInput = page.getByPlaceholder('Username');
         this.corePasswordInput = page.getByPlaceholder('Password');
         this.coreLoginButton = page.getByRole('button', { name: 'Login' });
         this.keycloakBanner = page.locator('body#keycloak-bg[data-page-id="login-login"]');
     }
     async detectLoginForm() {
-        const anyForm = this.usernameInput
-            .or(this.coreUsernameInput)
-            .or(this.keycloakBanner);
-        await anyForm.waitFor({ state: 'visible', timeout: 15000 });
-        if (await this.coreUsernameInput.isVisible()) {
+        // If we're on an app page that's about to redirect to KC, wait for
+        // the navigation to complete before looking for login form elements.
+        const url = this.page.url();
+        if (!url.includes('/auth/') && !url.includes('/login-callback')) {
+            await this.page
+                .waitForURL((u) => u.pathname.includes('/auth/'), {
+                timeout: 30000,
+                waitUntil: 'domcontentloaded',
+            })
+                .catch(() => { });
+        }
+        const kcUsername = this.page.locator('#username');
+        const anyForm = kcUsername
+            .or(this.coreLoginButton)
+            .or(this.keycloakBanner)
+            .first();
+        await anyForm.waitFor({ state: 'visible', timeout: 30000 });
+        if (await this.coreLoginButton.isVisible()) {
             this.useCore = true;
         }
         else if (await this.keycloakBanner.isVisible()) {
@@ -78,17 +90,6 @@ class LoginPage {
     async login(username, password) {
         try {
             await this.detectLoginForm();
-            if (await this.page
-                .locator('body#keycloak-bg[data-page-id="login-login"]')
-                .isVisible()) {
-                await (0, test_1.expect)(this.keycloakBanner).toBeVisible({ timeout: 15000 });
-            }
-            else {
-                const activeLoginButton = this.useCore
-                    ? this.coreLoginButton
-                    : this.loginButton;
-                await (0, test_1.expect)(activeLoginButton).toBeVisible({ timeout: 15000 });
-            }
         }
         catch (error) {
             return;

package/dist/pages/SM-8.10/NavigationPage.js

@@ -77,7 +77,13 @@ class NavigationPage {
                     .first();
                 await (0, test_1.expect)(loginEntryPoint).toBeVisible({ timeout: 60000 });
                 const isBannerVisible = await banner.isVisible().catch(() => false);
-                if (isBannerVisible) {
+                // Guard against the Modeler SPA rendering its header on the login
+                // page before the OAuth redirect fires. If the banner is visible
+                // but the URL is still a login/auth path, the user is NOT actually
+                // authenticated — wait for the real login form to appear.
+                const currentUrl = this.page.url();
+                const onLoginPath = currentUrl.includes('/login') || currentUrl.includes('/auth/');
+                if (isBannerVisible && !onLoginPath) {
                     const loginVisible = await loginPage.usernameInput.isVisible();
                     if (loginVisible) {
                         await loginPage.login(username, password);

package/dist/pages/SM-8.10/OperateProcessInstancePage.d.ts

@@ -21,6 +21,7 @@ declare class OperateProcessInstancePage {
     assertEitherIncidentOrCompletedIconVisible(): Promise<string>;
     assertProcessCompleteStatusWithRetry(timeout?: number, maxRetries?: number): Promise<void>;
     assertProcessVariableContainsText(variableName: string, text: string): Promise<void>;
+    private captureVariableJsonValue;
     assertResultVariableVisibleWithRetry(variableName: string): Promise<void>;
     assertActiveTokenIsPresent(): Promise<void>;
     assertVariablesListVisible(timeout?: number): Promise<void>;

package/dist/pages/SM-8.10/OperateProcessInstancePage.js

@@ -162,8 +162,21 @@ class OperateProcessInstancePage {
                 await (0, test_1.expect)(this.page.getByTestId(`variable-${variableName}`)).toContainText(text, { timeout: 8000 });
                 return;
             }
-            catch (error) {
-                console.log(`assertProcessVariableContainsText attempt ${attempt + 1}/${maxRetries} failed: ${error}`);
+            catch (uiError) {
+                // Document-typed variables render as a compact summary
+                // (e.g. "3 documents" or "<docId> <size>") that omits metadata
+                // such as filename, storeId, and contentType. Fall back to the
+                // raw JSON value from the variables API response.
+                try {
+                    const value = await this.captureVariableJsonValue(variableName);
+                    if (value != null && value.includes(text)) {
+                        return;
+                    }
+                }
+                catch (apiError) {
+                    console.log(`captureVariableJsonValue for ${variableName} attempt ${attempt + 1}/${maxRetries} failed: ${apiError}`);
+                }
+                console.log(`assertProcessVariableContainsText attempt ${attempt + 1}/${maxRetries} failed: ${uiError}`);
                 if (attempt < maxRetries - 1) {
                     await this.page.reload({ timeout: 10000 }).catch(() => { });
                     await this.page.waitForLoadState('domcontentloaded').catch(() => { });
@@ -172,6 +185,34 @@ class OperateProcessInstancePage {
         }
         throw new Error(`Failed to assert variable ${variableName} contains "${text}" after ${maxRetries} attempts.`);
     }
+    async captureVariableJsonValue(variableName) {
+        let capturedValue = null;
+        const responsePromise = this.page
+            .waitForResponse(async (resp) => {
+            if (!resp.url().includes('/v2/variables/search') ||
+                resp.status() !== 200) {
+                return false;
+            }
+            try {
+                const body = await resp.text();
+                const data = JSON.parse(body);
+                const item = data?.items?.find((i) => i.name === variableName);
+                if (item?.value != null) {
+                    capturedValue = String(item.value);
+                    return true;
+                }
+                return false;
+            }
+            catch {
+                return false;
+            }
+        }, { timeout: 15000 })
+            .catch(() => null);
+        await this.page.reload({ timeout: 10000 }).catch(() => { });
+        await this.page.waitForLoadState('domcontentloaded').catch(() => { });
+        await responsePromise;
+        return capturedValue;
+    }
     async assertResultVariableVisibleWithRetry(variableName) {
         const maxRetries = 3;
         for (let attempt = 0; attempt < maxRetries; attempt++) {

package/package.json

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