Merge pull request #24 from Wilzi/master

feat(request-error): send response statusMessage in error if exists

wrapper.js

@@ -60,7 +60,7 @@ RequestWrapper.prototype = {
 
       if (!this.requestOptions.allowEmptyResponse && !response.body) {
         logger.error('server_error', 'empty response data');
-        return reject(new SuiteRequestError('Empty http response', 500));
+        return reject(new SuiteRequestError('Empty http response', 500, response.statusMessage));
       }
 
       if (response.headers['content-type'] === 'application/json') {

wrapper.spec.js

@@ -117,6 +117,20 @@ describe('Wrapper', function() {
           expect(err).to.be.an.instanceof(SuiteRequestError);
           expect(err.message).to.eql('Empty http response');
           expect(err.code).to.eql(500);
+          expect(err.data).to.eql({ replyText: 'Empty http response' });
+          return;
+        }
+        throw new Error('Error should have been thrown');
+      });
+
+      it('should throw error with status message if response body is empty and status message exists', function *() {
+        apiResponse.body = '';
+        apiResponse.statusMessage = 'dummy status message';
+
+        try {
+          yield wrapper.send();
+        } catch (err) {
+          expect(err.data).to.eql(apiResponse.statusMessage);
           return;
         }
         throw new Error('Error should have been thrown');