Catch -1 body or response

Author: Stefano Sala

lib/request.js

@@ -42,7 +42,7 @@ class Request {
     return oauth.authorize(req);
   }
 
-  error(message, response, body, error) {
+  error(message, response, error) {
     if (!error) error = new Error(message);
     error.response = {
       statusCode: response.statusCode
@@ -98,10 +98,20 @@ class Request {
           return;
         }
 
+        if (response === -1 || body === -1) {
+          const msg = 'A meaningless error has occurred, returning -1. ' +
+            'This could be the result of an incorrect folder setup.';
+          const error = this.error(msg, response);
+          logger.error(msg);
+          reject(error);
+          cb(error, body, response);
+          return;
+        }
+
         // Handle error codes between 400 & 600
         if (response.statusCode >= 400 && response.statusCode < 600) {
           const msg = `Request failed with code: ${response.statusCode}`;
-          const error = this.error(msg, response, body);
+          const error = this.error(msg, response);
 
           logger.error(msg);
           reject(error);
@@ -127,7 +137,7 @@ class Request {
 
           if (json.hasOwnProperty('error') || json.hasOwnProperty('errors')) {
             const errorMessage = json.error || JSON.stringify(json.errors);
-            const serverError = this.error(errorMessage, response, body);
+            const serverError = this.error(errorMessage, response);
             cb(serverError, body, response);
             reject(serverError);
             return;
@@ -140,7 +150,7 @@ class Request {
         } catch (e) {
 
           const malformedMsg = `Error parsing response body: ${e.message || e}`;
-          const malformedError = this.error(malformedMsg, body, response, e);
+          const malformedError = this.error(malformedMsg, response, e);
 
           logger.error(malformedMsg);
           cb(malformedError, body, response);