fix(core/file-input): value not set on file select

It did work only on file drop
gnucoop · Jun 3, 2020 · 13de9fb · 13de9fb
1 parent 843e280
commit 13de9fb
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 2 deletions.
diff --git a/src/core/file-input/file-input.spec.ts b/src/core/file-input/file-input.spec.ts
@@ -42,6 +42,25 @@ describe('AjfFileInput', () => {
     expect(fileInput.value).toBeNull();
   });
 
+  it('should set its value on file selection.', async () => {
+    const fileInput = fixture.componentInstance;
+    const fileList = fileInput._nativeInput.nativeElement.files as FileList;
+    fileList.item = idx => {
+      if (idx === 0) {
+        return emptyPngFile;
+      }
+      return null;
+    };
+    const lastValue = fileInput.valueChange.pipe(shareReplay(1));
+    fileInput.onSelectFile();
+    await lastValue.pipe(take(1)).toPromise();
+    const {name, size, type, content} = fileInput.value as AjfFile;
+    expect(name).toEqual(emptyPngFile.name);
+    expect(size).toEqual(emptyPngFile.size);
+    expect(type).toEqual(emptyPngFile.type);
+    expect(content).toEqual(`data:image/png;base64,${emptyPng}`);
+  });
+
   it('should display the default drop message', () => {
     const el = fixture.nativeElement as HTMLElement;
     const dropMessages = el.getElementsByClassName('ajf-drop-message');

diff --git a/src/core/file-input/file-input.ts b/src/core/file-input/file-input.ts
@@ -149,10 +149,14 @@ export class AjfFileInput implements ControlValueAccessor {
 
   onSelectFile(): void {
     const files = this._nativeInput.nativeElement.files;
-    if (files == null || files.length !== 1 || files[0]) {
+    if (files == null) {
       return;
     }
-    this._processFileUpload(files[0]);
+    const file = files.item(0);
+    if (file == null) {
+      return;
+    }
+    this._processFileUpload(files.item(0) as File);
   }
 
   registerOnChange(fn: any): void {