@camunda/e2e-test-suite 0.0.1173 → 0.0.1174

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

@@ -13,6 +13,8 @@ declare class ClusterSecretsPage {
     readonly fromFileButton: Locator;
     readonly bulkImportTextArea: Locator;
     readonly dialogImportButton: Locator;
+    readonly bulkImportCancelButton: Locator;
+    readonly bulkImportErrorBanner: Locator;
     readonly secretsSearchBox: Locator;
     readonly clusterBanner: Locator;
     readonly clusterLink: (name: string) => Locator;

package/dist/pages/8.10/ClusterSecretsPage.js

@@ -18,6 +18,8 @@ class ClusterSecretsPage {
     fromFileButton;
     bulkImportTextArea;
     dialogImportButton;
+    bulkImportCancelButton;
+    bulkImportErrorBanner;
     secretsSearchBox;
     clusterBanner;
     clusterLink;
@@ -50,6 +52,11 @@ class ClusterSecretsPage {
         this.dialogImportButton = this.page
             .getByRole('dialog')
             .getByRole('button', { name: 'Import', exact: true });
+        this.bulkImportCancelButton = this.dialog.getByRole('button', {
+            name: 'Cancel',
+            exact: true,
+        });
+        this.bulkImportErrorBanner = this.dialog.getByText(/Imported \d+ secrets?, \d+ failed/);
         this.secretsSearchBox = page.getByRole('searchbox', { name: 'Filter table' });
         this.clusterBanner = this.page
             .getByRole('banner')
@@ -210,9 +217,28 @@ class ClusterSecretsPage {
         await this.bulkImportTextArea.click();
         await this.bulkImportTextArea.fill(mapped);
         await this.dialogImportButton.click();
-        await (0, test_1.expect)(this.dialog).not.toBeVisible({
-            timeout: 60000,
-        });
+        // This suite intentionally reuses a fixed cluster across nightly runs
+        // (see ensureClusterViaApi in test-setup.spec.ts), so some or all of
+        // these secrets can already exist from a previous run. The Console then
+        // leaves the dialog open with an "Imported X secrets, Y failed" banner
+        // instead of closing it, which used to hang here for 60s and fail setup
+        // outright. Tolerate that banner and close the dialog ourselves --
+        // assertSecretsOrCreate() (the very next call in createSetOfSecrets)
+        // independently verifies every requested secret is present and creates
+        // any that are genuinely missing, so this doesn't mask a real failure,
+        // it just defers the correctness check to the step that already owns it.
+        try {
+            await (0, test_1.expect)(this.dialog).not.toBeVisible({ timeout: 60000 });
+        }
+        catch (error) {
+            if (await this.bulkImportErrorBanner.isVisible().catch(() => false)) {
+                await this.bulkImportCancelButton.click();
+                await (0, test_1.expect)(this.dialog).not.toBeVisible({ timeout: 10000 });
+            }
+            else {
+                throw error;
+            }
+        }
         await (0, test_1.expect)(this.createNewSecretButton).toBeVisible({
             timeout: 60000,
         });

package/dist/pages/8.10/FormJsPage.js

@@ -61,7 +61,14 @@ class FormJsPage {
             .locator('.cds--toast-notification--success')
             .filter({ hasText: 'Form deployed!' });
     }
-    async generateAIForm(request = 'Create a form with the following fields: 1. A text field with the label "Full Name" 2. A number with the label "Count" 3. A date input with the label "Date of birth"4. A Checkbox with the label "Agree"') {
+    async generateAIForm(
+    // "date only, no time": asked for a plain date the generator sometimes
+    // emits a datetime component in its "date and time" subtype, which adds a
+    // second required time subfield that then blocks task completion. The
+    // task form fills it anyway (TaskDetailsPage.fillTimeSubfield), but asking
+    // for the date-only variant keeps the generated form matching what the
+    // tests model.
+    request = 'Create a form with the following fields: 1. A text field with the label "Full Name" 2. A number with the label "Count" 3. A date input (date only, no time) with the label "Date of birth" 4. A Checkbox with the label "Agree"') {
         // The "Successfully generated smart form." toast is delivered over a Pusher
         // websocket. A transient websocket/network drop during generation means the
         // toast never renders even though generation finished, leaving the old

package/dist/pages/8.10/ModelerCreatePage.js

@@ -421,6 +421,20 @@ class ModelerCreatePage {
         if (processId.length > 0) {
             await this.clickStartEventElement();
             await this.clickGeneralPropertiesPanel();
+            // Renaming the file does not reliably rename the process: hub only syncs
+            // the new file name into the BPMN process name from inside
+            // `renameDiagram`, which returns early when the submitted name already
+            // equals the stored one (camunda-hub
+            // `stores/current-diagram-store.tsx` -> `checkAndSyncProcessName`). Any
+            // rename attempt that commits server-side but is not observed in time --
+            // the case `enterDiagramName`'s retry loop exists for -- therefore
+            // leaves the process named "New BPMN diagram" while the breadcrumb shows
+            // the new name, which is exactly what the nightly screenshot showed.
+            // Tasklist and Operate list the process *name*, so the task can then
+            // never be found by `processName`. Set it explicitly, the same way
+            // `UtilitiesPage` already does for uploaded diagrams.
+            await this.clickNameInput();
+            await this.fillNamedInput(processName);
             await this.clickIdInput();
             await this.fillIdInput(processId);
             await this.clickStartEventElement();

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

@@ -24,6 +24,7 @@ declare class TaskDetailsPage {
     readonly decrementButton: Locator;
     readonly dateInput: Locator;
     readonly timeInput: Locator;
+    readonly timeSubfield: Locator;
     readonly checkbox: Locator;
     readonly selectDropdown: Locator;
     readonly tagList: Locator;
@@ -55,6 +56,7 @@ declare class TaskDetailsPage {
     clickIncrementButton(): Promise<void>;
     clickDecrementButton(): Promise<void>;
     fillDate(date: string): Promise<void>;
+    private fillTimeSubfield;
     enterTime(time: string): Promise<void>;
     checkCheckbox(): Promise<void>;
     selectDropdownValue(value: string): Promise<void>;

package/dist/pages/8.10/TaskDetailsPage.js

@@ -44,6 +44,7 @@ class TaskDetailsPage {
     decrementButton;
     dateInput;
     timeInput;
+    timeSubfield;
     checkbox;
     selectDropdown;
     tagList;
@@ -91,6 +92,7 @@ class TaskDetailsPage {
         this.decrementButton = page.getByRole('button', { name: 'Decrement' });
         this.dateInput = page.getByLabel('Date of Birth');
         this.timeInput = page.getByPlaceholder('hh:mm ?m');
+        this.timeSubfield = this.form.locator('.fjs-timepicker input').first();
         this.checkbox = this.form.getByLabel('Agree');
         this.selectDropdown = this.form.getByText('Select').last();
         this.tagList = page.getByPlaceholder('Search');
@@ -174,6 +176,50 @@ class TaskDetailsPage {
         await this.dateInput.click();
         await this.dateInput.fill(date);
         await this.dateInput.press('Enter');
+        await this.fillTimeSubfield();
+    }
+    // The AI form generator is non-deterministic about how it renders the
+    // requested "Date of birth" input: it sometimes emits a form-js datetime
+    // component in its "date and time" subtype, which renders a second - and
+    // equally required - time subfield next to the date input. The nightly
+    // trace shows form validation rejecting the completion on exactly that
+    // empty subfield ("Field is required." / "Please review 1 field"), so the
+    // task was never completed and no completion confirmation could appear.
+    // Fill the time subfield whenever the rendered form has one; the fixed,
+    // date-only forms every other spec deploys have no timepicker and skip this.
+    async fillTimeSubfield() {
+        const hasTimeSubfield = await this.timeSubfield
+            .waitFor({ state: 'visible', timeout: 5000 })
+            .then(() => true)
+            .catch(() => false);
+        if (!hasTimeSubfield) {
+            return;
+        }
+        // Whether the component is configured for 12h or 24h notation is up to the
+        // generator, so prefer picking one of the times the timepicker itself
+        // offers over typing a notation that may not parse.
+        await this.timeSubfield.click({ timeout: 30000 });
+        const offeredTime = this.page.locator('.fjs-dropdownlist-item').first();
+        const timeIsOffered = await offeredTime
+            .waitFor({ state: 'visible', timeout: 10000 })
+            .then(() => true)
+            .catch(() => false);
+        if (timeIsOffered) {
+            await offeredTime.click({ timeout: 30000 });
+        }
+        if ((await this.timeSubfield.inputValue()) !== '') {
+            return;
+        }
+        // No dropdown, or it committed nothing: type instead. form-js discards a
+        // value it cannot parse on blur, so the input keeping the value is what
+        // tells us the notation was accepted.
+        for (const time of ['09:00 AM', '09:00']) {
+            await this.timeSubfield.fill(time);
+            await this.timeSubfield.press('Tab');
+            if ((await this.timeSubfield.inputValue()) !== '') {
+                return;
+            }
+        }
     }
     async enterTime(time) {
         await this.timeInput.click();

package/package.json

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