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

package/utils/tests/commonUtil.test.js

@@ -8,11 +8,6 @@ import {
   validateCarouselCards,
   hasPersonalizationTags,
   checkForPersonalizationTokens,
-  isValidEmail,
-  isValidMobile,
-  formatPhoneNumber,
-  getMessageForDevice,
-  getTitleForDevice,
 } from "../commonUtils";
 import { skipTags } from "../tagValidations";
 import { SMS_TRAI_VAR } from '../../v2Containers/SmsTrai/Edit/constants';
@@ -27,17 +22,13 @@ 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 () => {
@@ -50,10 +41,11 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
-      standardErrors: ["empty"],
+      standardErrors: [undefined],
       liquidErrors: [],
       tabType: undefined
     });
@@ -68,6 +60,7 @@ describe("validateLiquidTemplateContent", () => {
       getLiquidTags,
       formatMessage,
       messages,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).not.toHaveBeenCalled();
@@ -84,6 +77,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -107,6 +101,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -129,6 +124,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -149,6 +145,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -169,6 +166,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -199,6 +197,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -219,6 +218,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
@@ -228,15 +228,9 @@ describe("validateLiquidTemplateContent", () => {
     });
   });
 
-  it("calls onError when API returns success but response.errors has validation errors (e.g. unsupported tag)", async () => {
+  it("calls onError for unsupported tags", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
-      cb({
-        askAiraResponse: {
-          errors: [{ message: "Unsupported tag: custom_tag" }],
-          data: []
-        },
-        isError: false
-      })
+      cb({ askAiraResponse: { errors: [], data: [{ name: "baz" }] }, isError: false })
     );
     await validateLiquidTemplateContent("foo", {
       getLiquidTags,
@@ -244,33 +238,18 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onError).toHaveBeenCalledWith({
       standardErrors: [],
-      liquidErrors: ["Unsupported tag: custom_tag"],
+      liquidErrors: [undefined],
       tabType: undefined
     });
     expect(onSuccess).not.toHaveBeenCalled();
   });
 
-  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 () => {
+  it("calls onSuccess for valid content", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "foo" }, { name: "bar" }] }, isError: false })
     );
@@ -280,6 +259,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onSuccess).toHaveBeenCalledWith("foo", undefined);
@@ -295,6 +275,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -312,6 +293,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -329,6 +311,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -346,6 +329,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -363,6 +347,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -380,6 +365,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -397,6 +383,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -414,6 +401,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       skipTags
     });
@@ -431,6 +419,7 @@ describe("validateLiquidTemplateContent", () => {
     };
     const onError = jest.fn();
     const onSuccess = jest.fn();
+    const tagLookupMap = {};
     const eventContextTags = [];
     await validateLiquidTemplateContent('', {
       getLiquidTags,
@@ -438,6 +427,7 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
     });
     expect(formatMessage).toHaveBeenCalledWith(messages.emailBodyEmptyError);
@@ -449,7 +439,7 @@ describe("validateLiquidTemplateContent", () => {
     expect(onSuccess).not.toHaveBeenCalled();
   });
 
-  it("calls onSuccess when API returns extracted tags from {% for %} template but no errors (extracted tags are not validated)", async () => {
+  it("should skip tags that appear in {% %} syntax (like order.items in for loop)", 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 })
@@ -460,13 +450,15 @@ 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("calls onSuccess when API returns extracted tags in {% %} blocks but no errors", async () => {
+  it("should skip tags that appear only inside {% %} blocks (like item.name in for loop)", 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 })
@@ -477,13 +469,15 @@ 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("calls onSuccess when API returns extracted tags not in content but no errors", async () => {
+  it("should validate tags that don't appear in content but are extracted by API", async () => {
     const content = 'Some content without the tag';
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "extractedTag" }] }, isError: false })
@@ -494,13 +488,19 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
-    expect(onSuccess).toHaveBeenCalledWith(content, undefined);
-    expect(onError).not.toHaveBeenCalled();
+    // 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();
   });
 
-  it("calls onSuccess when API returns tags outside {% %} but no errors", async () => {
+  it("should validate tags that appear outside {% %} blocks", 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 })
@@ -511,70 +511,20 @@ describe("validateLiquidTemplateContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
-    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).toHaveBeenCalledWith(content, undefined);
-    expect(onError).not.toHaveBeenCalled();
-  });
-
-  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
+    // 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();
+    expect(onSuccess).not.toHaveBeenCalled();
   });
 
-  it("calls onSuccess when API returns tags with dots in {% %} syntax but no errors", async () => {
+  it("should skip tags with dots that appear in {% %} syntax", async () => {
     const content = '{% assign myVar = order.items %} Some text';
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [{ name: "order.items" }] }, isError: false })
@@ -585,8 +535,10 @@ 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();
   });
@@ -634,170 +586,6 @@ describe("validateLiquidTemplateContent", () => {
   });
 });
 
-describe("isValidEmail", () => {
-  it("returns true for valid email addresses", () => {
-    expect(isValidEmail("user@example.com")).toBe(true);
-    expect(isValidEmail("test.user@domain.co")).toBe(true);
-    expect(isValidEmail("a@b.co")).toBe(true);
-  });
-
-  it("returns false for invalid email addresses", () => {
-    expect(isValidEmail("")).toBe(false);
-    expect(isValidEmail("invalid")).toBe(false);
-    expect(isValidEmail("@nodomain.com")).toBe(false);
-    expect(isValidEmail("noatsign.com")).toBe(false);
-    expect(isValidEmail("missingtld@domain")).toBe(false);
-    expect(isValidEmail("  user@example.com  ")).toBe(false);
-  });
-
-  it("returns true for edge case single-char local part", () => {
-    expect(isValidEmail("x@y.co")).toBe(true);
-  });
-});
-
-describe("isValidMobile", () => {
-  it("returns true for valid mobile numbers (8-15 digits, no leading zero)", () => {
-    expect(isValidMobile("12345678")).toBe(true);
-    expect(isValidMobile("9123456789")).toBe(true);
-    expect(isValidMobile("123456789012345")).toBe(true);
-  });
-
-  it("returns false for invalid mobile numbers", () => {
-    expect(isValidMobile("")).toBe(false);
-    expect(isValidMobile("01234567")).toBe(false);
-    expect(isValidMobile("1234567")).toBe(false);
-    expect(isValidMobile("1234567890123456")).toBe(false);
-    expect(isValidMobile("abc12345678")).toBe(false);
-    expect(isValidMobile("12345 67890")).toBe(false);
-  });
-
-  it("returns true for exactly 8 and exactly 15 digits", () => {
-    expect(isValidMobile("12345678")).toBe(true);
-    expect(isValidMobile("123456789012345")).toBe(true);
-  });
-});
-
-describe("formatPhoneNumber", () => {
-  it("returns empty string for falsy input", () => {
-    expect(formatPhoneNumber("")).toBe("");
-    expect(formatPhoneNumber(null)).toBe("");
-    expect(formatPhoneNumber(undefined)).toBe("");
-  });
-
-  it("strips non-digit characters", () => {
-    expect(formatPhoneNumber("91 234 567 890")).toBe("91234567890");
-    expect(formatPhoneNumber("+91-234567890")).toBe("91234567890");
-    expect(formatPhoneNumber("(123) 456-7890")).toBe("1234567890");
-  });
-
-  it("returns digits-only string unchanged", () => {
-    expect(formatPhoneNumber("9123456789")).toBe("9123456789");
-  });
-
-  it("returns empty string for whitespace-only input", () => {
-    expect(formatPhoneNumber("   ")).toBe("");
-  });
-});
-
-describe("hasPersonalizationTags", () => {
-  it("returns true when text has liquid tags {{ }}", () => {
-    expect(hasPersonalizationTags("Hello {{name}}")).toBe(true);
-    expect(hasPersonalizationTags("{{foo}}")).toBe(true);
-  });
-
-  it("returns true when text has bracket tags [ ]", () => {
-    expect(hasPersonalizationTags("Hello [event.name]")).toBe(true);
-    expect(hasPersonalizationTags("[tag]")).toBe(true);
-  });
-
-  it("returns false for empty or no tags", () => {
-    expect(hasPersonalizationTags("")).toBeFalsy();
-    expect(hasPersonalizationTags()).toBeFalsy();
-    expect(hasPersonalizationTags("plain text")).toBe(false);
-    expect(hasPersonalizationTags("only {{ open")).toBe(false);
-  });
-
-  it("returns false when only [ or only ] present without matching pair", () => {
-    expect(hasPersonalizationTags("only [ open")).toBe(false);
-    expect(hasPersonalizationTags("only ] close")).toBe(false);
-  });
-
-  it("returns true when liquid tags have content with spaces", () => {
-    expect(hasPersonalizationTags("Hello {{ customer.name }}")).toBe(true);
-  });
-});
-
-describe("getMessageForDevice", () => {
-  it("returns message for device from templateData", () => {
-    const templateData = {
-      versions: {
-        android: { base: { expandableDetails: { message: "Android msg" } } },
-        ios: { base: { expandableDetails: { message: "iOS msg" } } },
-      },
-    };
-    expect(getMessageForDevice(templateData, "android")).toBe("Android msg");
-    expect(getMessageForDevice(templateData, "ios")).toBe("iOS msg");
-  });
-
-  it("returns undefined for missing path", () => {
-    expect(getMessageForDevice(null, "android")).toBeUndefined();
-    expect(getMessageForDevice({}, "android")).toBeUndefined();
-    expect(getMessageForDevice({ versions: {} }, "android")).toBeUndefined();
-  });
-
-  it("returns undefined when base exists but expandableDetails is missing", () => {
-    expect(getMessageForDevice({ versions: { android: { base: {} } } }, "android")).toBeUndefined();
-  });
-});
-
-describe("getTitleForDevice", () => {
-  it("returns title for device from templateData", () => {
-    const templateData = {
-      versions: {
-        android: { base: { title: "Android title" } },
-        ios: { base: { title: "iOS title" } },
-      },
-    };
-    expect(getTitleForDevice(templateData, "android")).toBe("Android title");
-    expect(getTitleForDevice(templateData, "ios")).toBe("iOS title");
-  });
-
-  it("returns empty string for missing title", () => {
-    expect(getTitleForDevice(null, "android")).toBe("");
-    expect(getTitleForDevice({}, "android")).toBe("");
-    expect(getTitleForDevice({ versions: { android: { base: {} } } }, "android")).toBe("");
-  });
-
-  it("returns empty string when base.title is undefined", () => {
-    expect(getTitleForDevice({ versions: { android: { base: { title: undefined } } } }, "android")).toBe("");
-  });
-});
-
-describe("checkForPersonalizationTokens", () => {
-  it("returns true when formData contains liquid or bracket tokens", () => {
-    expect(checkForPersonalizationTokens({ 0: { content: "Hi {{name}}" } })).toBe(true);
-    expect(checkForPersonalizationTokens({ tab1: { message: "Hello [event.id]" } })).toBe(true);
-  });
-
-  it("returns false for empty or no tokens", () => {
-    expect(checkForPersonalizationTokens(null)).toBe(false);
-    expect(checkForPersonalizationTokens({})).toBe(false);
-    expect(checkForPersonalizationTokens({ 0: { content: "plain" } })).toBe(false);
-  });
-
-  it("returns false for non-object formData", () => {
-    expect(checkForPersonalizationTokens("string")).toBe(false);
-  });
-
-  it("returns true for two-level nested object containing token", () => {
-    expect(checkForPersonalizationTokens({ tab: { body: "Hi {{name}}" } })).toBe(true);
-  });
-
-  it("returns false for formData with array value (no string tokens)", () => {
-    expect(checkForPersonalizationTokens({ 0: { items: ["a", "b"] } })).toBe(false);
-  });
-});
-
 describe("validateMobilePushContent", () => {
   const formatMessage = jest.fn(msg => msg.id);
   const messages = {
@@ -805,6 +593,7 @@ describe("validateMobilePushContent", () => {
     somethingWentWrong: { id: "wrong" },
     unsupportedTagsValidationError: { id: "unsupported" }
   };
+  const tagLookupMap = { foo: true };
   const eventContextTags = [{ tagName: "foo" }];
   const onError = jest.fn();
   const onSuccess = jest.fn();
@@ -813,7 +602,7 @@ describe("validateMobilePushContent", () => {
     jest.clearAllMocks();
   });
 
-  it("calls onError for empty formData (validateMobilePushContent)", async () => {
+  it("calls onError for empty formData", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [] }, isError: false })
     );
@@ -822,17 +611,18 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
         currentTab: 1
       }
     );
     expect(onError).toHaveBeenCalled();
   });
 
-  it("calls onSuccess for valid android and ios content (validateMobilePushContent)", async () => {
+  it("calls onSuccess for valid android and ios content", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [] }, isError: false })
     );
@@ -843,6 +633,7 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       currentTab: 1
     });
@@ -860,6 +651,7 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       currentTab: 1
     });
@@ -877,6 +669,7 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       currentTab: 2
     });
@@ -894,6 +687,7 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onSuccess).toHaveBeenCalledWith(JSON.stringify(formData[0]), "android");
@@ -910,6 +704,7 @@ describe("validateMobilePushContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags
     });
     expect(onSuccess).toHaveBeenCalledWith("null", "android");
@@ -922,10 +717,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
         currentTab: 1,
       },
     );
@@ -939,10 +735,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
         currentTab: 1,
       },
     );
@@ -956,10 +753,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
         currentTab: 1,
       },
     );
@@ -973,10 +771,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
         currentTab: 1,
       },
     );
@@ -990,10 +789,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -1006,10 +806,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -1022,10 +823,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -1038,10 +840,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -1054,10 +857,11 @@ describe("validateMobilePushContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
       },
     );
     expect(onError).toHaveBeenCalled();
@@ -1071,6 +875,7 @@ describe("validateInAppContent", () => {
     somethingWentWrong: { id: "wrong" },
     unsupportedTagsValidationError: { id: "unsupported" }
   };
+  const tagLookupMap = { foo: true };
   const eventContextTags = [{ tagName: "foo" }];
   const onError = jest.fn();
   const onSuccess = jest.fn();
@@ -1079,7 +884,7 @@ describe("validateInAppContent", () => {
     jest.clearAllMocks();
   });
 
-  it("calls onError for empty formData (validateInAppContent)", async () => {
+  it("calls onError for empty formData", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [] }, isError: false })
     );
@@ -1088,16 +893,17 @@ describe("validateInAppContent", () => {
       {
         getLiquidTags,
         formatMessage,
-      messages,
-      onError,
-      onSuccess,
-      eventContextTags,
+        messages,
+        onError,
+        onSuccess,
+        tagLookupMap,
+        eventContextTags,
       }
     );
     expect(onError).toHaveBeenCalled();
   });
 
-  it("calls onSuccess for valid android and ios content (validateInAppContent)", async () => {
+  it("calls onSuccess for valid android and ios content", async () => {
     const getLiquidTags = jest.fn((content, cb) =>
       cb({ askAiraResponse: { errors: [], data: [] }, isError: false })
     );
@@ -1117,6 +923,7 @@ describe("validateInAppContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       singleTab: ANDROID,
     });
@@ -1143,6 +950,7 @@ describe("validateInAppContent", () => {
       messages,
       onError,
       onSuccess,
+      tagLookupMap,
       eventContextTags,
       singleTab: IOS,
     });
@@ -1169,11 +977,11 @@ describe("getChannelData", () => {
     expect(getChannelData("SMS", formData)).toBe("Hi Test");
   });
 
-  it("returns string with undefineds for SMS when base and template-name are empty or undefined", () => {
+  it("returns string with undefineds for SMS channel with missing fields", () => {
     const formData = { base: {}, "template-name": undefined };
     expect(getChannelData("SMS", formData)).toBe("undefined undefined");
   });
-  it("returns string with undefineds for SMS when formData is empty string", () => {
+  it("returns string with undefineds for SMS channel with missing fields", () => {
     expect(getChannelData("SMS", "")).toBe("undefined undefined");
   });