@camunda/e2e-test-suite 0.0.1170 → 0.0.1171

package/dist/pages/c8Run-8.10/OperateProcessesPage.d.ts

@@ -17,6 +17,6 @@ declare class OperateProcessesPage {
     clickProcessIncidentsCheckbox(): Promise<void>;
     clickRunningProcessInstancesCheckbox(): Promise<void>;
     clickFinishedProcessInstancesCheckbox(): Promise<void>;
-    clickProcessInstanceLink(processName: string): Promise<void>;
+    clickProcessInstanceLink(processName: string, processInstanceKey?: string): Promise<void>;
 }
 export { OperateProcessesPage };

package/dist/pages/c8Run-8.10/OperateProcessesPage.js

@@ -61,20 +61,33 @@ class OperateProcessesPage {
     async clickFinishedProcessInstancesCheckbox() {
         await this.processFinishedInstancesCheckbox.click({ timeout: 90000 });
     }
-    async clickProcessInstanceLink(processName) {
+    async clickProcessInstanceLink(processName, processInstanceKey) {
+        // Both Operate frontends label the instance-key cell link
+        // "View instance <key>" -- the aria-label in operate/client's
+        // InstancesTable, the operate.processes.instancesTable.viewInstance string
+        // in the orchestration-cluster webapp -- so passing a key pins the click
+        // to one instance. Without a key every row carrying the process name is a
+        // candidate and `.first()` opens an arbitrary instance of that process.
+        // Both frontends sort the list by start date descending, so a
+        // just-created instance is on the first page.
+        const instanceLink = processInstanceKey === undefined
+            ? this.page
+                .getByRole('row')
+                .filter({ hasText: new RegExp(`^(?:Select row)?${processName}\\d+`) })
+                .first()
+                .getByRole('link')
+            : this.page.getByRole('link', {
+                name: `View instance ${processInstanceKey}`,
+                exact: true,
+            });
         let retryCount = 0;
-        const maxRetries = 3;
+        const maxRetries = 4;
         while (retryCount < maxRetries) {
             try {
                 await (0, sleep_1.sleep)(1000);
                 await this.page.reload();
                 await (0, sleep_1.sleep)(1000);
-                await this.page
-                    .getByRole('row')
-                    .filter({ hasText: new RegExp(`^(?:Select row)?${processName}\\d+`) })
-                    .first()
-                    .getByRole('link')
-                    .click({ timeout: 30000 });
+                await instanceLink.click({ timeout: 30000 });
                 await (0, test_1.expect)(this.page.getByText('Instance History').first()).toBeVisible({
                     timeout: 10000,
                 });

package/dist/pages/c8Run-8.10/TaskDetailsPage.js

@@ -220,7 +220,27 @@ class TaskDetailsPage {
         // taskVariablesValueLabel) associated by htmlFor -- the old Carbon input
         // exposed the same string as an HTML title attribute, which getByTitle
         // matched directly; getByLabel is the equivalent for a real <label>.
-        await (0, test_1.expect)(this.page.getByLabel(variableName + ' Value')).toHaveValue(variableValue, { timeout: 60000 });
+        //
+        // The panel fetches the task's variables once, when the task is opened, so
+        // polling the DOM alone can never observe a value that was still being
+        // exported at that moment. Reload between attempts to force a re-fetch --
+        // same retry shape as taskAssertion/priorityAssertion above.
+        const valueInput = this.page.getByLabel(variableName + ' Value');
+        const maxRetries = 3;
+        for (let attempt = 1; attempt <= maxRetries; attempt++) {
+            try {
+                await (0, test_1.expect)(valueInput).toHaveValue(variableValue, { timeout: 30000 });
+                return;
+            }
+            catch (error) {
+                if (attempt === maxRetries) {
+                    throw error;
+                }
+                console.log(`Attempt ${attempt} failed. Retrying...`);
+                await this.page.reload();
+                await (0, sleep_1.sleep)(5000);
+            }
+        }
     }
 }
 exports.TaskDetailsPage = TaskDetailsPage;

package/dist/pages/c8Run-8.10/UtilitiesPage.d.ts

@@ -1,7 +1,8 @@
-import { Page, Locator } from '@playwright/test';
+import { APIRequestContext, Page, Locator } from '@playwright/test';
 import { TaskPanelPage } from '../c8Run-8.10/TaskPanelPage';
 import { TaskDetailsPage } from '../c8Run-8.10/TaskDetailsPage';
 export declare function navigateToApp(page: Page, appName: string): Promise<void>;
+export declare function openUserTaskOfProcessInstance(page: Page, request: APIRequestContext, processInstanceKey: string, maxRetries?: number, retryDelayMs?: number): Promise<string>;
 export declare function assertLocatorVisibleWithRetry(page: Page, locator: Locator, text: string, timeout?: number, maxRetries?: number): Promise<void>;
 export declare function validateURL(page: Page, URL: RegExp): Promise<void>;
 export declare function completeTaskWithRetry(taskPanelPage: TaskPanelPage, taskDetailsPage: TaskDetailsPage, taskName: string, taskPriority: string, userTaskName?: string, textInput?: string, maxRetries?: number): Promise<void>;

package/dist/pages/c8Run-8.10/UtilitiesPage.js

@@ -7,13 +7,42 @@
  * except in compliance with the Camunda License 1.0.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.completeTaskWithRetry = exports.validateURL = exports.assertLocatorVisibleWithRetry = exports.navigateToApp = void 0;
+exports.completeTaskWithRetry = exports.validateURL = exports.assertLocatorVisibleWithRetry = exports.openUserTaskOfProcessInstance = exports.navigateToApp = void 0;
 const test_1 = require("@playwright/test");
 const sleep_1 = require("../../utils/sleep");
 async function navigateToApp(page, appName) {
     page.goto('/' + appName.toLowerCase() + '/login');
 }
 exports.navigateToApp = navigateToApp;
+async function openUserTaskOfProcessInstance(page, request, processInstanceKey, maxRetries = 30, retryDelayMs = 2000) {
+    // A task card shows only the task and process display names (Task.tsx in
+    // camunda/camunda), so nothing in the Tasklist UI tells two instances of the
+    // same process apart -- opening "the first card named X" lands on whichever
+    // instance happens to sort first. Resolve the user task of THIS instance
+    // over the v2 API instead (c8Run serves it unauthenticated:
+    // camunda.security.authentication.unprotected-api is true in
+    // c8run/configuration/application.yaml, and api-tests-v2.spec.ts calls
+    // /v2/user-tasks/search the same way) and open it at its own route,
+    // /tasklist/{userTaskKey}. Retrying covers the export lag between creating
+    // the instance and its user task becoming searchable.
+    for (let attempt = 1; attempt <= maxRetries; attempt++) {
+        const response = await request.post('/v2/user-tasks/search', {
+            data: { filter: { processInstanceKey } },
+        });
+        if (response.ok()) {
+            const body = await response.json();
+            const userTaskKey = body?.items?.[0]?.userTaskKey;
+            if (userTaskKey != null) {
+                await page.goto('/tasklist/' + userTaskKey);
+                return String(userTaskKey);
+            }
+        }
+        await (0, sleep_1.sleep)(retryDelayMs);
+    }
+    throw new Error(`No user task found for process instance ${processInstanceKey} after ` +
+        `${maxRetries} attempts.`);
+}
+exports.openUserTaskOfProcessInstance = openUserTaskOfProcessInstance;
 async function assertLocatorVisibleWithRetry(page, locator, text, timeout = 20000, maxRetries = 3) {
     for (let attempt = 0; attempt < maxRetries; attempt++) {
         try {

package/dist/tests/c8Run-8.10/hto-user-flows.spec.js

@@ -6,6 +6,11 @@ const zeebeClient_1 = require("../../utils/zeebeClient");
 const UtilitiesPage_1 = require("../../pages/c8Run-8.10/UtilitiesPage");
 const sleep_1 = require("../../utils/sleep");
 const _setup_1 = require("../../test-setup.js");
+// The key of the User_Task_Editing_Variables instance this run owns. Operate
+// and Tasklist both resolve that instance by key, so a second live instance of
+// the same process -- another worker, a --repeat-each copy, a leftover from an
+// earlier run -- can no longer be picked up by mistake.
+let variableEditingInstanceKey;
 c8Run_8_10_1.test.beforeAll(async () => {
     await Promise.all([
         (0, zeebeClient_1.deploy)('./resources/User_Task_Editing_Variables.bpmn'),
@@ -15,7 +20,7 @@ c8Run_8_10_1.test.beforeAll(async () => {
         (0, zeebeClient_1.deploy)('./resources/Start_Form_Process.bpmn'),
         (0, zeebeClient_1.deploy)('./resources/Zeebe_User_Task_Process_With_Priority.bpmn'),
     ]);
-    await Promise.all([
+    const [, variableEditingInstances] = await Promise.all([
         (0, zeebeClient_1.createInstances)('Zeebe_User_Task', 1, 1),
         (0, zeebeClient_1.createInstances)('User_Task_Editing_Variables', 1, 1, {
             testVariable: 'testValue',
@@ -24,6 +29,7 @@ c8Run_8_10_1.test.beforeAll(async () => {
         (0, zeebeClient_1.createInstances)('Start_Form_Process', 1, 1),
         (0, zeebeClient_1.createInstances)('Zeebe_User_Task_Process_With_Priority', 1, 1),
     ]);
+    variableEditingInstanceKey = String(variableEditingInstances[0].processInstanceKey);
     await (0, sleep_1.sleep)(5000);
 });
 c8Run_8_10_1.test.describe('HTO User Flow Tests', () => {
@@ -36,13 +42,13 @@ c8Run_8_10_1.test.describe('HTO User Flow Tests', () => {
         await (0, _setup_1.captureScreenshot)(page, testInfo);
         await (0, _setup_1.captureFailureVideo)(page, testInfo);
     });
-    (0, c8Run_8_10_1.test)('User Task Editing Variables Flow', async ({ page, operateHomePage, operateProcessesPage, operateProcessInstancePage, taskPanelPage, taskListLoginPage, operateLoginPage, taskDetailsPage, }) => {
+    (0, c8Run_8_10_1.test)('User Task Editing Variables Flow', async ({ page, request, operateHomePage, operateProcessesPage, operateProcessInstancePage, taskPanelPage, taskListLoginPage, operateLoginPage, taskDetailsPage, }) => {
         await c8Run_8_10_1.test.step('View Process Instance in Operate, Edit the Variable & Assert the Variable is Updated in Tasklist', async () => {
             await (0, UtilitiesPage_1.navigateToApp)(page, 'operate');
             await operateLoginPage.login('demo', 'demo');
             await (0, test_1.expect)(operateHomePage.processesTab).toBeVisible({ timeout: 180000 });
             await operateHomePage.clickProcessesTab();
-            await operateProcessesPage.clickProcessInstanceLink('User_Task_Editing_Variables');
+            await operateProcessesPage.clickProcessInstanceLink('User_Task_Editing_Variables', variableEditingInstanceKey);
             await operateProcessInstancePage.activeIconAssertion();
             await (0, test_1.expect)(page.getByText('"testValue"')).toBeVisible({
                 timeout: 120000,
@@ -63,7 +69,8 @@ c8Run_8_10_1.test.describe('HTO User Flow Tests', () => {
             await (0, sleep_1.sleep)(10000);
             await (0, UtilitiesPage_1.navigateToApp)(page, 'tasklist');
             await taskListLoginPage.login('demo', 'demo');
-            await taskPanelPage.openTask('User_Task_Editing_Variables');
+            await taskPanelPage.taskListBannerIsVisible();
+            await (0, UtilitiesPage_1.openUserTaskOfProcessInstance)(page, request, variableEditingInstanceKey);
             await taskDetailsPage.clickAssignToMeButton();
             await (0, test_1.expect)(page.getByText('Assigning...')).not.toBeVisible({
                 timeout: 90000,

package/package.json

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