fixed test cases

glific · Feb 26, 2025 · 13184f5 · 13184f5
1 parent a3c2636
commit 13184f5
Show file tree

Hide file tree

Showing 11 changed files with 48 additions and 69 deletions.
diff --git a/src/containers/Chat/ChatMessages/AddToMessageTemplate/AddToMessageTemplate.test.tsx b/src/containers/Chat/ChatMessages/AddToMessageTemplate/AddToMessageTemplate.test.tsx
@@ -8,7 +8,7 @@ import AddToMessageTemplate from './AddToMessageTemplate';
 let resultReturned = false;
 
 const mocks = [
-  filterTemplatesQuery('', {}),
+  filterTemplatesQuery('', []),
   {
     request: {
       query: SAVE_MESSAGE_TEMPLATE_MUTATION,

diff --git a/src/containers/Flow/Flow.test.tsx b/src/containers/Flow/Flow.test.tsx
@@ -37,9 +37,9 @@ const mocks = [
     isActive: true,
     isPinned: false,
     isBackground: false,
-    name: 'Copy of Help',
-    keywords: [],
-    description: '',
+    name: 'Help flow',
+    keywords: ['help'],
+    description: 'Copy of Help',
     ignoreKeywords: false,
     addRoleIds: [],
     deleteRoleIds: [],
@@ -50,7 +50,7 @@ const mocks = [
     isBackground: false,
     name: 'Copy of Help',
     keywords: ['help', 'activity'],
-    description: 'Help flow',
+    description: 'Copy of Help',
     ignoreKeywords: false,
     addRoleIds: [],
     deleteRoleIds: [],
@@ -372,7 +372,7 @@ it('should create copy of a template flow', async () => {
 
   const { container, getByTestId } = render(copyFlow());
   await waitFor(() => {
-    expect(screen.getByText('Copy of Help')).toBeInTheDocument();
+    expect(screen.getAllByText('Copy of Help')[0]).toBeInTheDocument();
   });
 
   const button = getByTestId('submitActionButton');

diff --git a/src/containers/HSM/HSM.test.tsx b/src/containers/HSM/HSM.test.tsx
@@ -198,13 +198,19 @@ describe('Add mode', () => {
     fireEvent.click(screen.getByText('Add Variable'));
 
     await waitFor(() => {
-      expect(screen.getByText('Hi {{1}}')).toBeInTheDocument();
+      expect(screen.getAllByTestId('variable')).toHaveLength(1);
+    });
+
+    fireEvent.click(screen.getByText('Add Variable'));
+
+    await waitFor(() => {
+      expect(screen.getAllByTestId('variable')).toHaveLength(2);
     });
 
     fireEvent.click(screen.getAllByTestId('delete-variable')[0]);
 
     await waitFor(() => {
-      expect(screen.queryByPlaceholderText('Define value ')).not.toBeInTheDocument();
+      expect(screen.getAllByTestId('variable')).toHaveLength(1);
     });
   });
 

diff --git a/src/containers/HSM/TemplateVariables/TemplateVariables.tsx b/src/containers/HSM/TemplateVariables/TemplateVariables.tsx
@@ -56,7 +56,7 @@ export const TemplateVariables = ({
         <div className={styles.Variables}>
           {variables.length !== 0 && <h2>Set custom variable values for the message</h2>}
           {variables.map((variable: any, index: number) => (
-            <div key={variable.id} className={styles.VariableContainer}>
+            <div data-testid="variable" key={variable.id} className={styles.VariableContainer}>
               <div className={styles.Variable} key={index}>
                 <OutlinedInput
                   sx={{

diff --git a/src/containers/Organization/Onboarding/ReachOutToUs/ReachOutToUs.tsx b/src/containers/Organization/Onboarding/ReachOutToUs/ReachOutToUs.tsx
@@ -27,9 +27,9 @@ export const ReachOutToUs = ({ open, setOpen, handleStepChange, saveData }: Reac
   const [success, setSuccess] = useState(false);
 
   const helperText = (
-    <p>
+    <span>
       You can also reachout to us on <a href="mailto:support@glific.org">support@glific.org</a>.
-    </p>
+    </span>
   );
 
   const formFields = [

diff --git a/src/containers/SpeedSend/SpeedSend.module.css b/src/containers/SpeedSend/SpeedSend.module.css
@@ -83,13 +83,12 @@
 }
 
 .Warning {
-  color: #ff0000;
-  margin-left: -43px;
+  color: #ff0000 !important;
 }
 
 .IsActive {
   color: #555555;
   font-weight: 400;
   line-height: 18px;
   font-size: 16px;
-}
+}
diff --git a/src/containers/SpeedSend/SpeedSend.test.tsx b/src/containers/SpeedSend/SpeedSend.test.tsx
@@ -73,9 +73,7 @@ describe('test creating a speed send', () => {
   });
 
   test('should validate media', async () => {
-    mockedAxios.get.mockImplementationOnce(() =>
-      Promise.resolve({ data: { is_valid: true, message: 'valid media' } })
-    );
+    mockedAxios.get.mockImplementationOnce(() => Promise.resolve({ data: { is_valid: true, message: 'valid media' } }));
 
     render(addSpeedSendContainer);
 
@@ -135,9 +133,7 @@ describe('test creating a speed send', () => {
   });
 
   test('should create a speed send', async () => {
-    mockedAxios.get.mockImplementationOnce(() =>
-      Promise.resolve({ data: { is_valid: true, message: 'valid media' } })
-    );
+    mockedAxios.get.mockImplementationOnce(() => Promise.resolve({ data: { is_valid: true, message: 'valid media' } }));
 
     render(addSpeedSendContainer);
 
@@ -243,9 +239,7 @@ describe('test creating a speed send', () => {
 
 describe('test editing a speed send', () => {
   test('should render speed send form', async () => {
-    mockedAxios.get.mockImplementationOnce(() =>
-      Promise.resolve({ data: { is_valid: true, message: 'valid media' } })
-    );
+    mockedAxios.get.mockImplementationOnce(() => Promise.resolve({ data: { is_valid: true, message: 'valid media' } }));
 
     render(editSpeedSendContainer('2'));
 

diff --git a/src/containers/SpeedSend/SpeedSend.tsx b/src/containers/SpeedSend/SpeedSend.tsx
@@ -82,8 +82,7 @@ export const SpeedSend = () => {
     variables: { opts: { order: 'ASC' } },
   });
 
-  const [getSessionTemplate, { data: template, loading: templateLoading }] =
-    useLazyQuery<any>(GET_SPEED_SEND);
+  const [getSessionTemplate, { data: template, loading: templateLoading }] = useLazyQuery<any>(GET_SPEED_SEND);
 
   useEffect(() => {
     if (languages) {
@@ -137,11 +136,7 @@ export const SpeedSend = () => {
       const translationsCopy = JSON.parse(translationsValue);
       const currentLanguage = language?.id || languageIdValue.id;
 
-      if (
-        Object.keys(translationsCopy).length > 0 &&
-        translationsCopy[currentLanguage] &&
-        !location.state?.language
-      ) {
+      if (Object.keys(translationsCopy).length > 0 && translationsCopy[currentLanguage] && !location.state?.language) {
         let content = translationsCopy[currentLanguage];
         title = content.label;
         body = content.body || '';
@@ -156,16 +151,12 @@ export const SpeedSend = () => {
 
     if (languageOptions.length > 0 && languageIdValue) {
       if (location.state && location.state !== 'copy') {
-        const selectedLangauge = languageOptions.find(
-          (lang: any) => lang.label === location.state.language
-        );
+        const selectedLangauge = languageOptions.find((lang: any) => lang.label === location.state.language);
         navigate(location.pathname);
 
         setLanguageId(selectedLangauge);
       } else if (!language?.id) {
-        const selectedLangauge = languageOptions.find(
-          (lang: any) => lang.id === languageIdValue.id
-        );
+        const selectedLangauge = languageOptions.find((lang: any) => lang.id === languageIdValue.id);
         setLanguageId(selectedLangauge);
       } else {
         setLanguageId(language);
@@ -307,9 +298,7 @@ export const SpeedSend = () => {
           language: value,
           label: translationsCopy[translationId].label,
           body: translationsCopy[translationId].body,
-          type: translationsCopy[translationId].MessageMedia
-            ? translationsCopy[translationId].MessageMedia.type
-            : null,
+          type: translationsCopy[translationId].MessageMedia ? translationsCopy[translationId].MessageMedia.type : null,
           MessageMedia: translationsCopy[translationId].MessageMedia,
         });
       } else {
@@ -325,9 +314,7 @@ export const SpeedSend = () => {
   };
 
   const handleLanguageChange = (value: any) => {
-    const selected = languageOptions.find(
-      ({ label: languageLabel }: any) => languageLabel === value
-    );
+    const selected = languageOptions.find(({ label: languageLabel }: any) => languageLabel === value);
     if (selected && isEditing) {
       updateTranslation(selected);
     } else if (selected) {
@@ -360,21 +347,15 @@ export const SpeedSend = () => {
   const displayWarning = () => {
     if (type && type.id === 'STICKER') {
       setWarning(
-        <div className={styles.Warning}>
-          <ol>
-            <li>{t('Animated stickers are not supported.')}</li>
-            <li>{t('Captions along with stickers are not supported.')}</li>
-          </ol>
-        </div>
+        <span className={styles.Warning}>
+          <span>{t('Animated stickers are not supported.')}</span>
+          <br />
+          <span>{t('Captions along with stickers are not supported.')}</span>
+          <span></span>
+        </span>
       );
     } else if (type && type.id === 'AUDIO') {
-      setWarning(
-        <div className={styles.Warning}>
-          <ol>
-            <li>{t('Captions along with audio are not supported.')}</li>
-          </ol>
-        </div>
-      );
+      setWarning(<span className={styles.Warning}>{t('Captions along with audio are not supported.')}</span>);
     } else {
       setWarning(null);
     }
@@ -396,9 +377,7 @@ export const SpeedSend = () => {
           is: (val: any) => val && val.id,
           then: (schema) => schema.required(t('Attachment URL is required.')),
         }),
-      body: Yup.string()
-        .required(t('Message is required.'))
-        .max(1024, 'Maximum 1024 characters are allowed'),
+      body: Yup.string().required(t('Message is required.')).max(1024, 'Maximum 1024 characters are allowed'),
     },
     [['type', 'attachmentURL']]
   );

diff --git a/src/mocks/Chat.tsx b/src/mocks/Chat.tsx
@@ -767,6 +767,7 @@ export const createAndSendMessageMutation = (input: any) => ({
           body: 'Hey There Wow',
           insertedAt: '2020-06-25T13:36:43Z',
           id: '10388',
+          media: null,
           receiver: {
             id: '2',
           },

diff --git a/src/mocks/Collection.tsx b/src/mocks/Collection.tsx
@@ -453,17 +453,7 @@ export const deleteContactFromCollection = {
 export const getCollectionsList = (label?: any) => ({
   request: {
     query: GET_COLLECTIONS_LIST,
-    variables: {
-      filter: {
-        groupType: 'WABA',
-        label,
-      },
-      opts: {
-        limit: 50,
-        offset: 0,
-        order: 'ASC',
-      },
-    },
+    variables: { filter: { groupType: 'WABA', label: '' }, opts: { limit: 50, offset: 0, order: 'ASC' } },
   },
   result: {
     data: {

diff --git a/src/mocks/Groups.tsx b/src/mocks/Groups.tsx
@@ -835,6 +835,8 @@ export const waMessageReceivedSubscription = {
           __typename: 'WaGroup',
           id: '1',
         },
+        poll: null,
+        pollContent: '{}',
       },
     },
   },
@@ -867,6 +869,8 @@ export const waMessageSendSubscription = {
           __typename: 'WaGroup',
           id: '1',
         },
+        poll: null,
+        pollContent: '{}',
       },
     },
   },
@@ -893,6 +897,8 @@ export const waSentMessageCollectionQuery = {
         contact: null,
         contextMessage: null,
         waGroup: null,
+        poll: null,
+        pollContent: '{}',
       },
     },
   },
@@ -913,6 +919,8 @@ const updateWaSatatusMock = {
           __typename: 'WaGroup',
           id: '4',
         },
+        poll: null,
+        pollContent: '{}',
       },
     },
   },
@@ -1165,6 +1173,8 @@ export const getWaGroupQuery = {
           id: '1',
           label: 'Arkansas ducks',
           lastCommunicationAt: '2024-03-28T10:41:18Z',
+          fields: '{}',
+          groups: null,
           waManagedPhone: {
             id: '1',
             phone: '468/236-8754',