@camunda/e2e-test-suite 0.0.1170 → 0.0.1172

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

@@ -14,6 +14,8 @@ declare class TaskPanelPage {
     constructor(page: Page);
     openTask(name: string, assignButton?: boolean): Promise<void>;
     filterBy(option: 'All open' | 'Unassigned' | 'Assigned to me' | 'Completed'): Promise<void>;
+    private expandFilterPanel;
+    private static appliedFilterUrl;
     scrollToLastTask(name: string): Promise<void>;
     scrollToFirstTask(name: string): Promise<void>;
     clickCollapseFilter(): Promise<void>;

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

@@ -72,11 +72,66 @@ class TaskPanelPage {
         }
     }
     async filterBy(option) {
-        await this.filterOptions.click({ timeout: 45000 });
-        await (0, test_1.expect)(this.page.getByRole('link', { name: option })).toBeVisible({
-            timeout: 120000,
-        });
-        await this.page.getByRole('link', { name: option }).click({ timeout: 120000 });
+        // The task list refetches every 5s behind a suspense boundary, so a poll
+        // landing on the filter click tears the floating filter panel down before
+        // the router navigates: the panel closes but the filter never changes.
+        // Nothing downstream notices, so the test fails much later on an assertion
+        // that reads the still-unfiltered list -- which is how a dropped click on
+        // "Completed" surfaced as the completed-filter heading never appearing.
+        // Confirm the route actually carries the filter, and re-pick if it does not.
+        const maxAttempts = 3;
+        for (let attempt = 1; attempt <= maxAttempts; attempt++) {
+            await this.expandFilterPanel();
+            // Deliberately not `exact`: the panel spells the default filter
+            // "All open tasks" while callers pass "All open".
+            const filterLink = this.page.getByRole('link', { name: option });
+            // Only the first attempt waits out a slow first render. Later attempts
+            // re-pick from a panel that has already rendered once, so they stay
+            // short -- three attempts at the full budget would outlast the
+            // 12-minute test timeout and turn a dropped click into a timeout with
+            // no retry spent.
+            const renderTimeout = attempt === 1 ? 120000 : 15000;
+            await (0, test_1.expect)(filterLink).toBeVisible({ timeout: renderTimeout });
+            await filterLink.click({ timeout: renderTimeout });
+            const applied = await this.page
+                .waitForURL(TaskPanelPage.appliedFilterUrl(option), { timeout: 15000 })
+                .then(() => true)
+                .catch(() => false);
+            if (applied) {
+                return;
+            }
+            if (attempt < maxAttempts) {
+                console.warn(`Attempt ${attempt} did not apply the "${option}" filter. Retrying...`);
+            }
+        }
+        throw new Error(`Filter "${option}" was not applied after ${maxAttempts} attempts`);
+    }
+    // The filter panel renders either the collapsed rail (which carries the
+    // expand control) or the expanded nav -- never both -- and picking a filter
+    // leaves it expanded, so a retry must not click "expand" a second time.
+    async expandFilterPanel() {
+        const alreadyExpanded = await this.collapseFilter
+            .waitFor({ state: 'visible', timeout: 2000 })
+            .then(() => true)
+            .catch(() => false);
+        if (!alreadyExpanded) {
+            await this.filterOptions.click({ timeout: 45000 });
+        }
+    }
+    // Tasklist keeps the active filter in the `filter` search param. "All open"
+    // is the route's default and is stripped from the URL, so it is asserted as
+    // the absence of the param.
+    static appliedFilterUrl(option) {
+        const filterParam = {
+            'All open': undefined,
+            Unassigned: 'unassigned',
+            'Assigned to me': 'assigned-to-me',
+            Completed: 'completed',
+        };
+        const param = filterParam[option];
+        return param === undefined
+            ? /\/tasklist(?:\/[^?]*)?(?:\?(?!.*\bfilter=).*)?$/
+            : new RegExp(`[?&]filter=${param}(?:&|$)`);
     }
     async scrollToLastTask(name) {
         await this.page.getByText(name).last().scrollIntoViewIfNeeded();

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.1172",
   "description": "End-to-end test helpers for Camunda 8",
   "repository": {
     "type": "git",