mcdev 6.0.2 → 7.0.0

package/test/type.dataExtension.test.js

@@ -1,30 +1,34 @@
-import chai, { assert, expect } from 'chai';
+import * as chai from 'chai';
+const assert = chai.assert;
+const expect = chai.expect;
+
 import chaiFiles from 'chai-files';
 import cache from '../lib/util/cache.js';
 import * as testUtils from './utils.js';
 import handler from '../lib/index.js';
 chai.use(chaiFiles);
-const file = chaiFiles.file;
 
 describe('type: dataExtension', () => {
     beforeEach(() => {
         testUtils.mockSetup();
     });
+
     afterEach(() => {
         testUtils.mockReset();
     });
+
     describe('Retrieve ================', () => {
         it('Should retrieve a dataExtension', async () => {
             // WHEN
             await handler.retrieve('testInstance/testBU', ['dataExtension']);
             // THEN
-            assert.equal(process.exitCode, false, 'retrieve should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'retrieve should not have thrown an error');
             // get results from cache
             const result = cache.getCache();
             assert.equal(
                 result.dataExtension ? Object.keys(result.dataExtension).length : 0,
-                1,
-                'only one dataExtension expected'
+                5,
+                'only 5 dataExtension expected'
             );
             assert.deepEqual(
                 await testUtils.getActualJson('testExisting_dataExtension', 'dataExtension'),
@@ -34,9 +38,9 @@ describe('type: dataExtension', () => {
             );
             // check if MD file was created and equals expectations
             expect(
-                file(testUtils.getActualDoc('testExisting_dataExtension', 'dataExtension'))
+                await testUtils.getActualDoc('testExisting_dataExtension', 'dataExtension')
             ).to.equal(
-                file(testUtils.getExpectedFile('9999999', 'dataExtension', 'retrieve', 'md'))
+                await testUtils.getExpectedFile('9999999', 'dataExtension', 'retrieve', 'md')
             );
 
             assert.equal(
@@ -46,11 +50,12 @@ describe('type: dataExtension', () => {
             );
             return;
         });
+
         it('Should retrieve a shared dataExtension', async () => {
             // WHEN
             await handler.retrieve('testInstance/_ParentBU_', ['dataExtension']);
             // THEN
-            assert.equal(process.exitCode, false, 'retrieve should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'retrieve should not have thrown an error');
             // get results from cache
             const result = cache.getCache();
             assert.equal(
@@ -70,15 +75,13 @@ describe('type: dataExtension', () => {
             );
             // check if MD file was created and equals expectations
             expect(
-                file(
-                    testUtils.getActualDoc(
-                        'testExisting_dataExtensionShared',
-                        'dataExtension',
-                        '_ParentBU_'
-                    )
+                await testUtils.getActualDoc(
+                    'testExisting_dataExtensionShared',
+                    'dataExtension',
+                    '_ParentBU_'
                 )
             ).to.equal(
-                file(testUtils.getExpectedFile('1111111', 'dataExtension', 'retrieve', 'md'))
+                await testUtils.getExpectedFile('1111111', 'dataExtension', 'retrieve', 'md')
             );
 
             assert.equal(
@@ -89,15 +92,17 @@ describe('type: dataExtension', () => {
             return;
         });
     });
+
     describe('Deploy ================', () => {
         beforeEach(() => {
             testUtils.mockSetup(true);
         });
+
         it('Should create & update a dataExtension including a field rename', async () => {
             // WHEN
             const deployResult = await handler.deploy('testInstance/testBU', ['dataExtension']);
             // THEN
-            assert.equal(process.exitCode, false, 'deploy should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'deploy should not have thrown an error');
 
             assert.equal(
                 deployResult['testInstance/testBU']?.dataExtension
@@ -111,8 +116,8 @@ describe('type: dataExtension', () => {
             const result = cache.getCache();
             assert.equal(
                 result.dataExtension ? Object.keys(result.dataExtension).length : 0,
-                3,
-                'three dataExtensions expected'
+                7,
+                '7 dataExtensions expected'
             );
             // insert
             assert.deepEqual(
@@ -133,11 +138,12 @@ describe('type: dataExtension', () => {
             );
             return;
         });
+
         it('Should create & update a shared dataExtension', async () => {
             // WHEN
             const deployResult = await handler.deploy('testInstance/_ParentBU_', ['dataExtension']);
             // THEN
-            assert.equal(process.exitCode, false, 'deploy should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'deploy should not have thrown an error');
 
             assert.equal(
                 deployResult['testInstance/_ParentBU_']?.dataExtension
@@ -181,13 +187,14 @@ describe('type: dataExtension', () => {
             );
             return;
         });
+
         it('Should create & update a shared dataExtension with --fixShared & --skipInteraction', async () => {
             // WHEN
             handler.setOptions({ fixShared: 'testBU', skipInteraction: true, _runningTest: true });
 
             const deployResult = await handler.deploy('testInstance/_ParentBU_', ['dataExtension']);
             // THEN
-            assert.equal(process.exitCode, false, 'deploy should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'deploy should not have thrown an error');
 
             assert.equal(
                 deployResult['testInstance/_ParentBU_']?.dataExtension
@@ -232,6 +239,7 @@ describe('type: dataExtension', () => {
             return;
         });
     });
+
     describe('Templating ================', () => {
         it('Should create a dataExtension template via retrieveAsTemplate and build it', async () => {
             // GIVEN there is a template
@@ -241,11 +249,7 @@ describe('type: dataExtension', () => {
                 ['testExisting_dataExtension'],
                 'testSourceMarket'
             );
-            assert.equal(
-                process.exitCode,
-                false,
-                'retrieveAsTemplate should not have thrown an error'
-            );
+            assert.equal(process.exitCode, 0, 'retrieveAsTemplate should not have thrown an error');
 
             // WHEN
             assert.equal(
@@ -265,14 +269,10 @@ describe('type: dataExtension', () => {
             await handler.buildDefinition(
                 'testInstance/testBU',
                 'dataExtension',
-                'testExisting_dataExtension',
+                ['testExisting_dataExtension'],
                 'testTargetMarket'
             );
-            assert.equal(
-                process.exitCode,
-                false,
-                'buildDefinition should not have thrown an error'
-            );
+            assert.equal(process.exitCode, 0, 'buildDefinition should not have thrown an error');
             assert.deepEqual(
                 await testUtils.getActualDeployJson('testTemplated_dataExtension', 'dataExtension'),
                 await testUtils.getExpectedJson('9999999', 'dataExtension', 'build'),
@@ -285,6 +285,7 @@ describe('type: dataExtension', () => {
             );
             return;
         });
+
         it('Should create a dataExtension template via buildTemplate and build it', async () => {
             // download first before we test buildTemplate
             await handler.retrieve('testInstance/testBU', ['dataExtension']);
@@ -295,7 +296,7 @@ describe('type: dataExtension', () => {
                 ['testExisting_dataExtension'],
                 'testSourceMarket'
             );
-            assert.equal(process.exitCode, false, 'buildTemplate should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'buildTemplate should not have thrown an error');
             // WHEN
             assert.equal(
                 result.dataExtension ? Object.keys(result.dataExtension).length : 0,
@@ -314,14 +315,10 @@ describe('type: dataExtension', () => {
             await handler.buildDefinition(
                 'testInstance/testBU',
                 'dataExtension',
-                'testExisting_dataExtension',
+                ['testExisting_dataExtension'],
                 'testTargetMarket'
             );
-            assert.equal(
-                process.exitCode,
-                false,
-                'buildDefinition should not have thrown an error'
-            );
+            assert.equal(process.exitCode, 0, 'buildDefinition should not have thrown an error');
 
             assert.deepEqual(
                 await testUtils.getActualDeployJson('testTemplated_dataExtension', 'dataExtension'),
@@ -336,6 +333,7 @@ describe('type: dataExtension', () => {
             return;
         });
     });
+
     describe('Delete ================', () => {
         it('Should delete the dataExtension', async () => {
             // WHEN
@@ -345,11 +343,12 @@ describe('type: dataExtension', () => {
                 'testExisting_dataExtension'
             );
             // THEN
-            assert.equal(process.exitCode, false, 'delete should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'delete should not have thrown an error');
 
             assert.equal(isDeleted, true, 'should have deleted the item');
             return;
         });
+
         it('Should delete the dataExtensionField', async () => {
             // WHEN
             const isDeleted = await handler.deleteByKey(
@@ -358,12 +357,13 @@ describe('type: dataExtension', () => {
                 'testExisting_dataExtension.LastName'
             );
             // THEN
-            assert.equal(process.exitCode, false, 'delete should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'delete should not have thrown an error');
 
             assert.equal(isDeleted, true, 'should have deleted the item');
             return;
         });
     });
+
     describe('CI/CD ================', () => {
         it('Should return a list of files based on their type and key', async () => {
             // WHEN
@@ -373,11 +373,7 @@ describe('type: dataExtension', () => {
                 ['testExisting_dataExtension']
             );
             // THEN
-            assert.equal(
-                process.exitCode,
-                false,
-                'getFilesToCommit should not have thrown an error'
-            );
+            assert.equal(process.exitCode, 0, 'getFilesToCommit should not have thrown an error');
             assert.equal(fileList.length, 2, 'expected only 2 file paths (json, md)');
 
             assert.equal(

package/test/type.dataExtract.test.js

@@ -1,4 +1,6 @@
-import chai, { assert } from 'chai';
+import * as chai from 'chai';
+const assert = chai.assert;
+
 import chaiFiles from 'chai-files';
 import cache from '../lib/util/cache.js';
 import * as testUtils from './utils.js';
@@ -9,6 +11,7 @@ describe('type: dataExtract', () => {
     beforeEach(() => {
         testUtils.mockSetup();
     });
+
     afterEach(() => {
         testUtils.mockReset();
     });
@@ -18,7 +21,7 @@ describe('type: dataExtract', () => {
             // WHEN
             await handler.retrieve('testInstance/testBU', ['dataExtract']);
             // THEN
-            assert.equal(process.exitCode, false, 'retrieve should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'retrieve should not have thrown an error');
             // get results from cache
             const result = cache.getCache();
             assert.equal(
@@ -39,16 +42,18 @@ describe('type: dataExtract', () => {
             return;
         });
     });
+
     describe('Deploy ================', () => {
         beforeEach(() => {
             testUtils.mockSetup(true);
         });
+
         it('Should create & upsert a dataExtract', async () => {
             // WHEN
 
             await handler.deploy('testInstance/testBU', ['dataExtract']);
             // THEN
-            assert.equal(process.exitCode, false, 'deploy should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'deploy should not have thrown an error');
             // get results from cache
             const result = cache.getCache();
             assert.equal(
@@ -76,8 +81,10 @@ describe('type: dataExtract', () => {
             );
             return;
         });
+
         it('Should change the key during update via --changeKeyValue ');
     });
+
     describe('Templating ================', () => {
         it('Should create a dataExtract template via retrieveAsTemplate and build it', async () => {
             // buildTemplate
@@ -87,7 +94,7 @@ describe('type: dataExtract', () => {
                 ['testExisting_dataExtract'],
                 'testSourceMarket'
             );
-            assert.equal(process.exitCode, false, 'buildTemplate should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'buildTemplate should not have thrown an error');
             assert.equal(
                 result.dataExtract ? Object.keys(result.dataExtract).length : 0,
                 1,
@@ -102,14 +109,10 @@ describe('type: dataExtract', () => {
             await handler.buildDefinition(
                 'testInstance/testBU',
                 'dataExtract',
-                'testExisting_dataExtract',
+                ['testExisting_dataExtract'],
                 'testTargetMarket'
             );
-            assert.equal(
-                process.exitCode,
-                false,
-                'buildDefinition should not have thrown an error'
-            );
+            assert.equal(process.exitCode, 0, 'buildDefinition should not have thrown an error');
             assert.deepEqual(
                 await testUtils.getActualDeployJson('testTemplated_dataExtract', 'dataExtract'),
                 await testUtils.getExpectedJson('9999999', 'dataExtract', 'build'),
@@ -122,6 +125,7 @@ describe('type: dataExtract', () => {
             );
             return;
         });
+
         it('Should create a dataExtract template via buildTemplate and build it', async () => {
             // download first before we test buildTemplate
             await handler.retrieve('testInstance/testBU', ['dataExtract']);
@@ -132,7 +136,7 @@ describe('type: dataExtract', () => {
                 ['testExisting_dataExtract'],
                 'testSourceMarket'
             );
-            assert.equal(process.exitCode, false, 'buildTemplate should not have thrown an error');
+            assert.equal(process.exitCode, 0, 'buildTemplate should not have thrown an error');
             assert.equal(
                 result.dataExtract ? Object.keys(result.dataExtract).length : 0,
                 1,
@@ -147,14 +151,10 @@ describe('type: dataExtract', () => {
             await handler.buildDefinition(
                 'testInstance/testBU',
                 'dataExtract',
-                'testExisting_dataExtract',
+                ['testExisting_dataExtract'],
                 'testTargetMarket'
             );
-            assert.equal(
-                process.exitCode,
-                false,
-                'buildDefinition should not have thrown an error'
-            );
+            assert.equal(process.exitCode, 0, 'buildDefinition should not have thrown an error');
             assert.deepEqual(
                 await testUtils.getActualDeployJson('testTemplated_dataExtract', 'dataExtract'),
                 await testUtils.getExpectedJson('9999999', 'dataExtract', 'build'),
@@ -168,25 +168,18 @@ describe('type: dataExtract', () => {
             return;
         });
     });
+
     describe('Delete ================', () => {
-        it('Should NOT delete the item', async () => {
+        it('Should delete the item', async () => {
             // WHEN
             const isDeleted = await handler.deleteByKey(
                 'testInstance/testBU',
                 'dataExtract',
-                'testExisting_fileTranfer'
+                'testExisting_dataExtract'
             );
             // THEN
-            assert.equal(
-                process.exitCode,
-                1,
-                'deleteByKey should have thrown an error due to lack of support'
-            );
-            assert.equal(
-                isDeleted,
-                false,
-                'deleteByKey should have returned false due to lack of support'
-            );
+            assert.equal(process.exitCode, 0, 'deleteByKey should not have thrown an error');
+            assert.equal(isDeleted, true, 'deleteByKey should have returned true');
             return;
         });
     });

package/test/type.deliveryProfile.test.js

@@ -0,0 +1,45 @@
+import * as chai from 'chai';
+const assert = chai.assert;
+
+import chaiFiles from 'chai-files';
+import cache from '../lib/util/cache.js';
+import * as testUtils from './utils.js';
+import handler from '../lib/index.js';
+chai.use(chaiFiles);
+
+describe('type: deliveryProfile', () => {
+    beforeEach(() => {
+        testUtils.mockSetup();
+    });
+
+    afterEach(() => {
+        testUtils.mockReset();
+    });
+
+    describe('Retrieve ================', () => {
+        it('Should retrieve a deliveryProfile', async () => {
+            // WHEN
+            await handler.retrieve('testInstance/testBU', ['deliveryProfile']);
+            // THEN
+            assert.equal(process.exitCode, 0, 'retrieve should not have thrown an error');
+            // get results from cache
+            const result = cache.getCache();
+            assert.equal(
+                result.deliveryProfile ? Object.keys(result.deliveryProfile).length : 0,
+                1,
+                'only one deliveryProfile expected'
+            );
+            assert.deepEqual(
+                await testUtils.getActualJson('Default', 'deliveryProfile'),
+                await testUtils.getExpectedJson('9999999', 'deliveryProfile', 'get'),
+                'returned JSON was not equal expected'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                1,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+            return;
+        });
+    });
+});

package/test/type.emailSend.test.js

@@ -0,0 +1,144 @@
+import * as chai from 'chai';
+const assert = chai.assert;
+
+import chaiFiles from 'chai-files';
+import cache from '../lib/util/cache.js';
+import * as testUtils from './utils.js';
+import handler from '../lib/index.js';
+chai.use(chaiFiles);
+
+describe('type: emailSend', () => {
+    beforeEach(() => {
+        testUtils.mockSetup();
+    });
+
+    afterEach(() => {
+        testUtils.mockReset();
+    });
+
+    describe('Retrieve ================', () => {
+        it('Should retrieve a emailSend', async () => {
+            // WHEN
+            await handler.retrieve('testInstance/testBU', ['emailSend']);
+            // THEN
+            assert.equal(process.exitCode, 0, 'retrieve should not have thrown an error');
+            // get results from cache
+            const result = cache.getCache();
+            assert.equal(
+                result.emailSend ? Object.keys(result.emailSend).length : 0,
+                1,
+                'only 1 emailSend expected'
+            );
+            assert.deepEqual(
+                await testUtils.getActualJson('testExisting_emailSend', 'emailSend'),
+                await testUtils.getExpectedJson('9999999', 'emailSend', 'get'),
+                'returned JSON was not equal expected'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                16,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+            return;
+        });
+    });
+
+    describe('Deploy ================', () => {
+        beforeEach(() => {
+            testUtils.mockSetup(true);
+        });
+
+        it('Should create & upsert a emailSend', async () => {
+            // WHEN
+
+            await handler.deploy('testInstance/testBU', ['emailSend']);
+            // THEN
+            assert.equal(process.exitCode, 0, 'deploy should not have thrown an error');
+            // get results from cache
+            const result = cache.getCache();
+            assert.equal(
+                result.emailSend ? Object.keys(result.emailSend).length : 0,
+                2,
+                '2 emailSends expected'
+            );
+            // confirm created item
+            assert.deepEqual(
+                await testUtils.getActualJson('testNew_emailSend', 'emailSend'),
+                await testUtils.getExpectedJson('9999999', 'emailSend', 'post'),
+                'returned new-JSON was not equal expected for insert emailSend'
+            );
+            // confirm updated item
+            assert.deepEqual(
+                await testUtils.getActualJson('testExisting_emailSend', 'emailSend'),
+                await testUtils.getExpectedJson('9999999', 'emailSend', 'patch'),
+                'returned existing-JSON was not equal expected for update emailSend'
+            );
+            // check number of API calls
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                18,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+            return;
+        });
+    });
+
+    describe('Templating ================', () => {
+        it('Should create a emailSend template via buildTemplate and build it', async () => {
+            // download first before we test buildTemplate
+            await handler.retrieve('testInstance/testBU', ['emailSend']);
+            // buildTemplate
+            const result = await handler.buildTemplate(
+                'testInstance/testBU',
+                'emailSend',
+                ['testExisting_emailSend'],
+                'testSourceMarket'
+            );
+            assert.equal(process.exitCode, 0, 'buildTemplate should not have thrown an error');
+            assert.equal(
+                result.emailSend ? Object.keys(result.emailSend).length : 0,
+                1,
+                'only one emailSend expected'
+            );
+            assert.deepEqual(
+                await testUtils.getActualTemplateJson('testExisting_emailSend', 'emailSend'),
+                await testUtils.getExpectedJson('9999999', 'emailSend', 'template'),
+                'returned template JSON was not equal expected'
+            );
+            // buildDefinition
+            await handler.buildDefinition(
+                'testInstance/testBU',
+                'emailSend',
+                ['testExisting_emailSend'],
+                'testTargetMarket'
+            );
+            assert.equal(process.exitCode, 0, 'buildDefinition should not have thrown an error');
+            assert.deepEqual(
+                await testUtils.getActualDeployJson('testTemplated_emailSend', 'emailSend'),
+                await testUtils.getExpectedJson('9999999', 'emailSend', 'build'),
+                'returned deployment JSON was not equal expected'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                16,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+            return;
+        });
+    });
+
+    describe('Delete ================', () => {
+        it('Should delete the item', async () => {
+            // WHEN
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'emailSend',
+                'testExisting_emailSend'
+            );
+            // THEN
+            assert.equal(process.exitCode, 0, 'deleteByKey should not have thrown an error');
+            assert.equal(isDeleted, true, 'deleteByKey should have returned true');
+            return;
+        });
+    });
+});

package/test/type.event.test.js

@@ -0,0 +1,62 @@
+import * as chai from 'chai';
+const assert = chai.assert;
+// const expect = chai.expect;
+
+import chaiFiles from 'chai-files';
+// import cache from '../lib/util/cache.js';
+import * as testUtils from './utils.js';
+import handler from '../lib/index.js';
+chai.use(chaiFiles);
+// const file = chaiFiles.file;
+
+describe('type: event', () => {
+    beforeEach(() => {
+        testUtils.mockSetup();
+    });
+
+    afterEach(() => {
+        testUtils.mockReset();
+    });
+
+    describe('Retrieve ================', () => {});
+
+    describe('Deploy ================', () => {
+        beforeEach(() => {
+            testUtils.mockSetup(true);
+        });
+    });
+
+    describe('FixKeys ================', () => {
+        beforeEach(() => {
+            testUtils.mockSetup(true);
+        });
+
+        it('Should exit fixKeys because event is not supported intentionally', async () => {
+            // WHEN
+            handler.setOptions({ skipInteraction: { fixKeysReretrieve: false } });
+            const resultFixKeys = await handler.fixKeys('testInstance/testBU', ['event']);
+            // THEN
+            assert.equal(process.exitCode, 0, 'fixKeys should not have thrown an error');
+            // check which keys were fixed
+            assert.equal(
+                resultFixKeys['testInstance/testBU'],
+                undefined,
+                'expected to find no keys to be fixed'
+            );
+
+            // check number of API calls
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                0,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+            return;
+        });
+    });
+
+    describe('Templating ================', () => {});
+
+    describe('Delete ================', () => {});
+
+    describe('CI/CD ================', () => {});
+});