[fix] content-type check.

godaddy · Oct 9, 2018 · ead3029 · ead3029
1 parent 00c35f7
commit ead3029
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/fetchUtils.js b/src/fetchUtils.js
@@ -24,12 +24,13 @@ handlers.decode = function (response) {
   if (contentType.includes('application/json')) {
     return response.json()
       .then(data => handlers.finish(response, data));
-  } else if (contentType.includes('text/html') || contentType.includes('text/plain')) {
+  }
+  try {
     return response.text()
       .then(data => handlers.finish(response, data));
+  } catch (err) {
+    return Promise.reject(`Content-type ${contentType} not supported`);
   }
-
-  return Promise.reject(`Content-type ${contentType} not supported`);
 };
 
 /**

diff --git a/tests/fetchUtils.spec.js b/tests/fetchUtils.spec.js
@@ -138,6 +138,9 @@ describe('fetchUtils', () => {
     });
 
     it('rejects for unhandled content types', (done) => {
+      mockResponse.text = jest.fn().mockImplementation(() => {
+        throw new Error('`text` not implemented');
+      });
       mockResponse.headers.set('content-type', 'some/type');
       handlers.decode(mockResponse).catch(data => {
         expect(data).toContain('not supported');