@capillarytech/creatives-library 8.0.308 → 8.0.309

package/utils/tests/commonUtil.test.js

@@ -27,13 +27,17 @@ describe("validateLiquidTemplateContent", () => {
     somethingWentWrong: { id: "wrong" },
     unsupportedTagsValidationError: { id: "unsupported" }
   };
-  const tagLookupMap = { foo: true };
   const eventContextTags = [{ tagName: "bar" }];
   const onError = jest.fn();
   const onSuccess = jest.fn();
 
   beforeEach(() => {
     jest.clearAllMocks();
+    formatMessage.mockImplementation((msg, vars) =>
+      vars && vars.unsupportedTags != null
+        ? `${msg?.id}:${vars.unsupportedTags}`
+        : (msg?.id ?? "unsupported")
+    );
   });
 
   it("calls onError for empty content", async () => {
@@ -46,11 +50,10 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
-      standardErrors: [undefined],
+      standardErrors: ["empty"],
       liquidErrors: [],
       tabType: undefined
     });
@@ -65,7 +68,6 @@ describe("validateLiquidTemplateContent", () => {
       getLiquidTags,
       formatMessage,
       messages,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).not.toHaveBeenCalled();
@@ -82,7 +84,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -106,7 +107,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -129,7 +129,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -150,7 +149,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -171,7 +169,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -202,7 +199,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -223,7 +219,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -233,9 +228,15 @@ describe("validateLiquidTemplateContent", () => {
     });
   });
 
-  it("calls onError for unsupported tags", async () => {
+  it("calls onError when API returns success but response.errors has validation errors (e.g. unsupported tag)", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
-      cb({ askAiraResponse: { errors: [], data: [{ name: "baz" }] }, isError: false })
+      cb({
+        askAiraResponse: {
+          errors: [{ message: "Unsupported tag: custom_tag" }],
+          data: []
+        },
+        isError: false
+      })
     );
     await validateLiquidTemplateContent("foo", {
       getLiquidTags,
@@ -243,18 +244,33 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
       standardErrors: [],
-      liquidErrors: [undefined],
+      liquidErrors: ["Unsupported tag: custom_tag"],
       tabType: undefined
     });
     expect(onSuccess).not.toHaveBeenCalled();
   });
 
-  it("calls onSuccess for valid content", async () => {
+  it("calls onSuccess when API returns no errors and a single extracted tag (extracted tags are not validated)", async () => {
+    const getLiquidTags = jest.fn((content, cb) =>
+      cb({ askAiraResponse: { errors: [], data: [{ name: "foo" }] }, isError: false })
+    );
+    await validateLiquidTemplateContent("foo", {
+      getLiquidTags,
+      formatMessage,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags
+    });
+    expect(onSuccess).toHaveBeenCalledWith("foo", undefined);
+    expect(onError).not.toHaveBeenCalled();
+  });
+
+  it("calls onSuccess for valid content when API returns multiple extracted tags but no errors", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "foo" }, { name: "bar" }] }, isError: false })
     );
@@ -264,7 +280,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onSuccess).toHaveBeenCalledWith("foo", undefined);
@@ -280,7 +295,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -298,7 +312,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -316,7 +329,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -334,7 +346,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -352,7 +363,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -370,7 +380,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -388,7 +397,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -406,7 +414,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -424,7 +431,6 @@ describe("validateLiquidTemplateContent", () => {
     };
     const onError = jest.fn();
     const onSuccess = jest.fn();
-    const tagLookupMap = {};
     const eventContextTags = [];
     await validateLiquidTemplateContent('', {
       getLiquidTags,
@@ -432,7 +438,6 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
     });
     expect(formatMessage).toHaveBeenCalledWith(messages.emailBodyEmptyError);
@@ -444,7 +449,7 @@ describe("validateLiquidTemplateContent", () => {
     expect(onSuccess).not.toHaveBeenCalled();
   });
 
-  it("should skip tags that appear in {% %} syntax (like order.items in for loop)", async () => {
+  it("calls onSuccess when API returns extracted tags from {% for %} template but no errors (extracted tags are not validated)", async () => {
     const content = '{% for item in order.items %} Hello {% endfor %}';
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "order.items" }] }, isError: false })
@@ -455,15 +460,13 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
-    // order.items appears in {% %} syntax, so it should be skipped and validation should pass
     expect(onSuccess).toHaveBeenCalledWith(content, undefined);
     expect(onError).not.toHaveBeenCalled();
   });
 
-  it("should skip tags that appear only inside {% %} blocks (like item.name in for loop)", async () => {
+  it("calls onSuccess when API returns extracted tags in {% %} blocks but no errors", async () => {
     const content = '{% for item in order.items %} {{ item.name }} - {{ item.quantity }} {% endfor %}';
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "item.name" }, { name: "item.quantity" }] }, isError: false })
@@ -474,15 +477,13 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
-    // item.name and item.quantity appear inside {% for %} block, so they should be skipped
     expect(onSuccess).toHaveBeenCalledWith(content, undefined);
     expect(onError).not.toHaveBeenCalled();
   });
 
-  it("should validate tags that don't appear in content but are extracted by API", async () => {
+  it("calls onSuccess when API returns extracted tags not in content but no errors", async () => {
     const content = 'Some content without the tag';
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "extractedTag" }] }, isError: false })
@@ -493,19 +494,13 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
-    // extractedTag doesn't appear in content but was extracted by API, should be validated
-    expect(onError).toHaveBeenCalledWith({
-      standardErrors: [],
-      liquidErrors: [undefined],
-      tabType: undefined
-    });
-    expect(onSuccess).not.toHaveBeenCalled();
+    expect(onSuccess).toHaveBeenCalledWith(content, undefined);
+    expect(onError).not.toHaveBeenCalled();
   });
 
-  it("should validate tags that appear outside {% %} blocks", async () => {
+  it("calls onSuccess when API returns tags outside {% %} but no errors", async () => {
     const content = '{{ outsideTag }} {% for item in order.items %} {{ item.name }} {% endfor %}';
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "outsideTag" }, { name: "item.name" }] }, isError: false })
@@ -516,20 +511,70 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
-    // outsideTag appears outside {% %} block, so it should be validated
-    // item.name appears inside block, so it should be skipped
-    expect(onError).toHaveBeenCalledWith({
-      standardErrors: [],
-      liquidErrors: [undefined],
-      tabType: undefined
+    expect(onSuccess).toHaveBeenCalledWith(content, undefined);
+    expect(onError).not.toHaveBeenCalled();
+  });
+
+  it("calls onSuccess when API returns tag in {{ }} and no errors", async () => {
+    const content = 'Hello {{ unsupportedTag }} world';
+    const getLiquidTags = jest.fn((content, cb) =>
+      cb({ askAiraResponse: { errors: [], data: [{ name: "unsupportedTag" }] }, isError: false })
+    );
+    await validateLiquidTemplateContent(content, {
+      getLiquidTags,
+      formatMessage,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags
     });
-    expect(onSuccess).not.toHaveBeenCalled();
+    expect(onSuccess).toHaveBeenCalledWith(content, undefined);
+    expect(onError).not.toHaveBeenCalled();
   });
 
-  it("should skip tags with dots that appear in {% %} syntax", async () => {
+  it("calls onSuccess when API returns tag with spacing variants and no errors", async () => {
+    const content = '{{ tag}} and {{tag }} and {{ tag }}';
+    const getLiquidTags = jest.fn((content, cb) =>
+      cb({
+        askAiraResponse: {
+          errors: [],
+          data: [{ name: "tag" }]
+        },
+        isError: false
+      })
+    );
+    await validateLiquidTemplateContent(content, {
+      getLiquidTags,
+      formatMessage,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags
+    });
+    expect(onSuccess).toHaveBeenCalledWith(content, undefined);
+    expect(onError).not.toHaveBeenCalled();
+  });
+
+  it("calls onSuccess when API returns tag inside {% %} blocks but no errors", async () => {
+    const content = '{% for x in some.unsupported %} {{ x }} {% endfor %}';
+    const getLiquidTags = jest.fn((content, cb) =>
+      cb({ askAiraResponse: { errors: [], data: [{ name: "some.unsupported" }] }, isError: false })
+    );
+    await validateLiquidTemplateContent(content, {
+      getLiquidTags,
+      formatMessage,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags
+    });
+    expect(onSuccess).toHaveBeenCalledWith(content, undefined);
+    expect(onError).not.toHaveBeenCalled();
+  });
+
+  it("calls onSuccess when API returns tags with dots in {% %} syntax but no errors", async () => {
     const content = '{% assign myVar = order.items %} Some text';
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "order.items" }] }, isError: false })
@@ -540,10 +585,8 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
-    // order.items appears in {% %} syntax, so it should be skipped
     expect(onSuccess).toHaveBeenCalledWith(content, undefined);
     expect(onError).not.toHaveBeenCalled();
   });
@@ -762,7 +805,6 @@ describe("validateMobilePushContent", () => {
     somethingWentWrong: { id: "wrong" },
     unsupportedTagsValidationError: { id: "unsupported" }
   };
-  const tagLookupMap = { foo: true };
   const eventContextTags = [{ tagName: "foo" }];
   const onError = jest.fn();
   const onSuccess = jest.fn();
@@ -771,7 +813,7 @@ describe("validateMobilePushContent", () => {
     jest.clearAllMocks();
   });
 
-  it("calls onError for empty formData", async () => {
+  it("calls onError for empty formData (validateMobilePushContent)", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [] }, isError: false })
     );
@@ -780,18 +822,17 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
         currentTab: 1
       }
     );
     expect(onError).toHaveBeenCalled();
   });
 
-  it("calls onSuccess for valid android and ios content", async () => {
+  it("calls onSuccess for valid android and ios content (validateMobilePushContent)", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [] }, isError: false })
     );
@@ -802,7 +843,6 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       currentTab: 1
     });
@@ -820,7 +860,6 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       currentTab: 1
     });
@@ -838,7 +877,6 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       currentTab: 2
     });
@@ -856,7 +894,6 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onSuccess).toHaveBeenCalledWith(JSON.stringify(formData[0]), "android");
@@ -873,7 +910,6 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags
     });
     expect(onSuccess).toHaveBeenCalledWith("null", "android");
@@ -886,11 +922,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
         currentTab: 1,
       },
     );
@@ -904,11 +939,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
         currentTab: 1,
       },
     );
@@ -922,11 +956,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
         currentTab: 1,
       },
     );
@@ -940,11 +973,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
         currentTab: 1,
       },
     );
@@ -958,11 +990,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -975,11 +1006,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -992,11 +1022,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -1009,11 +1038,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -1026,11 +1054,10 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -1044,7 +1071,6 @@ describe("validateInAppContent", () => {
     somethingWentWrong: { id: "wrong" },
     unsupportedTagsValidationError: { id: "unsupported" }
   };
-  const tagLookupMap = { foo: true };
   const eventContextTags = [{ tagName: "foo" }];
   const onError = jest.fn();
   const onSuccess = jest.fn();
@@ -1053,7 +1079,7 @@ describe("validateInAppContent", () => {
     jest.clearAllMocks();
   });
 
-  it("calls onError for empty formData", async () => {
+  it("calls onError for empty formData (validateInAppContent)", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [] }, isError: false })
     );
@@ -1062,17 +1088,16 @@ describe("validateInAppContent", () => {
       {
         getLiquidTags,
         formatMessage,
-        messages,
-        onError,
-        onSuccess,
-        tagLookupMap,
-        eventContextTags,
+      messages,
+      onError,
+      onSuccess,
+      eventContextTags,
       }
     );
     expect(onError).toHaveBeenCalled();
   });
 
-  it("calls onSuccess for valid android and ios content", async () => {
+  it("calls onSuccess for valid android and ios content (validateInAppContent)", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [] }, isError: false })
     );
@@ -1092,7 +1117,6 @@ describe("validateInAppContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       singleTab: ANDROID,
     });
@@ -1119,7 +1143,6 @@ describe("validateInAppContent", () => {
       messages,
       onError,
       onSuccess,
-      tagLookupMap,
       eventContextTags,
       singleTab: IOS,
     });
@@ -1146,11 +1169,11 @@ describe("getChannelData", () => {
     expect(getChannelData("SMS", formData)).toBe("Hi Test");
   });
 
-  it("returns string with undefineds for SMS channel with missing fields", () => {
+  it("returns string with undefineds for SMS when base and template-name are empty or undefined", () => {
     const formData = { base: {}, "template-name": undefined };
     expect(getChannelData("SMS", formData)).toBe("undefined undefined");
   });
-  it("returns string with undefineds for SMS channel with missing fields", () => {
+  it("returns string with undefineds for SMS when formData is empty string", () => {
     expect(getChannelData("SMS", "")).toBe("undefined undefined");
   });