v1.2.2 (#11)

* Added test duplicating issue in #10.

* Correct linting errors.

* Resolve error with downloading thumbnails.

* 1.2.2

Author: jeremy-j-ackso

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "phaxio-official",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "description": "The official Node.JS library for Phaxio API v2.1.0. https://phaxio.com",
   "main": "index.js",
   "scripts": {

package.json

@@ -24,7 +24,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 };

src/account/index.js

@@ -1 +1 @@
-module.exports = errMessage => new Error(`PhaxioAPIError: ${errMessage}`);
+module.exports = (errMessage) => new Error(`PhaxioAPIError: ${errMessage}`);

src/error-handler/index.js

@@ -110,7 +110,7 @@ module.exports = class {
           // eslint-disable-next-line max-len
           return resolve(new Fax(this.apiKey, this.apiSecret, this.url, res.success, res.message, res.data));
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 
@@ -142,7 +142,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 
@@ -181,7 +181,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 };

src/faxes/index.js

@@ -14,7 +14,7 @@ function cancel(url, id, auth, agentOptions) {
         if (!res.success) return reject(errorHandler(res.message));
         return resolve(res);
       })
-      .catch(err => reject(err));
+      .catch((err) => reject(err));
   });
 }
 
@@ -38,7 +38,7 @@ function resend(url, id, auth, options = { id: null, callback_url: null }, agent
         if (!res.success) return reject(errorHandler(res.message));
         return resolve(res);
       })
-      .catch(err => reject(err));
+      .catch((err) => reject(err));
   });
 }
 
@@ -55,7 +55,7 @@ function testDelete(url, id, auth, agentOptions) {
         if (!res.success) return reject(errorHandler(res.message));
         return resolve(res);
       })
-      .catch(err => reject(err));
+      .catch((err) => reject(err));
   });
 }
 
@@ -72,7 +72,7 @@ function getInfo(url, id, auth, agentOptions) {
         if (!res.success) return reject(errorHandler(res.message));
         return resolve(res);
       })
-      .catch(err => reject(err));
+      .catch((err) => reject(err));
   });
 }
 
@@ -92,11 +92,11 @@ function getFile(url, id, auth, options = { id: null, thumbnail: null }, agentOp
       encoding: null,
     };
 
-    if (thumbnail !== null) req.qs.thumbnail = thumbnail;
+    if (thumbnail !== null) req.qs = { thumbnail };
 
     request(req)
-      .then(response => resolve(response))
-      .catch(err => reject(err));
+      .then((response) => resolve(response))
+      .catch((err) => reject(err));
   });
 }
 
@@ -113,7 +113,7 @@ function deleteFile(url, id, auth, agentOptions) {
         if (!res.success) return reject(errorHandler(res.message));
         return resolve(res);
       })
-      .catch(err => reject(err));
+      .catch((err) => reject(err));
   });
 }
 

src/faxes/shared-methods.js

@@ -33,7 +33,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 
@@ -60,7 +60,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 };

src/phax-code/index.js

@@ -24,7 +24,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 
@@ -50,7 +50,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 
@@ -67,7 +67,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 
@@ -93,7 +93,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 };

src/phone-number/index.js

@@ -36,7 +36,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 
@@ -55,7 +55,7 @@ module.exports = class {
           if (!res.success) return reject(errorHandler(res.message));
           return resolve(res);
         })
-        .catch(err => reject(err));
+        .catch((err) => reject(err));
     });
   }
 };

src/public/index.js

@@ -87,6 +87,14 @@ describe('shared methods', function () { // eslint-disable-line func-names
     });
 
     describe('function: getFile', () => {
+      it('should get a fax\'s thumbnail', () => {
+        return getFile(url, firstFax, auth, { id: firstFax, thumbnail: 'l' })
+          .then((gf) => {
+            expect(Buffer.isBuffer(gf)).to.be.ok();
+          })
+          .catch((err) => { throw err; });
+      });
+
       it('should get a fax\'s file', () => {
         return getFile(url, firstFax, auth, { id: firstFax })
           .then((gf) => {