mcdev 5.2.0 → 6.0.0

package/test/type.journey.test.js

@@ -1,10 +1,9 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert } from 'chai';
+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 cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: journey', () => {
     beforeEach(() => {
@@ -141,17 +140,44 @@ describe('type: journey', () => {
         });
     });
     describe('Delete ================', () => {
-        // TODO: add this test
-        it('Should delete the item'); // , async () => {
-        //     // WHEN
-        //     const result = await handler.deleteByKey('testInstance/testBU', 'mobileKeyword', [
-        //         'testExisting_keyword',
-        //     ]);
-        //     // THEN
-        //     assert.equal(process.exitCode, false, 'delete should not have thrown an error');
+        it('Should NOT delete the item due to missing version', async () => {
+            // WHEN
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'journey',
+                'testExisting_interaction'
+            );
+            // THEN
+            assert.equal(process.exitCode, true, 'delete should have thrown an error');
 
-        //     assert.equal(result, true, 'should have deleted the item');
-        //     return;
-        // });
+            assert.equal(isDeleted, false, 'should not have deleted the item');
+            return;
+        });
+        it('Should NOT delete the item due to unknown version', async () => {
+            // WHEN
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'journey',
+                'testExisting_interaction/2'
+            );
+            // THEN
+            assert.equal(process.exitCode, true, 'delete should have thrown an error');
+
+            assert.equal(isDeleted, false, 'should not have deleted the item');
+            return;
+        });
+        it('Should delete the item with version', async () => {
+            // WHEN
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'journey',
+                'testExisting_interaction/1'
+            );
+            // THEN
+            assert.equal(process.exitCode, false, 'delete should not have thrown an error');
+
+            assert.equal(isDeleted, true, 'should have deleted the item');
+            return;
+        });
     });
 });

package/test/type.mobileKeyword.test.js

@@ -1,12 +1,10 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert, expect } from 'chai';
+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 expect = chai.expect;
 const file = chaiFiles.file;
-const cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: mobileKeyword', () => {
     beforeEach(() => {
@@ -245,13 +243,15 @@ describe('type: mobileKeyword', () => {
     describe('Delete ================', () => {
         it('Should delete the item', async () => {
             // WHEN
-            const result = await handler.deleteByKey('testInstance/testBU', 'mobileKeyword', [
-                '4912312345678.TESTEXISTING_KEYWORD',
-            ]);
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'mobileKeyword',
+                '4912312345678.TESTEXISTING_KEYWORD'
+            );
             // THEN
             assert.equal(process.exitCode, false, 'delete should not have thrown an error');
 
-            assert.equal(result, true, 'should have deleted the item');
+            assert.equal(isDeleted, true, 'should have deleted the item');
             return;
         });
     });

package/test/type.mobileMessage.test.js

@@ -1,12 +1,10 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert, expect } from 'chai';
+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 expect = chai.expect;
 const file = chaiFiles.file;
-const cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: mobileMessage', () => {
     beforeEach(() => {
@@ -163,13 +161,15 @@ describe('type: mobileMessage', () => {
     describe('Delete ================', () => {
         it('Should delete the item', async () => {
             // WHEN
-            const result = await handler.deleteByKey('testInstance/testBU', 'mobileMessage', [
-                'NTIzOjc4OjA',
-            ]);
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'mobileMessage',
+                'NTIzOjc4OjA'
+            );
             // THEN
             assert.equal(process.exitCode, false, 'delete should not have thrown an error');
 
-            assert.equal(result, true, 'should have deleted the item');
+            assert.equal(isDeleted, true, 'should have deleted the item');
             return;
         });
     });

package/test/type.query.test.js

@@ -1,13 +1,10 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert, expect } from 'chai';
+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 expect = chai.expect;
 const file = chaiFiles.file;
-// const dir = chaiFiles.dir;
-const cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: query', () => {
     beforeEach(() => {
@@ -458,7 +455,7 @@ describe('type: query', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                33,
+                34,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -498,7 +495,7 @@ describe('type: query', () => {
             // check number of API calls
             assert.equal(
                 testUtils.getAPIHistoryLength(),
-                35,
+                36,
                 'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
             );
             return;
@@ -656,13 +653,15 @@ describe('type: query', () => {
     describe('Delete ================', () => {
         it('Should delete the item', async () => {
             // WHEN
-            const result = await handler.deleteByKey('testInstance/testBU', 'query', [
-                'testExisting_query',
-            ]);
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'query',
+                'testExisting_query'
+            );
             // THEN
             assert.equal(process.exitCode, false, 'delete should not have thrown an error');
 
-            assert.equal(result, true, 'should have deleted the item');
+            assert.equal(isDeleted, true, 'should have deleted the item');
             return;
         });
     });

package/test/type.script.test.js

@@ -1,13 +1,10 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert, expect } from 'chai';
+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 expect = chai.expect;
 const file = chaiFiles.file;
-// const dir = chaiFiles.dir;
-const cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: script', () => {
     beforeEach(() => {
@@ -358,7 +355,12 @@ describe('type: script', () => {
             assert.equal(
                 fileList[1].split('\\').join('/'),
                 'retrieve/testInstance/testBU/script/testExisting_script.script-meta.ssjs',
-                'wrong JSON path'
+                'wrong SSJS path'
+            );
+            assert.equal(
+                fileList[2].split('\\').join('/'),
+                'retrieve/testInstance/testBU/script/testExisting_script.script-meta.html',
+                'wrong HTML path'
             );
             return;
         });

package/test/type.transactionalEmail.test.js

@@ -1,10 +1,9 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert } from 'chai';
+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 cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: transactionalEmail', () => {
     beforeEach(() => {
@@ -142,17 +141,18 @@ describe('type: transactionalEmail', () => {
         });
     });
     describe('Delete ================', () => {
-        // TODO: add this test
-        it('Should delete the item'); // , async () => {
-        //     // WHEN
-        //     const result = await handler.deleteByKey('testInstance/testBU', 'mobileKeyword', [
-        //         'testExisting_keyword',
-        //     ]);
-        //     // THEN
-        //     assert.equal(process.exitCode, false, 'delete should not have thrown an error');
+        it('Should delete the item', async () => {
+            // WHEN
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'transactionalEmail',
+                'testExisting_temail'
+            );
+            // THEN
+            assert.equal(process.exitCode, false, 'delete should not have thrown an error');
 
-        //     assert.equal(result, true, 'should have deleted the item');
-        //     return;
-        // });
+            assert.equal(isDeleted, true, 'should have deleted the item');
+            return;
+        });
     });
 });

package/test/type.transactionalPush.test.js

@@ -1,10 +1,9 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert } from 'chai';
+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 cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: transactionalPush', () => {
     beforeEach(() => {
@@ -146,13 +145,11 @@ describe('type: transactionalPush', () => {
         // TODO: add this test
         it('Should delete the item'); // , async () => {
         //     // WHEN
-        //     const result = await handler.deleteByKey('testInstance/testBU', 'mobileKeyword', [
-        //         'testExisting_keyword',
-        //     ]);
+        //     const isDeleted = await handler.deleteByKey('testInstance/testBU', 'mobileKeyword', 'testExisting_keyword');
         //     // THEN
         //     assert.equal(process.exitCode, false, 'delete should not have thrown an error');
 
-        //     assert.equal(result, true, 'should have deleted the item');
+        //     assert.equal(isDeleted, true, 'should have deleted the item');
         //     return;
         // });
     });

package/test/type.transactionalSMS.test.js

@@ -1,12 +1,10 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert, expect } from 'chai';
+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 expect = chai.expect;
 const file = chaiFiles.file;
-const cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: transactionalSMS', () => {
     beforeEach(() => {
@@ -176,13 +174,11 @@ describe('type: transactionalSMS', () => {
         // TODO: add this test
         it('Should delete the item'); // , async () => {
         //     // WHEN
-        //     const result = await handler.deleteByKey('testInstance/testBU', 'mobileKeyword', [
-        //         'testExisting_keyword',
-        //     ]);
+        //     const isDeleted = await handler.deleteByKey('testInstance/testBU', 'mobileKeyword', 'testExisting_keyword');
         //     // THEN
         //     assert.equal(process.exitCode, false, 'delete should not have thrown an error');
 
-        //     assert.equal(result, true, 'should have deleted the item');
+        //     assert.equal(isDeleted, true, 'should have deleted the item');
         //     return;
         // });
     });

package/test/type.triggeredSend.test.js

@@ -1,10 +1,9 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert } from 'chai';
+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 cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
 
 describe('type: triggeredSend', () => {
     beforeEach(() => {
@@ -130,13 +129,15 @@ describe('type: triggeredSend', () => {
     describe('Delete ================', () => {
         it('Should delete the item', async () => {
             // WHEN
-            const result = await handler.deleteByKey('testInstance/testBU', 'triggeredSend', [
-                'testExisting_triggeredSend',
-            ]);
+            const isDeleted = await handler.deleteByKey(
+                'testInstance/testBU',
+                'triggeredSend',
+                'testExisting_triggeredSend'
+            );
             // THEN
             assert.equal(process.exitCode, false, 'delete should not have thrown an error');
 
-            assert.equal(result, true, 'should have deleted the item');
+            assert.equal(isDeleted, true, 'should have deleted the item');
             return;
         });
     });

package/test/type.user.test.js

@@ -1,13 +1,11 @@
-const chai = require('chai');
-const chaiFiles = require('chai-files');
-const assert = chai.assert;
+import chai, { assert, expect } from 'chai';
+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 expect = chai.expect;
 const file = chaiFiles.file;
-const cache = require('../lib/util/cache');
-const testUtils = require('./utils');
-const handler = require('../lib/index');
-const File = require('../lib/util/file');
+import File from '../lib/util/file.js';
 
 describe('type: user', () => {
     beforeEach(() => {

package/test/type.verification.test.js

@@ -0,0 +1,172 @@
+import chai, { assert } from 'chai';
+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: verification', () => {
+    beforeEach(() => {
+        testUtils.mockSetup();
+    });
+    afterEach(() => {
+        testUtils.mockReset();
+    });
+
+    describe('Retrieve ================', () => {
+        it('Should retrieve a verification', async () => {
+            // WHEN
+            const retrieved = await handler.retrieve('testInstance/testBU', ['verification']);
+            // THEN
+            assert.equal(process.exitCode, false, 'retrieve should not have thrown an error');
+            // get results from cache
+            const result = cache.getCache();
+            assert.equal(
+                result.verification ? Object.keys(result.verification).length : 0,
+                1,
+                'only one verification expected'
+            );
+            assert.equal(
+                retrieved['testInstance/testBU']?.verification
+                    ? Object.keys(retrieved['testInstance/testBU']?.verification).length
+                    : 0,
+                1,
+                'one verifications to be retrieved'
+            );
+
+            assert.deepEqual(
+                await testUtils.getActualJson(
+                    'testExisting_39f6a488-20eb-4ba0-b0b9',
+                    'verification'
+                ),
+                await testUtils.getExpectedJson('9999999', 'verification', 'get'),
+                'returned JSON was not equal expected'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                9,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+            return;
+        });
+    });
+    describe('Deploy ================', () => {
+        beforeEach(() => {
+            testUtils.mockSetup(true);
+        });
+        it('Should create & upsert a verification', async () => {
+            // WHEN
+
+            const deployed = await handler.deploy('testInstance/testBU', ['verification']);
+            // THEN
+            assert.equal(process.exitCode, false, 'deploy should not have thrown an error');
+            // get results from cache
+            const result = cache.getCache();
+            assert.equal(
+                result.verification ? Object.keys(result.verification).length : 0,
+                2,
+                'two verifications expected'
+            );
+            assert.equal(
+                deployed['testInstance/testBU']?.verification
+                    ? Object.keys(deployed['testInstance/testBU']?.verification).length
+                    : 0,
+                2,
+                'two verifications to be deployed'
+            );
+            // confirm created item
+            assert.deepEqual(
+                await testUtils.getActualJson('testNew_RANDOM_NEW_GUID', 'verification'),
+                await testUtils.getExpectedJson('9999999', 'verification', 'post'),
+                'returned new-JSON was not equal expected for insert verification'
+            );
+            // confirm updated item
+            assert.deepEqual(
+                await testUtils.getActualJson(
+                    'testExisting_39f6a488-20eb-4ba0-b0b9',
+                    'verification'
+                ),
+                await testUtils.getExpectedJson('9999999', 'verification', 'patch'),
+                'returned existing-JSON was not equal expected for update verification'
+            );
+            // check number of API calls
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                11,
+                'Unexpected number of requests made. Run testUtils.logAPIHistoryDebug() to see the requests'
+            );
+            return;
+        });
+    });
+    describe('Templating ================', () => {
+        it('Should create a verification template via buildTemplate and build it', async () => {
+            // download first before we test buildTemplate
+            await handler.retrieve('testInstance/testBU', ['verification']);
+            // buildTemplate
+            const result = await handler.buildTemplate(
+                'testInstance/testBU',
+                'verification',
+                ['testExisting_39f6a488-20eb-4ba0-b0b9'],
+                'testSourceMarket'
+            );
+            assert.equal(process.exitCode, false, 'buildTemplate should not have thrown an error');
+            assert.equal(
+                result.verification ? Object.keys(result.verification).length : 0,
+                1,
+                'only one verification expected'
+            );
+            assert.deepEqual(
+                await testUtils.getActualTemplateJson(
+                    'testExisting_39f6a488-20eb-4ba0-b0b9',
+                    'verification'
+                ),
+                await testUtils.getExpectedJson('9999999', 'verification', 'template'),
+                'returned template JSON was not equal expected'
+            );
+            // buildDefinition
+            await handler.buildDefinition(
+                'testInstance/testBU',
+                'verification',
+                'testExisting_39f6a488-20eb-4ba0-b0b9',
+                'testTargetMarket'
+            );
+            assert.equal(
+                process.exitCode,
+                false,
+                'buildDefinition should not have thrown an error'
+            );
+            assert.deepEqual(
+                await testUtils.getActualDeployJson(
+                    'testTemplated_39f6a488-20eb-4ba0-b0b9',
+                    'verification'
+                ),
+                await testUtils.getExpectedJson('9999999', 'verification', 'build'),
+                'returned deployment JSON was not equal expected'
+            );
+            assert.equal(
+                testUtils.getAPIHistoryLength(),
+                9,
+                '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',
+                'verification',
+                'testExisting_39f6a488-20eb-4ba0-b0b9'
+            );
+            // THEN
+            assert.equal(
+                process.exitCode,
+                0,
+                'deleteByKey should have thrown an error due to lack of support'
+            );
+            assert.equal(isDeleted, true, 'deleteByKey should have returned true for success');
+            return;
+        });
+    });
+});