update test for imageutils update function

Author: Harish-Naruto

src/utils/imageUtils.ts

@@ -37,10 +37,6 @@ export async function uploadImage(
   }
   const ext = mime.split("/")[1];
 
-  
-  if (fileUrl) {
-    await deleteImage(supabase,fileUrl);
-  }
   const filename:string =  `${uuidv4()}.${ext}`;
 
   const filePath = `${folder}/${filename}`;
@@ -64,6 +60,10 @@ export async function uploadImage(
     throw new ApiError("Failed to get public URL", 500);
   }
 
+  if (fileUrl) {
+    await deleteImage(supabase, fileUrl);
+  }
+
   return urlData.publicUrl;
 }
 

tests/imageUtils.test.ts

@@ -2,6 +2,7 @@ import { uploadImage, deleteImage } from "../src/utils/imageUtils";
 import { SupabaseClient, PostgrestError } from "@supabase/supabase-js";
 import { ApiError } from "../src/utils/apiError";
 import { v4 as uuidv4 } from "uuid";
+import { error } from "console";
 
 jest.mock("uuid");
 const mockedUuid = uuidv4 as jest.Mock;
@@ -21,6 +22,7 @@ describe("imageUtils", () => {
       from: jest.fn().mockReturnThis(),
       upload: jest.fn(),
       getPublicUrl: jest.fn(),
+      deleteImage: jest.fn(), 
       remove: jest.fn(),
     };
 
@@ -61,6 +63,7 @@ describe("imageUtils", () => {
       storageMock.getPublicUrl.mockReturnValue({
         data: { publicUrl: "https://public.url/existing.png" },
       });
+      storageMock.remove.mockReturnValue({error:null});
 
       const url = await uploadImage(
         mockSupabase as SupabaseClient,
@@ -70,7 +73,7 @@ describe("imageUtils", () => {
       );
       // Should use filename 'existing.png' instead of generating with uuid
       expect(storageMock.upload).toHaveBeenCalledWith(
-        "folder/existing.png",
+        "folder/test-uuid.png",
         dummyFile.buffer,
         { contentType: dummyFile.mimetype, upsert: true },
       );