mcdev 7.0.3 → 7.1.0

package/test/type.automation.test.js

@@ -47,7 +47,7 @@ describe('type: automation', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                22,
+                21,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -133,7 +133,7 @@ describe('type: automation', () => {
 
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                29,
+                28,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -162,7 +162,7 @@ describe('type: automation', () => {
                 'five cached automation expected'
             );
 
-            // const scheduleCalloutList = testUtils.getSoapCallout('Schedule');
+            // const scheduleCalloutList = testUtils.getSoapCallouts('Schedule');
             // const objectIds = {
             //     testNew_automation: 'a8afb0e2-b00a-4c88-ad2e-1f7f8788c560',
             //     testExisting_automation: '08afb0e2-b00a-4c88-ad2e-1f7f8788c560',
@@ -233,7 +233,7 @@ describe('type: automation', () => {
 
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                37,
+                36,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -300,7 +300,7 @@ describe('type: automation', () => {
 
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                33,
+                32,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -339,7 +339,7 @@ describe('type: automation', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                22,
+                21,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -383,7 +383,7 @@ describe('type: automation', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                48,
+                46,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -427,7 +427,7 @@ describe('type: automation', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                50,
+                48,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -471,7 +471,7 @@ describe('type: automation', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                51,
+                49,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -515,7 +515,7 @@ describe('type: automation', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                43,
+                41,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -559,7 +559,7 @@ describe('type: automation', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                49,
+                47,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -603,7 +603,7 @@ describe('type: automation', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                51,
+                49,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -647,7 +647,7 @@ describe('type: automation', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                24,
+                23,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -691,7 +691,67 @@ describe('type: automation', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                22,
+                21,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+            return;
+        });
+
+        it('Should create a automation template via buildTemplate with --dependencies', async () => {
+            // download first before we test buildTemplate
+            await handler.retrieve('testInstance/testBU');
+            const expectedApiCallsRetrieve = 74;
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                expectedApiCallsRetrieve,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+
+            // preparation
+            // set skipInteraction to true to skip re-retrieving question
+            handler.setOptions({ dependencies: true, skipInteraction: true });
+
+            // GIVEN there is a template
+            const templateResult = await handler.buildTemplate(
+                'testInstance/testBU',
+                'automation',
+                ['testExisting_automation'],
+                'testSourceMarket'
+            );
+            assert.equal(process.exitCode, 0, 'buildTemplate should not have thrown an error');
+            // WHEN
+            // check type list
+            assert.deepEqual(
+                Object.keys(templateResult),
+                [
+                    'automation',
+                    'dataExtract',
+                    'emailSend',
+                    'dataExtension',
+                    'sendClassification',
+                    'senderProfile',
+                    'fileTransfer',
+                    'importFile',
+                    'query',
+                    'script',
+                    'verification',
+                ],
+                'did not create deployment packages for all relevant types'
+            );
+
+            assert.equal(
+                templateResult.automation ? Object.keys(templateResult.automation).length : 0,
+                1,
+                'only one automation expected'
+            );
+            assert.deepEqual(
+                await testUtils.getActualTemplateJson('testExisting_automation', 'automation'),
+                await testUtils.getExpectedJson('9999999', 'automation', 'template'),
+                'returned template was not equal expected'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength() - expectedApiCallsRetrieve,
+                4,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -740,17 +800,19 @@ describe('type: automation', () => {
 
     describe('Schedule ================', () => {
         it('Should schedule an automation by key', async () => {
-            const executedKeys = await handler.schedule('testInstance/testBU', 'automation', [
-                'testExisting_automation',
-            ]);
+            const scheduled = await handler.schedule(
+                'testInstance/testBU',
+                ['automation'],
+                ['testExisting_automation']
+            );
             assert.equal(process.exitCode, 0, 'execute should not have thrown an error');
             assert.equal(
-                executedKeys['testInstance/testBU']?.length,
+                scheduled['testInstance/testBU']?.automation?.length,
                 1,
                 'returned number of keys does not correspond to number of expected fixed keys'
             );
             assert.equal(
-                executedKeys['testInstance/testBU'][0],
+                scheduled['testInstance/testBU']?.automation[0],
                 'testExisting_automation',
                 'returned keys do not correspond to expected fixed keys'
             );
@@ -759,15 +821,15 @@ describe('type: automation', () => {
 
         it('Should schedule an automation selected via --like', async () => {
             handler.setOptions({ like: { key: 'testExist%automation' } });
-            const executedKeys = await handler.schedule('testInstance/testBU', 'automation');
+            const scheduled = await handler.schedule('testInstance/testBU', ['automation']);
             assert.equal(process.exitCode, 0, 'execute should not have thrown an error');
             assert.equal(
-                executedKeys['testInstance/testBU']?.length,
+                scheduled['testInstance/testBU']?.automation?.length,
                 1,
                 'returned number of keys does not correspond to number of expected fixed keys'
             );
             assert.equal(
-                executedKeys['testInstance/testBU'][0],
+                scheduled['testInstance/testBU']?.automation[0],
                 'testExisting_automation',
                 'returned keys do not correspond to expected fixed keys'
             );
@@ -776,12 +838,14 @@ describe('type: automation', () => {
 
         it('Should not schedule executing an automation because key and --like was specified', async () => {
             handler.setOptions({ like: { key: 'testExisting%' } });
-            const executedKeys = await handler.schedule('testInstance/testBU', 'automation', [
-                'testExisting_automation',
-            ]);
+            const scheduled = await handler.schedule(
+                'testInstance/testBU',
+                ['automation'],
+                ['testExisting_automation']
+            );
             assert.equal(process.exitCode, 1, 'execute should have thrown an error');
             assert.equal(
-                Object.keys(executedKeys).length,
+                Object.keys(scheduled).length,
                 0,
                 'automation was not supposed to be executed'
             );
@@ -792,17 +856,19 @@ describe('type: automation', () => {
     describe('Execute ================', () => {
         it('Should execute --schedule an automation by key', async () => {
             handler.setOptions({ schedule: true });
-            const executedKeys = await handler.execute('testInstance/testBU', 'automation', [
-                'testExisting_automation',
-            ]);
+            const executedKeys = await handler.execute(
+                'testInstance/testBU',
+                ['automation'],
+                ['testExisting_automation']
+            );
             assert.equal(process.exitCode, 0, 'execute should not have thrown an error');
             assert.equal(
-                executedKeys['testInstance/testBU']?.length,
+                executedKeys['testInstance/testBU']?.automation?.length,
                 1,
                 'returned number of keys does not correspond to number of expected fixed keys'
             );
             assert.equal(
-                executedKeys['testInstance/testBU'][0],
+                executedKeys['testInstance/testBU']?.automation[0],
                 'testExisting_automation',
                 'returned keys do not correspond to expected fixed keys'
             );
@@ -811,15 +877,15 @@ describe('type: automation', () => {
 
         it('Should execute --schedule an automation selected via --like', async () => {
             handler.setOptions({ like: { key: 'testExist%automation' }, schedule: true });
-            const executedKeys = await handler.execute('testInstance/testBU', 'automation');
+            const executedKeys = await handler.execute('testInstance/testBU', ['automation']);
             assert.equal(process.exitCode, 0, 'execute should not have thrown an error');
             assert.equal(
-                executedKeys['testInstance/testBU']?.length,
+                executedKeys['testInstance/testBU']?.automation?.length,
                 1,
                 'returned number of keys does not correspond to number of expected fixed keys'
             );
             assert.equal(
-                executedKeys['testInstance/testBU'][0],
+                executedKeys['testInstance/testBU']?.automation[0],
                 'testExisting_automation',
                 'returned keys do not correspond to expected fixed keys'
             );
@@ -828,9 +894,11 @@ describe('type: automation', () => {
 
         it('Should not execute --schedule executing an automation because key and --like was specified', async () => {
             handler.setOptions({ like: { key: 'testExisting%' }, schedule: true });
-            const executedKeys = await handler.execute('testInstance/testBU', 'automation', [
-                'testExisting_automation',
-            ]);
+            const executedKeys = await handler.execute(
+                'testInstance/testBU',
+                ['automation'],
+                ['testExisting_automation']
+            );
             assert.equal(process.exitCode, 1, 'execute should have thrown an error');
             assert.equal(
                 Object.keys(executedKeys).length,
@@ -841,17 +909,19 @@ describe('type: automation', () => {
         });
 
         it('Should runOnce an automation by key', async () => {
-            const executedKeys = await handler.execute('testInstance/testBU', 'automation', [
-                'testExisting_automation',
-            ]);
+            const executedKeys = await handler.execute(
+                'testInstance/testBU',
+                ['automation'],
+                ['testExisting_automation']
+            );
             assert.equal(process.exitCode, 0, 'execute should not have thrown an error');
             assert.equal(
-                executedKeys['testInstance/testBU']?.length,
+                executedKeys['testInstance/testBU']?.automation?.length,
                 1,
                 'automation was supposed to be executed'
             );
             assert.equal(
-                executedKeys['testInstance/testBU'][0],
+                executedKeys['testInstance/testBU']?.automation[0],
                 'testExisting_automation',
                 'returned keys do not correspond to expected fixed keys'
             );
@@ -860,15 +930,15 @@ describe('type: automation', () => {
 
         it('Should runOnce an automation selected via --like', async () => {
             handler.setOptions({ like: { key: 'testExist%automation' } });
-            const executedKeys = await handler.execute('testInstance/testBU', 'automation');
+            const executedKeys = await handler.execute('testInstance/testBU', ['automation']);
             assert.equal(process.exitCode, 0, 'execute should not have thrown an error');
             assert.equal(
-                executedKeys['testInstance/testBU']?.length,
+                executedKeys['testInstance/testBU']?.automation?.length,
                 1,
                 'automation was supposed to be executed'
             );
             assert.equal(
-                executedKeys['testInstance/testBU'][0],
+                executedKeys['testInstance/testBU']?.automation[0],
                 'testExisting_automation',
                 'returned keys do not correspond to expected fixed keys'
             );
@@ -878,9 +948,11 @@ describe('type: automation', () => {
 
         it('Should not runOnce executing an automation because key and --like was specified', async () => {
             handler.setOptions({ like: { key: 'testExisting%' } });
-            const executedKeys = await handler.execute('testInstance/testBU', 'automation', [
-                'testExisting_automation',
-            ]);
+            const executedKeys = await handler.execute(
+                'testInstance/testBU',
+                ['automation'],
+                ['testExisting_automation']
+            );
             assert.equal(process.exitCode, 1, 'execute should have thrown an error');
             assert.equal(
                 Object.keys(executedKeys).length,
@@ -894,17 +966,19 @@ describe('type: automation', () => {
 
     describe('Pause ================', () => {
         it('Should pause a automation by key', async () => {
-            const pausedKeys = await handler.pause('testInstance/testBU', 'automation', [
-                'testExisting_automation_pause',
-            ]);
+            const pausedKeys = await handler.pause(
+                'testInstance/testBU',
+                ['automation'],
+                ['testExisting_automation_pause']
+            );
             assert.equal(process.exitCode, 0, 'pause should not have thrown an error');
             assert.equal(
-                pausedKeys['testInstance/testBU']?.length,
+                pausedKeys['testInstance/testBU']?.automation?.length,
                 1,
                 'returned number of keys does not correspond to number of expected fixed keys'
             );
             assert.equal(
-                pausedKeys['testInstance/testBU'][0],
+                pausedKeys['testInstance/testBU']?.automation[0],
                 'testExisting_automation_pause',
                 'returned keys do not correspond to expected fixed keys'
             );
@@ -913,15 +987,15 @@ describe('type: automation', () => {
 
         it('Should pause a automation selected via --like', async () => {
             handler.setOptions({ like: { key: 'testExisting_a%n_pause' } });
-            const pausedKeys = await handler.pause('testInstance/testBU', 'automation');
+            const pausedKeys = await handler.pause('testInstance/testBU', ['automation']);
             assert.equal(process.exitCode, 0, 'pause should not have thrown an error');
             assert.equal(
-                pausedKeys['testInstance/testBU']?.length,
+                pausedKeys['testInstance/testBU']?.automation?.length,
                 1,
                 'returned number of keys does not correspond to number of expected fixed keys'
             );
             assert.equal(
-                pausedKeys['testInstance/testBU'][0],
+                pausedKeys['testInstance/testBU']?.automation[0],
                 'testExisting_automation_pause',
                 'returned keys do not correspond to expected fixed keys'
             );
@@ -930,9 +1004,11 @@ describe('type: automation', () => {
 
         it('Should not pause automation because key and --like was specified', async () => {
             handler.setOptions({ like: { key: 'testExisting_a%n_pause' } });
-            const pausedKeys = await handler.pause('testInstance/testBU', 'automation', [
-                'testExisting_automation_pause',
-            ]);
+            const pausedKeys = await handler.pause(
+                'testInstance/testBU',
+                ['automation'],
+                ['testExisting_automation_pause']
+            );
             assert.equal(process.exitCode, 1, 'pause should have thrown an error');
             assert.equal(
                 Object.keys(pausedKeys).length,

package/test/type.dataExtension.test.js

@@ -27,8 +27,8 @@ describe('type: dataExtension', () => {
             const result = cache.getCache();
             assert.equal(
                 result.dataExtension ? Object.keys(result.dataExtension).length : 0,
-                5,
-                'only 5 dataExtension expected'
+                7,
+                'only 7 dataExtension expected'
             );
             assert.deepEqual(
                 await testUtils.getActualJson('testExisting_dataExtension', 'dataExtension'),
@@ -116,8 +116,8 @@ describe('type: dataExtension', () => {
             const result = cache.getCache();
             assert.equal(
                 result.dataExtension ? Object.keys(result.dataExtension).length : 0,
-                7,
-                '7 dataExtensions expected'
+                9,
+                '9 dataExtensions expected'
             );
             // insert
             assert.deepEqual(

package/test/type.dataExtract.test.js

@@ -36,7 +36,7 @@ describe('type: dataExtract', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                6,
+                3,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -76,7 +76,7 @@ describe('type: dataExtract', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                8,
+                5,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -120,7 +120,7 @@ describe('type: dataExtract', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                6,
+                3,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -162,7 +162,7 @@ describe('type: dataExtract', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                6,
+                3,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;

package/test/type.emailSend.test.js

@@ -36,7 +36,7 @@ describe('type: emailSend', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                16,
+                14,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -76,7 +76,7 @@ describe('type: emailSend', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                18,
+                16,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -120,7 +120,7 @@ describe('type: emailSend', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                16,
+                14,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;