@camunda/e2e-test-suite 0.0.1256 → 0.0.1258

package/dist/pages/BrowseAllConnectorMarketplacePage.js

@@ -66,25 +66,25 @@ class BrowseAllConnectorMarketplacePage {
                 continue;
             }
             await addButton.click({ timeout: 60000 });
-            if (await this.completeImportReviewIfNecessary(retryButton)) {
+            if (await this.completeImportReviewIfNecessary(retryButton, addedStatus)) {
                 break;
             }
             if (attempt === 3) {
                 throw new Error('Browse All could not import the Marketplace connector after 3 attempts.');
             }
         }
-        await this.availableElementsHeading.waitFor({
-            state: 'visible',
-            timeout: 120000,
-        });
+        await this.availableElementsHeading
+            .or(addedStatus)
+            .waitFor({ state: 'visible', timeout: 120000 });
         await browseAllDialog
             .getByRole('button', { name: 'Close' })
             .click({ timeout: 60000 });
         await browseAllDialog.waitFor({ state: 'hidden', timeout: 60000 });
     }
-    async completeImportReviewIfNecessary(retryButton) {
+    async completeImportReviewIfNecessary(retryButton, addedStatus) {
         await this.importModal
             .or(this.availableElementsHeading)
+            .or(addedStatus)
             .or(retryButton)
             .waitFor({ state: 'visible', timeout: 120000 });
         if (await retryButton.isVisible()) {
@@ -107,6 +107,7 @@ class BrowseAllConnectorMarketplacePage {
         }
         await this.importModal.waitFor({ state: 'hidden', timeout: 120000 });
         await this.availableElementsHeading
+            .or(addedStatus)
             .or(retryButton)
             .waitFor({ state: 'visible', timeout: 120000 });
         return !(await retryButton.isVisible());

package/dist/tests/api/v2/deployment/deployment-api-tests.spec.js

@@ -6,6 +6,7 @@ const assertions_1 = require("../../../../utils/api/assertions");
 const deployments_1 = require("../../../../utils/api/deployments");
 const bpmn_1 = require("../../../../utils/api/bpmn");
 const eventually_1 = require("../../../../utils/api/eventually");
+const config_1 = require("../../../../utils/api/config");
 const capabilities_1 = require("../../../../utils/api/capabilities");
 const keys_1 = require("../../../../utils/api/keys");
 api_1.test.describe('Deployments', () => {
@@ -124,11 +125,16 @@ api_1.test.describe('Resource deletion', () => {
             body: {},
             tenant: 'none',
         });
-        // The endpoint answers 200 with a `DeleteResourceResponse` body naming
-        // the deleted resource (and the batch operation that clears its history),
-        // not a bare 204.
-        const deletion = (await (0, assertions_1.assertOk)(response));
-        (0, test_1.expect)(String(deletion.resourceKey)).toBe(key);
+        // From 8.9 the endpoint answers 200 with a `DeleteResourceResponse` body
+        // naming the deleted resource (and the batch operation that clears its
+        // history). 8.8 has no response body and answers a bare 204.
+        if ((0, config_1.atLeastVersion)('8.9')) {
+            const deletion = (await (0, assertions_1.assertOk)(response));
+            (0, test_1.expect)(String(deletion.resourceKey)).toBe(key);
+        }
+        else {
+            await (0, assertions_1.assertNoContent)(response);
+        }
         // The test is the deletion, so cleanup has nothing left to do.
         resources.forget('resource', key);
         await (0, eventually_1.eventually)(async () => {

package/dist/tests/api/v2/identity/group-api-tests.spec.js

@@ -135,7 +135,9 @@ api_1.test.describe('Groups', () => {
     (0, api_1.test)('rejects updating an absent group', async ({ api }) => {
         const response = await api.put('/groups/{groupId}', {
             params: { groupId: (0, bpmn_1.uniqueId)('absent-group') },
-            body: { name: 'does not exist' },
+            // 8.8 requires `description` on update and rejects the body with 400
+            // before the group is looked up.
+            body: { name: 'does not exist', description: 'does not exist' },
             tenant: 'none',
         });
         await (0, assertions_1.assertNotFound)(response);

package/dist/tests/api/v2/process-instance/process-instance-api-tests.spec.js

@@ -137,7 +137,10 @@ api_1.test.describe('Process instances', () => {
         const key = instance.processInstanceKey;
         resources.track('processInstance', key);
         await (0, eventually_1.eventually)(async () => {
-            const response = await api.post('/process-instances/{processInstanceKey}/incidents/search', { params: { processInstanceKey: key }, body: { filter: {} } });
+            const response = await api.post('/process-instances/{processInstanceKey}/incidents/search', 
+            // 8.8's per-instance incident query has no `filter` property and
+            // rejects one with 400, so none is sent.
+            { params: { processInstanceKey: key }, body: {} });
             await (0, assertions_1.assertPaginated)(response, { exactItems: 0 });
         });
     });

package/package.json

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