@capillarytech/creatives-library 8.0.303 → 8.0.304

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

@@ -202,114 +202,6 @@ exports[`Test SlideBoxContent container Email component isTestAndPreviewMode IIF
 </SlideBoxContent__CreativesWrapper>
 `;
 
-exports[`Test SlideBoxContent container InApp vs InAppWrapper rendering conditions renders InApp when isFullMode=false and loyalty module enabled 1`] = `
-<SlideBoxContent__CreativesWrapper>
-  <Component
-    getDefaultTags=""
-    isFullMode={false}
-    key="creatives-inapp-create"
-    location={
-      Object {
-        "pathname": "/inapp/create",
-        "query": Object {
-          "isEditFromCampaigns": undefined,
-          "module": "library",
-          "type": "embedded",
-        },
-        "search": "",
-      }
-    }
-    onCreateComplete={[MockFunction]}
-    templateData={
-      Object {
-        "mode": "create",
-      }
-    }
-  />
-</SlideBoxContent__CreativesWrapper>
-`;
-
-exports[`Test SlideBoxContent container InApp vs InAppWrapper rendering conditions renders InApp when not full mode, not loyalty and new editor disabled 1`] = `
-<SlideBoxContent__CreativesWrapper>
-  <Component
-    getDefaultTags=""
-    isFullMode={false}
-    key="creatives-inapp-create"
-    location={
-      Object {
-        "pathname": "/inapp/create",
-        "query": Object {
-          "isEditFromCampaigns": undefined,
-          "module": "library",
-          "type": "embedded",
-        },
-        "search": "",
-      }
-    }
-    onCreateComplete={[MockFunction]}
-    templateData={
-      Object {
-        "mode": "create",
-      }
-    }
-  />
-</SlideBoxContent__CreativesWrapper>
-`;
-
-exports[`Test SlideBoxContent container InApp vs InAppWrapper rendering conditions renders InAppWrapper when full mode and new editor enabled 1`] = `
-<SlideBoxContent__CreativesWrapper>
-  <Component
-    getDefaultTags=""
-    isFullMode={true}
-    key="creatives-inapp-create"
-    location={
-      Object {
-        "pathname": "/inapp/create",
-        "query": Object {
-          "isEditFromCampaigns": undefined,
-          "module": "default",
-          "type": false,
-        },
-        "search": "",
-      }
-    }
-    onCreateComplete={[MockFunction]}
-    templateData={
-      Object {
-        "mode": "create",
-      }
-    }
-  />
-</SlideBoxContent__CreativesWrapper>
-`;
-
-exports[`Test SlideBoxContent container InApp vs InAppWrapper rendering conditions renders InAppWrapper when isFullMode=true and new editor disabled 1`] = `
-<SlideBoxContent__CreativesWrapper>
-  <Component
-    getDefaultTags=""
-    isFullMode={true}
-    key="creatives-inapp-create"
-    location={
-      Object {
-        "pathname": "/inapp/create",
-        "query": Object {
-          "isEditFromCampaigns": undefined,
-          "module": "default",
-          "type": false,
-        },
-        "search": "",
-      }
-    }
-    onCreateComplete={[MockFunction]}
-    templateData={
-      Object {
-        "mode": "create",
-      }
-    }
-  />
-</SlideBoxContent__CreativesWrapper>
-`;
-
 exports[`Test SlideBoxContent container Should handle isTestAndPreviewMode IIFE implementation correctly 1`] = `
 <SlideBoxContent__CreativesWrapper>
   <ForwardRef
@@ -1211,26 +1103,17 @@ exports[`Test SlideBoxContent container Should render correct component for what
 
 exports[`Test SlideBoxContent container Should render correct component for whatsapp channel create mode 2`] = `
 <SlideBoxContent__CreativesWrapper>
-  <Component
+  <Connect(Connect(UserIsAuthenticated(InjectIntl(CreativesCommon))))
+    date={0}
     getDefaultTags=""
-    key="creatives-inapp-create"
-    location={
-      Object {
-        "pathname": "/inapp/create",
-        "query": Object {
-          "isEditFromCampaigns": undefined,
-          "module": "library",
-          "type": "embedded",
-        },
-        "search": "",
-      }
-    }
+    key="creatives-inapp-wrapper"
     onCreateComplete={[MockFunction]}
     templateData={
       Object {
         "mode": "create",
       }
     }
+    type=""
   />
 </SlideBoxContent__CreativesWrapper>
 `;

package/v2Containers/EmailWrapper/components/EmailHTMLEditor.js

@@ -956,8 +956,8 @@ const EmailHTMLEditor = (props) => {
       }
     };
 
-    // If liquid enabled, validate first using extractTags API (skip in full/standalone mode)
-    if (isLiquidEnabled && getLiquidTags && !isFullMode) {
+    // If liquid enabled, validate first using extractTags API
+    if (isLiquidEnabled && getLiquidTags) {
       // Note: API validation errors are already cleared at the start of handleSave
       // This ensures fresh validation on every save attempt
 

package/v2Containers/EmailWrapper/components/__tests__/EmailHTMLEditor.test.js

@@ -1176,10 +1176,9 @@ describe('EmailHTMLEditor', () => {
       mockGetAllIssues.mockReturnValue([]);
 
       // Set subject and content via component interactions
-      // Use isFullMode: false (library mode) to test liquid validation path
       const { rerender } = renderWithIntl({
         isGetFormData: false,
-        isFullMode: false,
+        isFullMode: true,
         metaEntities: {
           tags: {
             standard: [{ name: 'customer.name' }],
@@ -1207,7 +1206,7 @@ describe('EmailHTMLEditor', () => {
             <EmailHTMLEditor
               {...defaultProps}
               isGetFormData
-              isFullMode={false}
+              isFullMode
               metaEntities={{
                 tags: {
                   standard: [{ name: 'customer.name' }],
@@ -1232,10 +1231,9 @@ describe('EmailHTMLEditor', () => {
       mockGetAllIssues.mockReturnValue([]);
 
       // Set subject and content via component interactions
-      // Use isFullMode: false (library mode) to test liquid validation path
       const { rerender } = renderWithIntl({
         isGetFormData: false,
-        isFullMode: false,
+        isFullMode: true,
         isLiquidEnabled: true,
         getLiquidTags,
       });
@@ -1255,7 +1253,7 @@ describe('EmailHTMLEditor', () => {
       await act(async () => {
         rerender(
           <IntlProvider locale="en" messages={{}}>
-            <EmailHTMLEditor {...defaultProps} isGetFormData isFullMode={false} getLiquidTags={getLiquidTags} />
+            <EmailHTMLEditor {...defaultProps} isGetFormData isFullMode getLiquidTags={getLiquidTags} />
           </IntlProvider>
         );
       });
@@ -1283,10 +1281,9 @@ describe('EmailHTMLEditor', () => {
       mockGetAllIssues.mockReturnValue([]);
 
       // Set subject and content via component interactions
-      // Use isFullMode: false (library mode) to test liquid validation path
       const { rerender } = renderWithIntl({
         isGetFormData: false,
-        isFullMode: false,
+        isFullMode: true,
         isLiquidEnabled: true,
         getLiquidTags,
         showLiquidErrorInFooter,
@@ -1308,7 +1305,7 @@ describe('EmailHTMLEditor', () => {
       await act(async () => {
         rerender(
           <IntlProvider locale="en" messages={{}}>
-            <EmailHTMLEditor {...defaultProps} isGetFormData isFullMode={false} getLiquidTags={getLiquidTags} showLiquidErrorInFooter={showLiquidErrorInFooter} onValidationFail={onValidationFail} />
+            <EmailHTMLEditor {...defaultProps} isGetFormData isFullMode getLiquidTags={getLiquidTags} showLiquidErrorInFooter={showLiquidErrorInFooter} onValidationFail={onValidationFail} />
           </IntlProvider>
         );
       });
@@ -1328,7 +1325,13 @@ describe('EmailHTMLEditor', () => {
         return Promise.resolve(true);
       });
 
-      const getFormdata = jest.fn();
+      const emailActions = {
+        ...defaultProps.emailActions,
+        transformEmailTemplate: jest.fn((obj, callback) => callback(obj)),
+        createTemplate: jest.fn((obj, callback) => {
+          callback({ templateId: { _id: '123', versions: {} } });
+        }),
+      };
       const getLiquidTags = jest.fn((content, callback) => {
         callback({ askAiraResponse: { data: [] }, isError: false });
       });
@@ -1336,12 +1339,12 @@ describe('EmailHTMLEditor', () => {
       mockGetAllIssues.mockReturnValue([]);
 
       // Set subject and content via component interactions
-      // Use isFullMode: false (library mode) to test liquid validation path
       const { rerender } = renderWithIntl({
         isGetFormData: false,
-        isFullMode: false,
+        isFullMode: true,
+        isLiquidEnabled: true,
         getLiquidTags,
-        getFormdata,
+        emailActions,
         templateName: 'New Template',
       });
       const input = screen.getByTestId('subject-input');
@@ -1360,7 +1363,7 @@ describe('EmailHTMLEditor', () => {
       await act(async () => {
         rerender(
           <IntlProvider locale="en" messages={{}}>
-            <EmailHTMLEditor {...defaultProps} isGetFormData isFullMode={false} getLiquidTags={getLiquidTags} getFormdata={getFormdata} templateName="New Template" />
+            <EmailHTMLEditor {...defaultProps} isGetFormData isFullMode getLiquidTags={getLiquidTags} emailActions={emailActions} templateName="New Template" />
           </IntlProvider>
         );
       });
@@ -1369,9 +1372,8 @@ describe('EmailHTMLEditor', () => {
         expect(validateLiquidTemplateContent).toHaveBeenCalled();
       }, { timeout: 5000 });
 
-      // In library mode (isFullMode=false), save proceeds via getFormdata
       await waitFor(() => {
-        expect(getFormdata).toHaveBeenCalled();
+        expect(emailActions.createTemplate).toHaveBeenCalled();
       }, { timeout: 5000 });
     });
 
@@ -2709,98 +2711,4 @@ describe('EmailHTMLEditor', () => {
       expect(screen.getByTestId('html-editor')).toBeInTheDocument();
     });
   });
-
-  describe('isFullMode - skip liquid validation on save', () => {
-    it('skips liquid validation when isFullMode is true', async () => {
-      validateLiquidTemplateContent.mockClear();
-      const getLiquidTags = jest.fn((content, callback) => {
-        callback({ askAiraResponse: { data: [] }, isError: false });
-      });
-      const getFormdata = jest.fn();
-      // Ensure no HTML/Label/Liquid errors from HtmlEditor
-      mockGetAllIssues.mockReturnValue([]);
-
-      const { rerender } = renderWithIntl({
-        isGetFormData: false,
-        isFullMode: true,
-        isLiquidEnabled: true,
-        getLiquidTags,
-        getFormdata,
-      });
-      const input = screen.getByTestId('subject-input');
-      await act(async () => {
-        fireEvent.change(input, { target: { value: 'Valid Subject' } });
-      });
-      const changeButton = screen.getByTestId('trigger-content-change');
-      await act(async () => {
-        fireEvent.click(changeButton);
-      });
-      await act(async () => {
-        await new Promise((resolve) => setTimeout(resolve, 100));
-      });
-      // Trigger save
-      await act(async () => {
-        rerender(
-          <IntlProvider locale="en" messages={{}}>
-            <EmailHTMLEditor {...defaultProps} isGetFormData isFullMode getLiquidTags={getLiquidTags} getFormdata={getFormdata} />
-          </IntlProvider>
-        );
-      });
-
-      // In full mode, liquid validation should be skipped entirely
-      await act(async () => {
-        await new Promise((resolve) => setTimeout(resolve, 200));
-      });
-      expect(validateLiquidTemplateContent).not.toHaveBeenCalled();
-    });
-
-    it('proceeds directly to save without liquid validation in full mode', async () => {
-      validateLiquidTemplateContent.mockClear();
-      const getLiquidTags = jest.fn((content, callback) => {
-        callback({ askAiraResponse: { data: [] }, isError: false });
-      });
-      const emailActions = {
-        ...defaultProps.emailActions,
-        transformEmailTemplate: jest.fn((obj, callback) => callback(obj)),
-        createTemplate: jest.fn((obj, callback) => {
-          callback({ templateId: { _id: '123', versions: {} } });
-        }),
-      };
-      // Ensure no HTML/Label/Liquid errors from HtmlEditor
-      mockGetAllIssues.mockReturnValue([]);
-
-      const { rerender } = renderWithIntl({
-        isGetFormData: false,
-        isFullMode: true,
-        getLiquidTags,
-        emailActions,
-        templateName: 'New Template',
-      });
-      const input = screen.getByTestId('subject-input');
-      await act(async () => {
-        fireEvent.change(input, { target: { value: 'Valid Subject' } });
-      });
-      const changeButton = screen.getByTestId('trigger-content-change');
-      await act(async () => {
-        fireEvent.click(changeButton);
-      });
-      await act(async () => {
-        await new Promise((resolve) => setTimeout(resolve, 100));
-      });
-      // Trigger save
-      await act(async () => {
-        rerender(
-          <IntlProvider locale="en" messages={{}}>
-            <EmailHTMLEditor {...defaultProps} isGetFormData isFullMode getLiquidTags={getLiquidTags} emailActions={emailActions} templateName="New Template" />
-          </IntlProvider>
-        );
-      });
-
-      // Should skip liquid validation and proceed to save directly
-      await waitFor(() => {
-        expect(emailActions.createTemplate).toHaveBeenCalled();
-      }, { timeout: 5000 });
-      expect(validateLiquidTemplateContent).not.toHaveBeenCalled();
-    });
-  });
 });