fix: make {path, type} enumerable and close #7

Author: Roland Groza

src/file.spec.ts

@@ -1,3 +1,4 @@
+// tslint:disable: forin
 import {COMMON_MIME_TYPES, toFileWithPath} from './file';
 
 describe('toFile()', () => {
@@ -22,6 +23,21 @@ describe('toFile()', () => {
         expect(fileWithPath.path).toBe(path);
     });
 
+    test('{path} is enumerable', () => {
+        const path = '/test/test.json';
+        const file = new File([], 'test.json');
+        const fileWithPath = toFileWithPath(file, path);
+
+        expect(Object.keys(fileWithPath)).toContain('path');
+
+        const keys = [];
+        for (const key in fileWithPath) {
+            keys.push(key);
+        }
+
+        expect(keys).toContain('path');
+    });
+
     it('uses the File {name} as {path} if not provided', () => {
         const name = 'test.json';
         const file = new File([], name);
@@ -50,6 +66,20 @@ describe('toFile()', () => {
         }
     });
 
+    test('{type} is enumerable', () => {
+        const file = new File([], 'test.gif');
+        const fileWithPath = toFileWithPath(file);
+
+        expect(Object.keys(fileWithPath)).toContain('type');
+
+        const keys = [];
+        for (const key in fileWithPath) {
+            keys.push(key);
+        }
+
+        expect(keys).toContain('type');
+    });
+
     it('sets the {type} from extension regardless of case', () => {
         const types = Array.from(COMMON_MIME_TYPES.values());
         const files = Array.from(COMMON_MIME_TYPES.keys())

src/file.ts

@@ -28,7 +28,8 @@ export function toFileWithPath(file: File, path?: string): FileWithPath {
                 ? webkitRelativePath
                 : file.name,
         writable: false,
-        configurable: false
+        configurable: false,
+        enumerable: true
     });
     return f;
 }
@@ -53,7 +54,8 @@ function withMimeType(file: File) {
             Object.defineProperty(file, 'type', {
                 value: type,
                 writable: false,
-                configurable: false
+                configurable: false,
+                enumerable: true
             });
         }
     }