Merge pull request #28 from peter-vasarhelyi/parse-json

feat(wrapper): parse response if content-type contains application/json

wrapper.js

@@ -63,7 +63,7 @@ RequestWrapper.prototype = {
         return reject(new SuiteRequestError('Empty http response', 500, response.statusMessage));
       }
 
-      if (response.headers['content-type'] === 'application/json') {
+      if (this._isJsonResponse(response)) {
         try {
           response.body = JSON.parse(response.body);
         } catch (ex) {
@@ -91,6 +91,10 @@ RequestWrapper.prototype = {
 
   },
 
+  _isJsonResponse(response) {
+    return response.headers['content-type'] &&
+      response.headers['content-type'].indexOf('application/json') !== -1;
+  },
 
   _getLogParameters: function(extraParametersToLog) {
     var requestParametersToLog = _.pick(this.requestOptions, ['method', 'host', 'url']);

wrapper.spec.js

@@ -151,6 +151,16 @@ describe('Wrapper', function() {
       }
       throw new Error('Error should have been thrown');
     });
+
+    it('should parse JSON if content-type header contains charset too', function *() {
+      const testJson = { text: 'Test JSON text' };
+      apiResponse.headers['content-type'] = 'application/json; charset=utf-8';
+      apiResponse.body = JSON.stringify(testJson);
+
+      const response = yield wrapper.send();
+
+      expect(response.body).to.eql(testJson);
+    });
   });