Merge pull request #57 from call-0f-code/member-Profile-update-fix

fix issue with profile update

Author: callofcode07

src/controllers/member.controller.ts

@@ -74,7 +74,7 @@ export const updateAMember =
   (supabase: SupabaseClient) => async (req: Request, res: Response) => {
 
    const { memberId } = req.params;
-
+  
     if(!memberId) throw new ApiError("No memberId provided", 400);
 
     const parsedBody = JSON.parse(req.body.memberData);
@@ -84,7 +84,7 @@ export const updateAMember =
       const oldData = await memberService.getDetails(memberId);
       const oldImage = oldData?.profilePhoto;
 
-      if(oldImage) await uploadImage(supabase, req.file, "members", oldImage);
+      if(oldImage) imageUrl = await uploadImage(supabase, req.file, "members", oldImage);
       else imageUrl = await uploadImage(supabase, req.file, "members");
     }
     if (imageUrl) parsedBody.profilePhoto = imageUrl;
@@ -155,4 +155,4 @@ export const getUserInterviews = async (req: Request, res: Response) => {
 
   const interviews = await memberService.getInterviews(memberId);
   res.status(200).json({ success: true, interviews });
-};
+};

tests/Member.test.ts

@@ -180,8 +180,10 @@ describe('Member Controller - updateAMember', () => {
     );
 
     expect(memberService.updateMember).toHaveBeenCalledWith('abc-123', {
+      profilePhoto: 'https://new.url/image.png',
     });
 
+
     expect(res.status).toHaveBeenCalledWith(200);
     expect(res.json).toHaveBeenCalledWith({
       success: true,