Merge pull request #450 from vlucas/inspectOnFailure

Fix #418 - Add 'inspectOnFailure' true by default

package.json

@@ -5,7 +5,7 @@
   "homepage": "http://frisbyjs.com",
   "author": "Vance Lucas <vance@vancelucas.com>",
   "license": {
-    "type": "BSD"
+    "type": "BSD-3-Clause"
   },
   "repository": {
     "type": "git",

src/frisby.js

@@ -19,6 +19,7 @@ const _globalSetupDefaults = {
       'Content-Type': 'application/json',
       'User-Agent': 'frisby/' + version + ' (+https://github.com/vlucas/frisby)',
     },
+    inspectOnFailure: true,
     timeout: 5000,
   },
 };

src/frisby/spec.js

@@ -228,7 +228,7 @@ class FrisbySpec {
       } else {
         return response;
       }
-    }, err => onRejected ? onRejected(err) : Promise.reject(err));
+    }, this._handleError(onRejected));
     return this;
   }
 
@@ -247,10 +247,24 @@ class FrisbySpec {
    */
   catch(onRejected) {
     this._ensureHasFetched();
-    this._fetch = this._fetch.catch(err => onRejected ? onRejected(err) : Promise.reject(err));
+    this._fetch = this._fetch.catch(this._handleError(onRejected));
     return this;
   }
 
+  _handleError(onRejected) {
+    return (err) => {
+      if (onRejected) {
+        return onRejected(err);
+      }
+
+      if (this._setupDefaults.request.inspectOnFailure) {
+        this.inspectLog("\nFAILURE JSON:", JSON.stringify(this._response.json, null, 4));
+      }
+
+      return Promise.reject(err);
+    }
+  }
+
   /**
    * Return internal promise used by Frisby.js
    * Note: Using this will break the chainability of Frisby.js method calls
@@ -320,10 +334,8 @@ class FrisbySpec {
     });
   }
 
-  inspectLog() {
-    let params = Array.prototype.slice.call(arguments);
-
-    console.log.apply(null, params); // eslint-disable-line no-console
+  inspectLog(...args) {
+    console.log.call(null, ...args); // eslint-disable-line no-console
     return this;
   }