mcdev 7.0.4 → 7.1.1

package/test/type.automation.test.js

@@ -162,12 +162,6 @@ describe('type: automation', () => {
                 'five cached automation expected'
             );
 
-            // const scheduleCalloutList = testUtils.getSoapCallouts('Schedule');
-            // const objectIds = {
-            //     testNew_automation: 'a8afb0e2-b00a-4c88-ad2e-1f7f8788c560',
-            //     testExisting_automation: '08afb0e2-b00a-4c88-ad2e-1f7f8788c560',
-            // };
-
             assert.equal(
                 deployed['testInstance/testBU'].automation
                     ? Object.keys(deployed['testInstance/testBU'].automation).length
@@ -190,32 +184,6 @@ describe('type: automation', () => {
                 'expected specific automation to have been deployed'
             );
 
-            // insert
-            // assert.equal(
-            //     scheduleCalloutList.find((item) => item.includes(objectIds['testNew_automation'])),
-            //     await testUtils.getExpectedFile(
-            //         '9999999',
-            //         'automation',
-            //         'schedule-callout-new',
-            //         'xml'
-            //     ),
-            //     'sent metadata was not equal expected for create'
-            // );
-
-            // update
-            // assert.deepEqual(
-            //     scheduleCalloutList.find((item) =>
-            //         item.includes(objectIds['testExisting_automation'])
-            //     ),
-            //     await testUtils.getExpectedFile(
-            //         '9999999',
-            //         'automation',
-            //         'schedule-callout-existing',
-            //         'xml'
-            //     ),
-            //     'sent metadata was not equal expected for create'
-            // );
-
             assert.deepEqual(
                 await testUtils.getActualJson('testExisting_automation', 'automation'),
                 await testUtils.getExpectedJson('9999999', 'automation', 'update'),
@@ -696,6 +664,66 @@ describe('type: automation', () => {
             );
             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 = 77;
+            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;
+        });
     });
 
     describe('Delete ================', () => {
@@ -740,17 +768,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 +789,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 +806,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 +824,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 +845,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 +862,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 +877,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 +898,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 +916,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 +934,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 +955,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 +972,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.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;

package/test/type.event.test.js

@@ -338,6 +338,4 @@ describe('type: event', () => {
             return;
         });
     });
-
-    describe('CI/CD ================', () => {});
 });

package/test/type.importFile.test.js

@@ -26,8 +26,8 @@ describe('type: importFile', () => {
             const result = cache.getCache();
             assert.equal(
                 result.importFile ? Object.keys(result.importFile).length : 0,
-                2,
-                'only 2 importFile expected'
+                3,
+                'unexpected number of items retrieved'
             );
             assert.deepEqual(
                 await testUtils.getActualJson('testExisting_importFile', 'importFile'),
@@ -39,9 +39,14 @@ describe('type: importFile', () => {
                 await testUtils.getExpectedJson('9999999', 'importFile', 'get-sms'),
                 'returned JSON was not equal expected'
             );
+            assert.deepEqual(
+                await testUtils.getActualJson('testExisting_importFileDataImport', 'importFile'),
+                await testUtils.getExpectedJson('9999999', 'importFile', 'get-dataImport'),
+                'returned JSON was not equal expected'
+            );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                14,
+                15,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -92,8 +97,8 @@ describe('type: importFile', () => {
             const result = cache.getCache();
             assert.equal(
                 result.importFile ? Object.keys(result.importFile).length : 0,
-                3,
-                '3 importFiles expected'
+                4,
+                'unexptected number of importFiles in cache'
             );
             // confirm created item
             assert.deepEqual(
@@ -110,7 +115,7 @@ describe('type: importFile', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                16,
+                17,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -194,7 +199,7 @@ describe('type: importFile', () => {
             );
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                14,
+                15,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;