Merge pull request #27 from mauchede/fix/validation

Reject only 5XX errors

Author: mauchede

src/hydra/fetchJsonLd.js

@@ -22,11 +22,11 @@ export default function fetchJsonLd(url, options = {}) {
 
   return fetch(url, options)
     .then(response => {
-      if (204 === response.status) {
+      const { headers, status } = response;
+      if (204 === status) {
         return Promise.resolve({ response });
       }
-
-      if (false === response.ok || !response.headers.has('Content-Type') || !response.headers.get('Content-Type').includes(jsonLdMimeType)) {
+      if (500 <= status || !headers.has('Content-Type') || !headers.get('Content-Type').includes(jsonLdMimeType)) {
         return Promise.reject({ response });
       }
 

src/hydra/fetchJsonLd.test.js

@@ -46,8 +46,8 @@ test('fetch an error', () => {
 test('fetch an empty document', () => {
   fetch.mockResponseOnce('', {status: 204, statusText: 'No Content', headers: new Headers({'Content-Type': 'text/html'})});
 
-  return fetchJsonLd('/foo.jsonld').then(({response}) => {
-    expect(response.ok).toBe(true);
-    expect(response.body).toBe(undefined);
+  return fetchJsonLd('/foo.jsonld').then(data => {
+    expect(data.response.ok).toBe(true);
+    expect(data.body).toBe(undefined);
   });
 });