perf(fetch): add content body, when throw Error

1. Throw content with Error, check result.body on null or undefined

Author: Alexander Vakhitov

.gitignore

@@ -42,4 +42,6 @@ jspm_packages
 
 .npmrc
 dist
-umd
+umd
+.idea
+.DS_Store

src/callAPIMethod.js

@@ -26,12 +26,16 @@ export default function callAPI(
 
     return fetch(url, {...defaults.fetchOptions, ...fetchOptions, ...options})
         .then( result => {
+            const promiseCallback = result.body ? result[method]() : new Promise(resolve => resolve(result.body));
 
-            if (!result.ok) {
-                throw new APIError(result.status, result.statusText);
+            if (result.ok) {
+                return promiseCallback;
             }
 
-            return result[method]();
+            return promiseCallback
+                .then( body => {
+                    throw new APIError(result.status, result.statusText, body);
+                });
         })
         .catch( error => error);
 }

src/createAPI.js

@@ -5,8 +5,8 @@ import applyMiddleware from './applyMiddleware';
 const createAPI = (
     resources = {},
     middleware = [],
-    APINamespace = '',
-    fetchOptions = {}
+    APINamespace,
+    fetchOptions
 ) => Object.keys(resources).reduce( (api, resourceId) => {
     api[resourceId] = Object.keys(resources[resourceId].methods)
         .reduce( (resource, method) => {

src/error.js

@@ -1,6 +1,7 @@
-function APIError(code, message) {
+function APIError(code, message, body) {
     this.code = code;
     this.message = (message || '');
+    this.body = body;
 }
 
 Object.setPrototypeOf(APIError.prototype, Error.prototype);

test/callAPIMethod.spec.js

@@ -13,9 +13,12 @@ const ResponseGood = new Response(
     }
 );
 
-const ResponseBad = new Response(null, {
-    status: 500
-});
+const ResponseBad = new Response(
+    null,
+    {
+        status: 500
+    }
+);
 
 test.serial('default params', async t => {
     fetchMock.get(matcher, {});
@@ -41,7 +44,6 @@ test.serial('200 reponse', async t => {
     fetchMock.restore();
 });
 
-
 test.serial('500 reponse', async t => {
     fetchMock.get(matcher, ResponseBad);
 
@@ -79,7 +81,7 @@ test.serial('fetch options', async t => {
         credentials: 'omit',
         method: 'POST',
         mode: 'cors'
-    }), 'correct options';
+    }, 'correct options');
     t.true(spyResponseGood.calledOnce);
 
     spyResponseGood.restore();

test/createAPI.spec.js

@@ -33,7 +33,7 @@ test('supports resource prefix insted of namespace', t => {
         }
     };
     const resources = { user };
-    const API = createAPI(resources, [], 'https://example.com');
+    const API = createAPI(resources, [], 'https://example.com', {});
 
     API.user.get(1);
 
@@ -65,7 +65,7 @@ test('general behaviour', t => {
 
     const middleware = [ _ => _, _ => _];
 
-    const API = createAPI(resources, middleware, 'https://example.com');
+    const API = createAPI(resources, middleware, 'https://example.com', {});
 
     t.deepEqual(Object.keys(API), ['user', 'project'], 'resources');
 

yarn.lock

@@ -808,6 +808,14 @@ babel-polyfill@^6.22.0:
     core-js "^2.4.0"
     regenerator-runtime "^0.10.0"
 
+babel-polyfill@^6.23.0:
+  version "6.23.0"
+  resolved "http://storage.mds.yandex.net/get-npm/45674/babel-polyfill-6.23.0.tgz#8364ca62df8eafb830499f699177466c3b03499d"
+  dependencies:
+    babel-runtime "^6.22.0"
+    core-js "^2.4.0"
+    regenerator-runtime "^0.10.0"
+
 babel-preset-es2015@^6.16.0:
   version "6.22.0"
   resolved "https://registry.yarnpkg.com/babel-preset-es2015/-/babel-preset-es2015-6.22.0.tgz#af5a98ecb35eb8af764ad8a5a05eb36dc4386835"