fix: onStart should ignore beforeUpload false

Author: zombieJ

src/AjaxUploader.tsx

@@ -108,13 +108,14 @@ class AjaxUploader extends Component<UploadProps> {
     // Batch upload files
     Promise.all(postFiles).then(fileList => {
       const { onBatchStart } = this.props;
-      const enabledFiles = fileList.filter(file => file);
 
       onBatchStart?.(fileList.map(({ origin, parsedFile }) => ({ file: origin, parsedFile })));
 
-      enabledFiles.forEach(file => {
-        this.post(file);
-      });
+      fileList
+        .filter(file => file.parsedFile !== null)
+        .forEach(file => {
+          this.post(file);
+        });
     });
   };
 

tests/uploader.spec.js

@@ -516,8 +516,10 @@ describe('uploader', () => {
         return true;
       });
 
-      const onBatchStart = await testWrapper({ beforeUpload });
+      const onStart = jest.fn();
+      const onBatchStart = await testWrapper({ beforeUpload, onStart });
 
+      expect(onStart).toHaveBeenCalledTimes(1);
       expect(beforeUpload).toHaveBeenCalledTimes(2);
       expect(onBatchStart).toHaveBeenCalledWith(
         files.map(file =>