Refactor isDropImage and test

Lemoncode · Oct 1, 2024 · abfd1ba · abfd1ba
1 parent 2512069
commit abfd1ba
Showing 1 changed file with 10 additions and 40 deletions.
diff --git a/src/pods/canvas/canvas.util.spec.ts b/src/pods/canvas/canvas.util.spec.ts
@@ -1,3 +1,5 @@
+import { isDropImageFile } from './canvas.util';
+
 describe('isDropImageFile', () => {
   it('should return True when there is an image in the item when dropped', () => {
     //Arrange
@@ -12,19 +14,11 @@ describe('isDropImageFile', () => {
       },
     } as unknown as React.DragEvent<HTMLDivElement>;
 
-    const MockisDropImageFile = (e: React.DragEvent<HTMLDivElement>) => {
-      return (
-        e.dataTransfer.items.length > 0 &&
-        e.dataTransfer.items[0].kind === 'file' &&
-        e.dataTransfer.items[0].type.startsWith('image/')
-      );
-    };
-
     //Act
-    MockisDropImageFile(mockDatatransfer);
+    const result = isDropImageFile(mockDatatransfer);
 
     //Assert
-    expect(MockisDropImageFile(mockDatatransfer)).toBe(true);
+    expect(result).toBe(true);
   });
 
   it('should return False when there is not an image in the item when dropped', () => {
@@ -40,19 +34,11 @@ describe('isDropImageFile', () => {
       },
     } as unknown as React.DragEvent<HTMLDivElement>;
 
-    const MockisDropImageFile = (e: React.DragEvent<HTMLDivElement>) => {
-      return (
-        e.dataTransfer.items.length > 0 &&
-        e.dataTransfer.items[0].kind === 'file' &&
-        e.dataTransfer.items[0].type.startsWith('image/')
-      );
-    };
-
     //Act
-    MockisDropImageFile(mockDatatransfer);
+    const result = isDropImageFile(mockDatatransfer);
 
     //Assert
-    expect(MockisDropImageFile(mockDatatransfer)).toBe(false);
+    expect(result).toBe(false);
   });
 
   it('should return False when the kind in the item is not file', () => {
@@ -68,19 +54,11 @@ describe('isDropImageFile', () => {
       },
     } as unknown as React.DragEvent<HTMLDivElement>;
 
-    const MockisDropImageFile = (e: React.DragEvent<HTMLDivElement>) => {
-      return (
-        e.dataTransfer.items.length > 0 &&
-        e.dataTransfer.items[0].kind === 'file' &&
-        e.dataTransfer.items[0].type.startsWith('image/')
-      );
-    };
-
     //Act
-    MockisDropImageFile(mockDatatransfer);
+    const result = isDropImageFile(mockDatatransfer);
 
     //Assert
-    expect(MockisDropImageFile(mockDatatransfer)).toBe(false);
+    expect(result).toBe(false);
   });
 
   it('should return False when no items are dropped', () => {
@@ -91,18 +69,10 @@ describe('isDropImageFile', () => {
       },
     } as unknown as React.DragEvent<HTMLDivElement>;
 
-    const MockisDropImageFile = (e: React.DragEvent<HTMLDivElement>) => {
-      return (
-        e.dataTransfer.items.length > 0 &&
-        e.dataTransfer.items[0].kind === 'file' &&
-        e.dataTransfer.items[0].type.startsWith('image/')
-      );
-    };
-
     //Act
-    MockisDropImageFile(mockDatatransfer);
+    const result = isDropImageFile(mockDatatransfer);
 
     //Assert
-    expect(MockisDropImageFile(mockDatatransfer)).toBe(false);
+    expect(result).toBe(false);
   });
 });