fix http 204 no content

dputko · dputko · commit fe78d3a92a7d · 2017-04-05T16:52:50.000+05:00
diff --git a/lib/transports/http.js b/lib/transports/http.js
@@ -123,6 +123,9 @@ class HTTPTransport {
 
     return this.fetch(request.url, request.options)
       .then(function (response) {
+        if (response.status === 204) {
+          return
+        }
         return parseResponse(response, decoders, responseCallback)
           .then(function (data) {
             if (response.ok) {
diff --git a/tests/transports/http.js b/tests/transports/http.js
@@ -302,4 +302,16 @@ describe('Test the HTTPTransport', function () {
         expect(responseCallback).toHaveBeenCalledTimes(1)
       })
   })
+
+  it('should not throw on "204 no content"', function () {
+    const url = 'http://www.example.com/'
+    const link = new document.Link(url, 'delete')
+    const transport = new transports.HTTPTransport({
+      fetch: testUtils.mockedFetch(null, null, 204)
+    })
+    const params = {}
+
+    return transport.action(link, decoders, params)
+      .catch(error => expect(error).toBeNull())
+  })
 })
