mcdev 7.10.1 → 8.0.0

package/test/type.journey.test.js

@@ -475,6 +475,7 @@ describe('type: journey', () => {
         it('Should create a journey template via buildTemplate with --dependencies', async () => {
             // download first before we test buildTemplate
             await handler.retrieve('testInstance/testBU', ['journey', 'asset']);
+            assert.equal(process.exitCode, 0, 'retrieve should not have thrown an error');
 
             handler.setOptions({ dependencies: true, retrieve: true });
 
@@ -1290,7 +1291,7 @@ describe('type: journey', () => {
 
             assert.deepEqual(
                 replace['testInstance/testBU'].journey,
-                ['testExisting_journey_Multistep', 'testExisting_temail'],
+                ['testExisting_temail', 'testExisting_journey_Multistep'],
                 'should have found the right journeys that need updating'
             );
 
@@ -1302,4 +1303,132 @@ describe('type: journey', () => {
             return;
         });
     });
+
+    describe('Audit ================', () => {
+        it('Should show audit log of a transactional journey and version', async () => {
+            const audit = await handler.audit('testInstance/testBU', {
+                journey: ['testExisting_temail/2'],
+            });
+            // THEN
+            assert.equal(process.exitCode, 0, 'audit should not have thrown an error');
+
+            assert.deepEqual(
+                audit['testInstance/testBU'].journey,
+                ['testExisting_temail'],
+                'should have returned the right journeys'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                2,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+
+            return;
+        });
+
+        it('Should show audit log of a transactional journey with all its versions', async () => {
+            const audit = await handler.audit('testInstance/testBU', {
+                journey: ['testExisting_temail'],
+            });
+            // THEN
+            assert.equal(process.exitCode, 0, 'audit should not have thrown an error');
+
+            assert.deepEqual(
+                audit['testInstance/testBU'].journey,
+                ['testExisting_temail'],
+                'should have returned the right journeys'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                3,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+
+            return;
+        });
+
+        it('Should not show audit log of a transactional journey with a too high version', async () => {
+            const audit = await handler.audit('testInstance/testBU', {
+                journey: ['testExisting_temail/99'],
+            });
+            // THEN
+            assert.equal(process.exitCode, 404, 'audit should have thrown an error');
+
+            assert.deepEqual(
+                audit['testInstance/testBU'].journey,
+                [],
+                'should have returned the right journeys'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                2,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+
+            return;
+        });
+
+        it('Should show audit log of a multi-step journey and version', async () => {
+            const audit = await handler.audit('testInstance/testBU', {
+                journey: ['testExisting_journey_Multistep/1'],
+            });
+            // THEN
+            assert.equal(process.exitCode, 0, 'audit should not have thrown an error');
+
+            assert.deepEqual(
+                audit['testInstance/testBU'].journey,
+                ['testExisting_journey_Multistep'],
+                'should have returned the right journeys'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                2,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+
+            return;
+        });
+
+        it('Should show audit log of a multi-step journey with all its versions', async () => {
+            const audit = await handler.audit('testInstance/testBU', {
+                journey: ['testExisting_journey_Multistep'],
+            });
+            // THEN
+            assert.equal(process.exitCode, 0, 'audit should not have thrown an error');
+
+            assert.deepEqual(
+                audit['testInstance/testBU'].journey,
+                ['testExisting_journey_Multistep'],
+                'should have returned the right journeys'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                4,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+
+            return;
+        });
+
+        it('Should show audit log of a multi-step journey with all its versions via /*', async () => {
+            const audit = await handler.audit('testInstance/testBU', {
+                journey: ['testExisting_journey_Multistep'],
+            });
+            // THEN
+            assert.equal(process.exitCode, 0, 'audit should not have thrown an error');
+
+            assert.deepEqual(
+                audit['testInstance/testBU'].journey,
+                ['testExisting_journey_Multistep'],
+                'should have returned the right journeys'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                4,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+
+            return;
+        });
+    });
 });

package/test/type.script.test.js

@@ -373,6 +373,7 @@ describe('type: script', () => {
         it('Should create a script template via buildTemplate with --dependencies', async () => {
             // download first before we test buildTemplate
             await handler.retrieve('testInstance/testBU', ['script', 'asset']);
+            assert.equal(process.exitCode, 0, 'retrieve should not have thrown an error');
 
             handler.setOptions({ dependencies: true, skipInteraction: true });
 

package/test/type.senderProfile.test.js

@@ -152,6 +152,7 @@ describe('type: senderProfile', () => {
         it('Should create a senderProfile template via buildTemplate with --dependencies', async () => {
             // download first before we test buildTemplate
             handler.setOptions({ dependencies: true, retrieve: true });
+            assert.equal(process.exitCode, 0, 'retrieve should not have thrown an error');
 
             // GIVEN there is a template
             const templatedItems = await handler.buildTemplate(

package/test/utils.js

@@ -205,6 +205,8 @@ export function mockSetup(isDeploy) {
 
         handler.setOptions(resetOptions);
     }
+    File.prettierConfig = null;
+
     apimock = new MockAdapter(axiosInstance, { onNoMatch: 'throwException' });
     // set access_token to mid to allow for autorouting of mock to correct resources
     apimock.onPost(authResources.success.url).reply((config) => {

package/types/mcdev.d.js

@@ -243,8 +243,12 @@
  * @property {string} startDate example: '2021-05-07T09:00:00'
  * @property {string} endDate example: '2021-05-07T09:00:00'
  * @property {string} icalRecur example: 'FREQ=DAILY;UNTIL=20790606T160000;INTERVAL=1'
+ * @property {string} [iCalRecur] same as icalRecur but returned by legacy-API; example: 'FREQ=DAILY;UNTIL=20790606T160000;INTERVAL=1'
  * @property {string} timezoneName example: 'W. Europe Standard Time'; see this.definition.timeZoneMapping
+ * @property {string} [timeZone] same as timezoneName but returned by legacy-API; example: 'W. Europe Standard Time'; see this.definition.timeZoneMapping
+ * @property {string} [description] kept in legacy API only, exact description of what this schedule does
  * @property {number} [timezoneId] see this.definition.timeZoneMapping
+ * @property {number} [timeZoneId] same as timezoneId but returned by legacy-API; see this.definition.timeZoneMapping
  * @property {number} [rangeTypeId] ?
  * @property {any} [pattern] ?
  * @property {any} [scheduledTime] ?

package/test/resources/9999999/asset/v1/content/assets/query/post-response-assetType.idIN219,220,221,222,223,224,225,226,227,228,230,232.json

@@ -1,32 +0,0 @@
-{
-    "count": 1,
-    "page": 1,
-    "pageSize": 50,
-    "links": {},
-    "items": [
-        {
-            "id": 1209971,
-            "customerKey": "mobileMessage_test",
-            "assetType": { "id": 230, "name": "jsonmessage", "displayName": "JSON Message" },
-            "name": "mobileMessage_test",
-            "createdDate": "2022-12-07T02:49:36.857-06:00",
-            "createdBy": {
-                "id": 700301950,
-                "email": "joern.berkefeld@accenture.com",
-                "name": "J├Ârn Berkefeld (ASGR)",
-                "userId": "700301950"
-            },
-            "modifiedDate": "2022-12-07T02:49:57.44-06:00",
-            "modifiedBy": {
-                "id": 700301950,
-                "email": "joern.berkefeld@accenture.com",
-                "name": "J├Ârn Berkefeld (ASGR)",
-                "userId": "700301950"
-            },
-            "status": { "id": 1, "name": "Draft" },
-            "category": { "id": 89397, "name": "Content Builder", "parentId": 0 },
-            "availableViews": ["push"],
-            "modelVersion": 2
-        }
-    ]
-}