Fix unit tests after earlier update

The `s3.upload` was using a promise version internally now, so the tests needed to be updated to match that API.
mediamonks · Jul 6, 2022 · 6c75e93 · 6c75e93
1 parent b538c98
commit 6c75e93
Show file tree

Hide file tree

Showing 2 changed files with 48 additions and 20 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "s3-batch-upload",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "description": "Super fast batched S3 folder uploads from CLI or API.",
   "main": "./index.js",
   "types": "./index.d.ts",

diff --git a/test/Uploader.spec.ts b/test/Uploader.spec.ts
@@ -15,8 +15,12 @@ describe('Uploader', () => {
       this.timeout(10000);
 
       const s3 = {
-        upload(_, cb) {
-          cb(null);
+        upload() {
+          return {
+            async promise() {
+              return null;
+            }
+          }
         }
       };
       spy(s3, "upload");
@@ -50,8 +54,12 @@ describe('Uploader', () => {
       this.timeout(10000);
 
       const s3 = {
-        upload(_, cb) {
-          cb(null);
+        upload() {
+          return {
+            async promise() {
+              return null;
+            }
+          }
         }
       };
       spy(s3, "upload");
@@ -87,8 +95,12 @@ describe('Uploader', () => {
       this.timeout(5000);
 
       const s3 = {
-        upload(_, cb) {
-          cb(null);
+        upload() {
+          return {
+            async promise() {
+              return null;
+            }
+          }
         }
       };
       spy(s3, "upload");
@@ -123,13 +135,17 @@ describe('Uploader', () => {
         this.timeout(10000);
 
         const s3 = {
-          upload(_, cb) {
-            cb(null);
+          upload() {
+            return {
+              async promise() {
+                return null;
+              }
+            }
           },
-          headObject(_) {
+          headObject() {
             return {
-              promise() {
-                return Promise.resolve(null)
+              async promise() {
+                return null;
               }
             }
           }
@@ -161,15 +177,19 @@ describe('Uploader', () => {
         this.timeout(10000);
 
         const s3 = {
-          upload(_, cb) {
-            cb(null);
+          upload() {
+            return {
+              async promise() {
+                return null;
+              }
+            }
           },
           headObject(_) {
             return {
-              promise() {
+              async promise() {
                 const err: any = new Error()
                 err.code = 'NotFound'
-                return Promise.reject(err)
+                throw err;
               }
             }
           }
@@ -248,8 +268,12 @@ describe('Uploader', () => {
         this.timeout(5000);
 
         const s3 = {
-          upload(_, cb) {
-            cb(null);
+          upload() {
+            return {
+              async promise() {
+                return null;
+              }
+            }
           }
         };
         spy(s3, "upload");
@@ -275,8 +299,12 @@ describe('Uploader', () => {
         this.timeout(10000);
 
         const s3 = {
-          upload(_, cb) {
-            cb(null);
+          upload() {
+            return {
+              async promise() {
+                return null;
+              }
+            }
           }
         };
         spy(s3, "upload");