fix(useUpload): imporve test coverage

nemo-shen · Feb 16, 2024 · cb98850 · cb98850
1 parent 79c6448
commit cb98850
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/packages/core/useUpload/index.test.ts b/packages/core/useUpload/index.test.ts
@@ -118,6 +118,15 @@ describe('useUpload', () => {
       'image-jpg',
       'image-png1',
     ])
+
+    const { files: files2, append: append2 } = useUpload({
+      url: '#',
+      accept: '.png',
+    })
+    expect(
+      append2(new File([''], 'image-jpg.jpg', { type: 'image/jpg' }))
+    ).rejects.toThrow('Invalid file type.')
+    expect(append2(new File([''], 'image-png1.png', { type: 'image/png' }))).rejects.toThrow('Invalid file type.')
   })
 
   test('max-count', async () => {

diff --git a/packages/core/useUpload/index.ts b/packages/core/useUpload/index.ts
@@ -107,10 +107,8 @@ export const useUpload = (options: UseUploadOptions): UseUploadReturn => {
 
   const append = async (file: File | File[] | FileList) => {
     let appendFiles = []
-    if (file instanceof FileList) {
+    if ('length' in file) {
       appendFiles = Array.from(file)
-    } else if (Array.isArray(file)) {
-      appendFiles = file
     } else if (file instanceof File) {
       appendFiles = [file]
     }