feat: update dependencies

Author: Roland Groza

.github/workflows/test.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node_version: [10, 12, 14]
+        node_version: [12, 14, 16]
 
     steps:
       - uses: actions/checkout@v2

jest.config.js

@@ -1,7 +1,8 @@
 module.exports = {
+    testEnvironment: "jsdom",
     globals: {
         'ts-jest': {
-            tsConfig: './tsconfig.spec.json',
+            tsconfig: './tsconfig.spec.json',
             diagnostics: true
         }
     },

package.json

@@ -46,27 +46,27 @@
     "test": "jest --watch"
   },
   "dependencies": {
-    "tslib": "^2.0.3"
+    "tslib": "^2.4.0"
   },
   "devDependencies": {
-    "@babel/core": "^7.12.3",
-    "@rollup/plugin-commonjs": "^15.1.0",
-    "@rollup/plugin-node-resolve": "^9.0.0",
-    "@types/jest": "^26.0.15",
-    "@types/node": "^14.14.2",
-    "babel-jest": "^26.6.0",
-    "camelcase": "^6.1.0",
-    "jest": "^26.6.0",
-    "jest-environment-jsdom": "^26.6.0",
+    "@babel/core": "^7.17.10",
+    "@rollup/plugin-commonjs": "^22.0.0",
+    "@rollup/plugin-node-resolve": "^13.2.1",
+    "@types/jest": "^27.4.1",
+    "@types/node": "^17.0.30",
+    "babel-jest": "^28.0.3",
+    "camelcase": "^6.3.0",
+    "jest": "^28.0.3",
+    "jest-environment-jsdom": "^28.0.2",
     "npm-run-all": "^4.1.5",
-    "rollup": "^2.32.1",
+    "rollup": "^2.70.2",
     "rollup-plugin-sourcemaps": "^0.6.3",
     "rollup-plugin-terser": "^7.0.2",
-    "ts-jest": "^26.4.1",
+    "ts-jest": "next",
     "tslint": "^6.1.3",
-    "typescript": "^4.0.3"
+    "typescript": "^4.6.4"
   },
   "engines": {
-    "node": ">= 10"
+    "node": ">= 12"
   }
 }

src/file-selector.spec.ts

@@ -209,7 +209,7 @@ it('filters thumbnail cache files', async () => {
     expect(items).toHaveLength(0);
 });
 
-it('should throw if reading dir entries fails', async done => {
+it('should throw if reading dir entries fails', done => {
     const mockFiles = sortFiles([
         createFile('ping.json', {ping: true}),
         createFile('pong.json', {pong: true})
@@ -222,15 +222,12 @@ it('should throw if reading dir entries fails', async done => {
         ], 1, 1))
     ]);
 
-    try {
-        await fromEvent(evt);
-        done.fail('Getting the files should have failed');
-    } catch (err) {
-        done();
-    }
+    fromEvent(evt)
+        .then(() => done.fail('Getting the files should have failed'))
+        .catch(() => done());
 });
 
-it('should throw if reading file entry fails', async done => {
+it('should throw if reading file entry fails', done => {
     const mockFiles = sortFiles([
         createFile('ping.json', {ping: true}),
         createFile('pong.json', {pong: true})
@@ -243,24 +240,18 @@ it('should throw if reading file entry fails', async done => {
         ], 1, 1))
     ]);
 
-    try {
-        await fromEvent(evt);
-        done.fail('Getting the files should have failed');
-    } catch (err) {
-        done();
-    }
+    fromEvent(evt)
+        .then(() => done.fail('Getting the files should have failed'))
+        .catch(() => done());
 });
 
-it('should throw if DataTransferItem is not a File', async done => {
+it('should throw if DataTransferItem is not a File', done => {
     const item = dataTransferItem(null, 'file');
     const evt = dragEvtFromFilesAndItems([], [item]);
 
-    try {
-        await fromEvent(evt);
-        done.fail('Getting the files should have failed');
-    } catch (err) {
-        done();
-    }
+    fromEvent(evt)
+        .then(() => done.fail('Getting the files should have failed'))
+        .catch(() => done());
 });
 
 
@@ -402,7 +393,7 @@ function inputEvtFromFiles(...files: File[]): Event {
             if (p === 'target') {
                 return input;
             }
-            return t[p];
+            return (t as any)[p];
         }
     });
 }

src/file.spec.ts

@@ -56,6 +56,7 @@ describe('toFile()', () => {
     });
 
     it('uses the File {webkitRelativePath} as {path} if it exists', () => {
+        const name = 'test.json';
         const path = 'test/test.json';
         const file = new File([], name);
         Object.defineProperty(file, 'webkitRelativePath', {
@@ -116,7 +117,7 @@ describe('toFile()', () => {
                 expect(d).toEqual(data);
                 done();
             } catch (e) {
-                done.fail(e);
+                done.fail(e as Error);
             }
         };
 

src/file.ts

@@ -88,7 +88,7 @@ export const COMMON_MIME_TYPES = new Map([
 export function toFileWithPath(file: FileWithPath, path?: string): FileWithPath {
     const f = withMimeType(file);
     if (typeof f.path !== 'string') { // on electron, path is already set to the absolute path
-        const {webkitRelativePath} = file as FileWithWebkitPath;
+        const {webkitRelativePath} = file;
         Object.defineProperty(f, 'path', {
             value: typeof path === 'string'
                 ? path
@@ -107,19 +107,10 @@ export function toFileWithPath(file: FileWithPath, path?: string): FileWithPath
     return f;
 }
 
-interface DOMFile extends Blob {
-    readonly lastModified: number;
-    readonly name: string;
-}
-
-export interface FileWithPath extends DOMFile {
+export interface FileWithPath extends File {
     readonly path?: string;
 }
 
-interface FileWithWebkitPath extends File {
-    readonly webkitRelativePath?: string;
-}
-
 function withMimeType(file: FileWithPath) {
     const {name} = file;
     const hasExtension = name && name.lastIndexOf('.') !== -1;

tsconfig.spec.json

@@ -3,7 +3,8 @@
     "compilerOptions": {
         "module": "commonjs",
         "target": "es5",
-        "suppressImplicitAnyIndexErrors": true
+        "suppressImplicitAnyIndexErrors": true,
+        "esModuleInterop": true
     },
     "include": [
         "src/**/*.ts"