@capillarytech/creatives-library 8.0.165 → 8.0.166

package/package.json

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

package/v2Components/NavigationBar/saga.js

@@ -6,13 +6,15 @@ export default [
   {
     key: 'navigation',
     saga: function* navigationConfigSaga() {
-      yield all(CapCollapsibleLeftNavigationSagas.map(saga => saga.call()));
+      yield all(CapCollapsibleLeftNavigationSagas.map((saga) => saga.call()));
     },
   },
   {
     key: 'analyticsBotSaga',
     saga: function* analyticsBotSagaFn() {
-      yield all(analyticsBotSaga.map(saga => saga.call()));
+      if (analyticsBotSaga && Array.isArray(analyticsBotSaga)) {
+        yield all(analyticsBotSaga.map((saga) => saga.call()));
+      }
     },
   },
 ];

package/v2Components/NavigationBar/tests/saga.test.js

@@ -1,34 +1,78 @@
 import { expectSaga } from "redux-saga-test-plan";
-import rootSagas from "../saga";
 import * as matchers from "redux-saga-test-plan/matchers";
 import CapCollapsibleLeftNavigationSagas from '@capillarytech/cap-ui-library/CapCollapsibleLeftNavigation/saga';
-import { analyticsBotSaga } from "@capillarytech/cap-ui-library/CapAskAira";
+
+import rootSagas from "../saga";
+
+// Mock the analyticsBotSaga module
+jest.mock('@capillarytech/cap-ui-library/CapAskAira', () => ({
+  analyticsBotSaga: [
+    {
+      * call() { yield true; },
+    },
+    {
+      * call() { yield true; },
+    },
+  ],
+}));
 
 describe("analyticsBotSagaFn", () => {
-  test.concurrent("should call all analytics bot sagas", () => {
-    const { saga: analyticsBotSagaFn } = rootSagas.find(
-      s => s.key === "analyticsBotSaga"
-    );
+  const { saga: analyticsBotSagaFn } = rootSagas.find(
+    (s) => s.key === "analyticsBotSaga"
+  );
+
+  test.concurrent("should handle when analyticsBotSaga is valid array", () => {
+    // Mock module with valid array
+    jest.mock('@capillarytech/cap-ui-library/CapAskAira', () => ({
+      analyticsBotSaga: [
+        {
+          * call() { yield true; },
+        },
+        {
+          * call() { yield true; },
+        },
+      ],
+    }));
+
     return expectSaga(analyticsBotSagaFn)
       .provide([
-        [matchers.call.fn(analyticsBotSaga[0]), undefined],
-        [matchers.call.fn(analyticsBotSaga[1]), undefined]
+        [matchers.call.fn(() => {}), undefined],
       ])
       .run();
   });
+
+  test.concurrent("should handle when analyticsBotSaga is undefined", () => {
+    // Mock module with undefined analyticsBotSaga
+    jest.mock('@capillarytech/cap-ui-library/CapAskAira', () => ({
+      analyticsBotSaga: undefined,
+    }));
+
+    return expectSaga(analyticsBotSagaFn)
+      .run();
+  });
+
+  test.concurrent("should handle when analyticsBotSaga is not an array", () => {
+    // Mock module with non-array analyticsBotSaga
+    jest.mock('@capillarytech/cap-ui-library/CapAskAira', () => ({
+      analyticsBotSaga: {},
+    }));
+
+    return expectSaga(analyticsBotSagaFn)
+      .run();
+  });
 });
 
 describe("navigationConfigSaga", () => {
-    it.concurrent("should call all analytics bot sagas", () => {
-      const { saga: navigationConfigSaga } = rootSagas.find(
-        s => s.key === "navigation"
-      );
-  
-      return expectSaga(navigationConfigSaga)
-        .provide([
-          [matchers.call.fn(CapCollapsibleLeftNavigationSagas[0]), undefined],
-          [matchers.call.fn(CapCollapsibleLeftNavigationSagas[1]), undefined]
-        ])
-        .run();
-    });
+  it.concurrent("should call all analytics bot sagas", () => {
+    const { saga: navigationConfigSaga } = rootSagas.find(
+      (s) => s.key === "navigation"
+    );
+
+    return expectSaga(navigationConfigSaga)
+      .provide([
+        [matchers.call.fn(CapCollapsibleLeftNavigationSagas[0]), undefined],
+        [matchers.call.fn(CapCollapsibleLeftNavigationSagas[1]), undefined],
+      ])
+      .run();
   });
+});

package/v2Containers/Email/index.js

@@ -2696,19 +2696,7 @@ export class Email extends React.Component { // eslint-disable-line react/prefer
     const previewSubject = formData['template-subject'];
     const testOrPreviewProps = {channel: EMAIL, content: previewContent, subject: previewSubject};
     
-    // Call our handler instead of the props methods to set the flag
-    const handleClick = () => {
-      if (action === 'TEST') {
-        // For test action, open our Test & Preview slidebox
-        this.handleTestAndPreview();
-      } else {
-        // For preview action, call the original method
-        const { onPreviewContentClicked } = this.props;
-        if (onPreviewContentClicked) {
-          onPreviewContentClicked(testOrPreviewProps);
-        }
-      }
-    };
+    const { onPreviewContentClicked, onTestContentClicked } = this.props;
     
     return (
       <CapButton
@@ -2718,7 +2706,7 @@ export class Email extends React.Component { // eslint-disable-line react/prefer
           padding: '0 6px',
         }}
         type="flat"
-        onClick={handleClick}
+        onClick={() => action === 'PREVIEW' ? onPreviewContentClicked(testOrPreviewProps) : onTestContentClicked(testOrPreviewProps)}
       >
         <CapIcon type={action === 'PREVIEW' ? "eye" : "lab"}/>
         {action === 'PREVIEW' ? <FormattedMessage {...messages.preview} /> : <FormattedMessage {...messages.testMessage} />}
@@ -2773,7 +2761,7 @@ export class Email extends React.Component { // eslint-disable-line react/prefer
       tags = this.props.supportedTags;
     }
     const { showImageSelectionBox = false } = this.state;
-    const showTestAndPreview = !showImageSelectionBox && getDefaultTags === 'outbound';
+     const showTestAndPreview = !showImageSelectionBox && getDefaultTags === 'outbound' && !this.props.showTestAndPreviewSlidebox;
     return (
       <div className="email-container">
         <CapSpin spinning={isLoading}>