@capillarytech/creatives-library 8.0.248 → 8.0.249

package/v2Containers/WebPush/Create/utils/payloadBuilder.test.js

@@ -23,8 +23,9 @@ describe('createWebPushPayload', () => {
     brandIconSrc: '',
     brandIconUrl: '',
     buttons: [],
-    onClickBehaviour: ON_CLICK_BEHAVIOUR_OPTIONS.OPEN_SITE,
+    onClickBehaviour: ON_CLICK_BEHAVIOUR_OPTIONS.SITE_URL,
     redirectUrl: '',
+    websiteLink: 'https://example.com',
   };
 
   describe('Basic Payload Structure', () => {
@@ -44,7 +45,8 @@ describe('createWebPushPayload', () => {
                 title: 'Test Title',
                 message: 'Test Message',
                 onClickAction: {
-                  type: ACTION_TYPES.OPEN_SITE,
+                  type: ACTION_TYPES.SITE_URL,
+                  url: 'https://example.com',
                 },
               },
             },
@@ -221,14 +223,16 @@ describe('createWebPushPayload', () => {
       expect(payload.versions.base.content.webpush.onClickAction.url).toBe('https://example.com');
     });
 
-    it('should include onClickAction with OPEN_SITE type when OPEN_SITE is selected', () => {
+    it('should include onClickAction with SITE_URL type when SITE_URL is selected', () => {
       const payload = createWebPushPayload({
         ...defaultParams,
-        onClickBehaviour: ON_CLICK_BEHAVIOUR_OPTIONS.OPEN_SITE,
+        onClickBehaviour: ON_CLICK_BEHAVIOUR_OPTIONS.SITE_URL,
+        websiteLink: 'https://example.com',
       });
 
       expect(payload.versions.base.content.webpush.onClickAction).toEqual({
-        type: ACTION_TYPES.OPEN_SITE,
+        type: ACTION_TYPES.SITE_URL,
+        url: 'https://example.com',
       });
     });
 
@@ -370,8 +374,9 @@ describe('createWebPushPayload', () => {
         brandIconSrc: '',
         brandIconUrl: '',
         buttons: [],
-        onClickBehaviour: ON_CLICK_BEHAVIOUR_OPTIONS.OPEN_SITE,
+        onClickBehaviour: ON_CLICK_BEHAVIOUR_OPTIONS.SITE_URL,
         redirectUrl: '',
+        websiteLink: 'https://example.com',
       });
 
       expect(payload.name).toBe('');
@@ -381,7 +386,8 @@ describe('createWebPushPayload', () => {
       expect(payload.versions.base.content.webpush.image).toBeUndefined();
       expect(payload.versions.base.content.webpush.brandIcon).toBeUndefined();
       expect(payload.versions.base.content.webpush.onClickAction).toEqual({
-        type: ACTION_TYPES.OPEN_SITE,
+        type: ACTION_TYPES.SITE_URL,
+        url: 'https://example.com',
       });
       expect(payload.versions.base.content.webpush.ctas).toBeUndefined();
     });

package/v2Containers/WebPush/constants.js

@@ -79,14 +79,14 @@ export const WEBPUSH_BUTTON_TYPES = {
 
 // OnClick Behaviour Options
 export const ON_CLICK_BEHAVIOUR_OPTIONS = {
-  OPEN_SITE: 'OPEN_SITE',
+  SITE_URL: 'SITE_URL',
   REDIRECT_TO_URL: 'REDIRECT_TO_URL',
 };
 
 // Action Types
 export const ACTION_TYPES = {
   URL: 'URL',
-  OPEN_SITE: 'OPEN_SITE',
+  SITE_URL: 'SITE_URL',
 };
 
 // WebPush Content Field Names
@@ -126,3 +126,7 @@ export const TLD_PATTERN = /^https?:\/\/[^/?#:]+\.\w{2,}(?:\/|$|\?|#|:)/i;
 // Utility Constants
 // Stable empty array to prevent unnecessary re-renders when used as default prop values
 export const EMPTY_ARRAY = [];
+
+export const EXTERNAL_URL = 'EXTERNAL_URL';
+export const SITE_URL = 'SITE_URL';
+export const URL = 'URL';

package/v2Containers/WebPush/tests/selectors.test.js

@@ -839,5 +839,122 @@ describe('WebPush selectors', () => {
       });
     });
   });
+
+  describe('accountData selector (from templates state)', () => {
+    // This tests the accountData selector logic used in mapStateToProps
+    const createAccountDataSelector = () => {
+      return (state) => {
+        const templatesState = state.get('templates');
+        if (!templatesState) {
+          return {};
+        }
+        const templates = templatesState.toJS();
+        return templates?.selectedWebPushAccount || {};
+      };
+    };
+
+    it.concurrent('should select selectedWebPushAccount from templates state', () => {
+      const mockTemplatesState = fromJS({
+        selectedWebPushAccount: {
+          id: 123,
+          accountId: 123,
+          name: 'Test WebPush Account',
+          configs: {
+            websiteLink: 'https://example.com',
+          },
+        },
+      });
+
+      const mockState = fromJS({
+        templates: mockTemplatesState,
+      });
+
+      const selector = createAccountDataSelector();
+      const result = selector(mockState);
+
+      expect(result).toEqual({
+        id: 123,
+        accountId: 123,
+        name: 'Test WebPush Account',
+        configs: {
+          websiteLink: 'https://example.com',
+        },
+      });
+    });
+
+    it.concurrent('should return empty object when templates state does not exist', () => {
+      const mockState = fromJS({
+        otherState: { value: 'test' },
+      });
+
+      const selector = createAccountDataSelector();
+      const result = selector(mockState);
+
+      expect(result).toEqual({});
+    });
+
+    it.concurrent('should return empty object when selectedWebPushAccount does not exist', () => {
+      const mockTemplatesState = fromJS({
+        weCrmAccounts: [],
+        templates: [],
+      });
+
+      const mockState = fromJS({
+        templates: mockTemplatesState,
+      });
+
+      const selector = createAccountDataSelector();
+      const result = selector(mockState);
+
+      expect(result).toEqual({});
+    });
+
+    it.concurrent('should handle selectedWebPushAccount with websiteLink in configs', () => {
+      const mockTemplatesState = fromJS({
+        selectedWebPushAccount: {
+          id: 456,
+          accountId: 456,
+          name: 'Account with Website',
+          configs: {
+            websiteLink: 'https://test.com',
+            otherConfig: 'value',
+          },
+        },
+      });
+
+      const mockState = fromJS({
+        templates: mockTemplatesState,
+      });
+
+      const selector = createAccountDataSelector();
+      const result = selector(mockState);
+
+      expect(result.configs.websiteLink).toBe('https://test.com');
+      expect(result.id).toBe(456);
+    });
+
+    it.concurrent('should handle selectedWebPushAccount without websiteLink', () => {
+      const mockTemplatesState = fromJS({
+        selectedWebPushAccount: {
+          id: 789,
+          accountId: 789,
+          name: 'Account without Website',
+          configs: {
+            otherConfig: 'value',
+          },
+        },
+      });
+
+      const mockState = fromJS({
+        templates: mockTemplatesState,
+      });
+
+      const selector = createAccountDataSelector();
+      const result = selector(mockState);
+
+      expect(result.configs.websiteLink).toBeUndefined();
+      expect(result.id).toBe(789);
+    });
+  });
 });