Merge pull request #152 from Geode-solutions/fix/auto_upload

fix(FileUploader): prop file doesn't auto upload

Author: JulienChampagnol

components/FileSelector.vue

@@ -2,7 +2,7 @@
   <FetchingData v-if="loading" />
   <FileUploader
     v-else
-    v-bind="{ multiple, accept, files }"
+    v-bind="{ multiple, accept, files, auto_upload }"
     @files_uploaded="files_uploaded_event"
   />
 </template>
@@ -21,18 +21,18 @@
     multiple: { type: Boolean, required: true },
     supported_feature: { type: String, required: false, default: null },
     files: { type: Array, required: false, default: [] },
+    auto_upload: { type: Boolean, required: false, default: true },
   })
 
-  const { multiple, supported_feature } = props
-
+  const { auto_upload, multiple, supported_feature } = props
   const accept = ref("")
   const loading = ref(false)
 
   const toggle_loading = useToggle(loading)
 
   function files_uploaded_event(value) {
     if (value.length) {
-      emit("update_values", { files: value })
+      emit("update_values", { files: value, auto_upload: false })
       emit("increment_step")
     }
   }

components/FileUploader.vue

@@ -40,6 +40,7 @@
     multiple: { type: Boolean, required: true },
     accept: { type: String, required: true },
     files: { type: Array, required: false, default: [] },
+    auto_upload: { type: Boolean, required: false, default: false },
   })
 
   const { multiple, accept } = toRefs(props)
@@ -81,7 +82,7 @@
     toggle_loading()
   }
 
-  if (props.files.length) {
+  if (props.files.length && props.auto_upload) {
     files.value = props.files
     upload_files()
   }

test/components/FileSelector.nuxt.test.js

@@ -54,6 +54,7 @@ describe("FileSelector.vue", async () => {
     const v_file_input = file_uploader.findComponent(components.VFileInput)
     await v_file_input.trigger("click")
     const files = [new File(["fake_file"], "fake_file.txt")]
+    const auto_upload = false
     await v_file_input.setValue(files)
     await v_file_input.trigger("change")
     const v_btn = wrapper.findComponent(components.VBtn)
@@ -63,6 +64,7 @@ describe("FileSelector.vue", async () => {
     expect(wrapper.emitted().update_values).toHaveLength(1)
     expect(wrapper.emitted().update_values[0][0]).toEqual({
       files,
+      auto_upload,
     })
   })
 
@@ -88,11 +90,6 @@ describe("FileSelector.vue", async () => {
     })
 
     await flushPromises()
-
-    expect(wrapper.emitted()).toHaveProperty("update_values")
-    expect(wrapper.emitted().update_values).toHaveLength(1)
-    expect(wrapper.emitted().update_values[0][0]).toEqual({
-      files,
-    })
+    expect(wrapper.componentVM.files).toEqual(files)
   })
 })