Improve Code Coverage in src/screens/OrganizationPeople/OrganizationP…

…eople.tsx (#3133) * chore: test coverage increase to 100% in organizationpeople component * unused before each hook removed * removed testing console logs * Use consistent translation function for error handling
PalisadoesFoundation · Jan 4, 2025 · 1f2379b · 1f2379b
1 parent 5550711
commit 1f2379b
Show file tree

Hide file tree

Showing 3 changed files with 737 additions and 16 deletions.
diff --git a/src/screens/OrganizationPeople/AddMember.tsx b/src/screens/OrganizationPeople/AddMember.tsx
@@ -110,10 +110,7 @@ function AddMember(): JSX.Element {
         orgId: currentUrl,
       });
     } catch (error: unknown) {
-      if (error instanceof Error) {
-        toast.error(error.message);
-        console.log(error.message);
-      }
+      errorHandler(tCommon, error);
     }
   };
 
@@ -388,7 +385,7 @@ function AddMember(): JSX.Element {
                             >
                               {userDetails.user.image ? (
                                 <img
-                                  src={userDetails.user.image ?? undefined}
+                                  src={userDetails.user.image}
                                   alt="avatar"
                                   className={styles.TableImage}
                                 />
@@ -443,7 +440,10 @@ function AddMember(): JSX.Element {
         show={createNewUserModalisOpen}
         onHide={toggleCreateNewUserModal}
       >
-        <Modal.Header className={styles.createUserModalHeader}>
+        <Modal.Header
+          className={styles.createUserModalHeader}
+          data-testid="createUser"
+        >
           <Modal.Title>Create User</Modal.Title>
         </Modal.Header>
         <Modal.Body>