@capillarytech/creatives-library 8.0.201-alpha.0 → 8.0.201

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@capillarytech/creatives-library",
   "author": "meharaj",
-  "version": "8.0.201-alpha.0",
+  "version": "8.0.201",
   "description": "Capillary creatives ui",
   "main": "./index.js",
   "module": "./index.es.js",

package/v2Containers/CreativesContainer/tests/SlideBoxContent.test.js

@@ -5,6 +5,7 @@ import { SlideBoxContent } from '../SlideBoxContent';
 import mockdata from '../../mockdata';
 import { templateDetailsImage, templateDetailsVideo, templateDetailsText, templateDetails_ } from '../../Viber/tests/mockData';
 import { CREATE, RICH_MEDIA } from '../../WeChat/Wrapper/constants';
+import * as commonUtils from '../../../utils/common';
 
 const {
   whatsappEditTemplateData,
@@ -50,6 +51,11 @@ describe('Test SlideBoxContent container', () => {
     global.Date = originalDate;
   });
 
+  afterEach(() => {
+    // Clean up all mocks after each test
+    jest.restoreAllMocks();
+  });
+
   const renderFunction = (channel, mode, templateData, additionalProps = {}) => {
     renderedComponent = shallowWithIntl(
       <SlideBoxContent
@@ -704,9 +710,9 @@ describe('Test SlideBoxContent container', () => {
   // Mobile Push Loyalty Module Tests
   describe('Mobile Push with Loyalty Module', () => {
     it('Should render MobliPushEdit for loyalty module in library mode (edit)', () => {
-      renderFunction('mobilepush', 'editTemplate', { 
+      renderFunction('MOBILEPUSH', 'editTemplate', { 
         _id: 'test-id', 
-        type: 'mobilepush' // Use lowercase to match constants.MOBILE_PUSH
+        type: 'MOBILEPUSH' // Use uppercase to match constants.MOBILE_PUSH
       }, { 
         isFullMode: false, 
         isLoyaltyModule: true 
@@ -715,9 +721,9 @@ describe('Test SlideBoxContent container', () => {
     });
 
     it('Should render MobilePushNew for loyalty module in full mode (edit)', () => {
-      renderFunction('mobilepush', 'editTemplate', { 
+      renderFunction('MOBILEPUSH', 'editTemplate', { 
         _id: 'test-id', 
-        type: 'mobilepush' // Use lowercase to match constants.MOBILE_PUSH
+        type: 'MOBILEPUSH' // Use uppercase to match constants.MOBILE_PUSH
       }, { 
         isFullMode: true, 
         isLoyaltyModule: true 
@@ -726,9 +732,9 @@ describe('Test SlideBoxContent container', () => {
     });
 
     it('Should render MobilepushWrapper for loyalty module in library mode (create)', () => {
-      renderFunction('mobilepush', 'createTemplate', { 
+      renderFunction('MOBILEPUSH', 'createTemplate', { 
         mode: 'create',
-        type: 'mobilepush' // Use lowercase to match constants.MOBILE_PUSH
+        type: 'MOBILEPUSH' // Use uppercase to match constants.MOBILE_PUSH
       }, { 
         isFullMode: false, 
         isLoyaltyModule: true 
@@ -737,9 +743,9 @@ describe('Test SlideBoxContent container', () => {
     });
 
     it('Should render MobilePushNew for loyalty module in full mode (create)', () => {
-      renderFunction('mobilepush', 'createTemplate', { 
+      renderFunction('MOBILEPUSH', 'createTemplate', { 
         mode: 'create',
-        type: 'mobilepush' // Use lowercase to match constants.MOBILE_PUSH
+        type: 'MOBILEPUSH' // Use uppercase to match constants.MOBILE_PUSH
       }, { 
         isFullMode: true, 
         isLoyaltyModule: true 
@@ -748,9 +754,9 @@ describe('Test SlideBoxContent container', () => {
     });
 
     it('Should render MobilePushNew for non-loyalty module in library mode (edit)', () => {
-      renderFunction('mobilepush', 'editTemplate', { 
+      renderFunction('MOBILEPUSH', 'editTemplate', { 
         _id: 'test-id', 
-        type: 'mobilepush' // Use lowercase to match constants.MOBILE_PUSH
+        type: 'MOBILEPUSH' // Use uppercase to match constants.MOBILE_PUSH
       }, { 
         isFullMode: false, 
         isLoyaltyModule: false 
@@ -759,15 +765,37 @@ describe('Test SlideBoxContent container', () => {
     });
 
     it('Should render MobilePushNew for non-loyalty module in library mode (create)', () => {
-      renderFunction('mobilepush', 'createTemplate', { 
+      renderFunction('MOBILEPUSH', 'createTemplate', { 
         mode: 'create',
-        type: 'mobilepush' // Use lowercase to match constants.MOBILE_PUSH
+        type: 'MOBILEPUSH' // Use uppercase to match constants.MOBILE_PUSH
       }, { 
         isFullMode: false, 
         isLoyaltyModule: false 
       });
       expect(renderedComponent).toMatchSnapshot();
     });
+
+    it('Should render MobilePushNew for non-loyalty module in full mode (edit)', () => {
+      renderFunction('MOBILEPUSH', 'editTemplate', { 
+        _id: 'test-id', 
+        type: 'MOBILEPUSH' // Use uppercase to match constants.MOBILE_PUSH
+      }, { 
+        isFullMode: true, 
+        isLoyaltyModule: false 
+      });
+      expect(renderedComponent).toMatchSnapshot();
+    });
+
+    it('Should render MobilePushNew for non-loyalty module in full mode (create)', () => {
+      renderFunction('MOBILEPUSH', 'createTemplate', { 
+        mode: 'create',
+        type: 'MOBILEPUSH' // Use uppercase to match constants.MOBILE_PUSH
+      }, { 
+        isFullMode: true, 
+        isLoyaltyModule: false 
+      });
+      expect(renderedComponent).toMatchSnapshot();
+    });
   });
 
   // Additional edge case tests to cover uncovered lines
@@ -790,9 +818,7 @@ describe('Test SlideBoxContent container', () => {
       // This should cover line 251 in SMS content processing
       // Need isDltEnabled = true and updatedSmsEditor as array
       // Mock hasTraiDltFeature to return true
-      const commonUtils = require('../../../utils/common');
-      const originalHasTraiDltFeature = commonUtils.hasTraiDltFeature;
-      commonUtils.hasTraiDltFeature = jest.fn(() => true);
+      jest.spyOn(commonUtils, 'hasTraiDltFeature').mockReturnValue(true);
       
       const smsTemplateData = {
         type: 'SMS',
@@ -808,8 +834,6 @@ describe('Test SlideBoxContent container', () => {
         isFullMode: false
       });
       
-      // Restore original function
-      commonUtils.hasTraiDltFeature = originalHasTraiDltFeature;
       expect(renderedComponent).toMatchSnapshot();
     });
 

package/v2Containers/CreativesContainer/tests/__snapshots__/SlideBoxContent.test.js.snap

@@ -683,17 +683,251 @@ exports[`Test SlideBoxContent container Email component isTestAndPreviewMode IIF
 </SlideBoxContent__CreativesWrapper>
 `;
 
-exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for loyalty module in full mode (create) 1`] = `<SlideBoxContent__CreativesWrapper />`;
+exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for loyalty module in full mode (create) 1`] = `
+<SlideBoxContent__CreativesWrapper>
+  <ForwardRef
+    date={0}
+    getDefaultTags=""
+    isFullMode={true}
+    key="creatives-mobilepush-wrapper"
+    onCreateComplete={[MockFunction]}
+    query={
+      Object {
+        "isEditFromCampaigns": undefined,
+        "module": "default",
+        "type": false,
+      }
+    }
+    templateData={
+      Object {
+        "mode": "create",
+        "type": "MOBILEPUSH",
+      }
+    }
+    type=""
+  />
+</SlideBoxContent__CreativesWrapper>
+`;
 
-exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for loyalty module in full mode (edit) 1`] = `<SlideBoxContent__CreativesWrapper />`;
+exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for loyalty module in full mode (edit) 1`] = `
+<SlideBoxContent__CreativesWrapper>
+  <ForwardRef
+    getDefaultTags=""
+    isFullMode={true}
+    location={
+      Object {
+        "pathname": "/mobilepush/edit/",
+        "query": Object {
+          "isEditFromCampaigns": undefined,
+          "module": "default",
+          "type": false,
+        },
+      }
+    }
+    params={
+      Object {
+        "id": "test-id",
+      }
+    }
+    route={
+      Object {
+        "name": "mobilepush",
+      }
+    }
+    templateData={
+      Object {
+        "_id": "test-id",
+        "type": "MOBILEPUSH",
+      }
+    }
+    type=""
+  />
+</SlideBoxContent__CreativesWrapper>
+`;
 
-exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for non-loyalty module in library mode (create) 1`] = `<SlideBoxContent__CreativesWrapper />`;
+exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for non-loyalty module in full mode (create) 1`] = `
+<SlideBoxContent__CreativesWrapper>
+  <ForwardRef
+    date={0}
+    getDefaultTags=""
+    isFullMode={true}
+    key="creatives-mobilepush-wrapper"
+    onCreateComplete={[MockFunction]}
+    query={
+      Object {
+        "isEditFromCampaigns": undefined,
+        "module": "default",
+        "type": false,
+      }
+    }
+    templateData={
+      Object {
+        "mode": "create",
+        "type": "MOBILEPUSH",
+      }
+    }
+    type=""
+  />
+</SlideBoxContent__CreativesWrapper>
+`;
 
-exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for non-loyalty module in library mode (edit) 1`] = `<SlideBoxContent__CreativesWrapper />`;
+exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for non-loyalty module in full mode (edit) 1`] = `
+<SlideBoxContent__CreativesWrapper>
+  <ForwardRef
+    getDefaultTags=""
+    isFullMode={true}
+    location={
+      Object {
+        "pathname": "/mobilepush/edit/",
+        "query": Object {
+          "isEditFromCampaigns": undefined,
+          "module": "default",
+          "type": false,
+        },
+      }
+    }
+    params={
+      Object {
+        "id": "test-id",
+      }
+    }
+    route={
+      Object {
+        "name": "mobilepush",
+      }
+    }
+    templateData={
+      Object {
+        "_id": "test-id",
+        "type": "MOBILEPUSH",
+      }
+    }
+    type=""
+  />
+</SlideBoxContent__CreativesWrapper>
+`;
 
-exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilepushWrapper for loyalty module in library mode (create) 1`] = `<SlideBoxContent__CreativesWrapper />`;
+exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for non-loyalty module in library mode (create) 1`] = `
+<SlideBoxContent__CreativesWrapper>
+  <ForwardRef
+    date={0}
+    getDefaultTags=""
+    isFullMode={false}
+    key="creatives-mobilepush-wrapper"
+    onCreateComplete={[MockFunction]}
+    query={
+      Object {
+        "isEditFromCampaigns": undefined,
+        "module": "library",
+        "type": "embedded",
+      }
+    }
+    templateData={
+      Object {
+        "mode": "create",
+        "type": "MOBILEPUSH",
+      }
+    }
+    type=""
+  />
+</SlideBoxContent__CreativesWrapper>
+`;
+
+exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilePushNew for non-loyalty module in library mode (edit) 1`] = `
+<SlideBoxContent__CreativesWrapper>
+  <ForwardRef
+    getDefaultTags=""
+    isFullMode={false}
+    location={
+      Object {
+        "pathname": "/mobilepush/edit/",
+        "query": Object {
+          "isEditFromCampaigns": undefined,
+          "module": "library",
+          "type": "embedded",
+        },
+      }
+    }
+    params={
+      Object {
+        "id": "test-id",
+      }
+    }
+    route={
+      Object {
+        "name": "mobilepush",
+      }
+    }
+    templateData={
+      Object {
+        "_id": "test-id",
+        "type": "MOBILEPUSH",
+      }
+    }
+    type=""
+  />
+</SlideBoxContent__CreativesWrapper>
+`;
 
-exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobliPushEdit for loyalty module in library mode (edit) 1`] = `<SlideBoxContent__CreativesWrapper />`;
+exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobilepushWrapper for loyalty module in library mode (create) 1`] = `
+<SlideBoxContent__CreativesWrapper>
+  <Connect(MobilepushWrapper)
+    date={0}
+    isFullMode={false}
+    key="creatives-mobilepush-wrapper"
+    query={
+      Object {
+        "isEditFromCampaigns": undefined,
+        "module": "library",
+        "type": "embedded",
+      }
+    }
+    templateData={
+      Object {
+        "mode": "create",
+        "type": "MOBILEPUSH",
+      }
+    }
+    type=""
+  />
+</SlideBoxContent__CreativesWrapper>
+`;
+
+exports[`Test SlideBoxContent container Mobile Push with Loyalty Module Should render MobliPushEdit for loyalty module in library mode (edit) 1`] = `
+<SlideBoxContent__CreativesWrapper>
+  <ForwardRef
+    getDefaultTags=""
+    isFullMode={false}
+    location={
+      Object {
+        "pathname": "/mobilepush/edit/",
+        "query": Object {
+          "isEditFromCampaigns": undefined,
+          "module": "library",
+          "type": "embedded",
+        },
+      }
+    }
+    params={
+      Object {
+        "id": "test-id",
+      }
+    }
+    route={
+      Object {
+        "name": "mobilepush",
+      }
+    }
+    templateData={
+      Object {
+        "_id": "test-id",
+        "type": "MOBILEPUSH",
+      }
+    }
+    type=""
+  />
+</SlideBoxContent__CreativesWrapper>
+`;
 
 exports[`Test SlideBoxContent container Should handle isTestAndPreviewMode IIFE implementation correctly 1`] = `
 <SlideBoxContent__CreativesWrapper>

package/v2Containers/MobilePushNew/sagas.js

@@ -24,11 +24,10 @@ export function* createTemplate({template, callback}) {
       errorMsg,
     });
   } catch (error) {
-    // Call callback with error if provided, otherwise dispatch failure action
+    // Always dispatch failure action first, then call callback if provided
+    yield put({ type: types.CREATE_TEMPLATE_FAILURE, error, errorMsg });
     if (callback) {
       yield call(callback, new Error(errorMsg || error.message || error.toString()));
-    } else {
-      yield put({ type: types.CREATE_TEMPLATE_FAILURE, error, errorMsg });
     }
   }
 }
@@ -96,11 +95,10 @@ export function* editTemplate({ template, callback }) {
       yield call(callback, editResponseData);
     }
   } catch (error) {
-    // Call callback with error if provided, otherwise dispatch failure action
+    // Always dispatch failure action first, then call callback if provided
+    yield put({ type: types.EDIT_TEMPLATE_FAILURE, error, errorMsg });
     if (callback) {
       yield call(callback, new Error(errorMsg || error.message || error.toString()));
-    } else {
-      yield put({ type: types.EDIT_TEMPLATE_FAILURE, error, errorMsg });
     }
   }
 }

package/v2Containers/MobilePushNew/tests/sagas.test.js

@@ -121,6 +121,7 @@ describe('MobilePushNew Sagas', () => {
           [matchers.call.fn(Api.createMobilePushTemplateV2), throwError(error)],
         ])
         .call(Api.createMobilePushTemplateV2, template)
+        .put({ type: types.CREATE_TEMPLATE_FAILURE, error, errorMsg: undefined })
         .call(callback, new Error('Network error'))
         .run();
     });
@@ -392,6 +393,7 @@ describe('MobilePushNew Sagas', () => {
           [matchers.call.fn(Api.createMobilePushTemplateV2), throwError(error)],
         ])
         .call(Api.createMobilePushTemplateV2, template)
+        .put({ type: types.EDIT_TEMPLATE_FAILURE, error, errorMsg: undefined })
         .call(callback, new Error('Network error'))
         .run();
     });