Move test for client/Response.parse in order not to confuse code coverage

Author: Olle Johannesson

spec/amadeus/client/response.test.js

@@ -47,26 +47,27 @@ describe('Response', () => {
         expect(response.data).toEqual('b');
       });
 
-      it('should not parse if not json', () => {
-        response.contentType = 'plain/text';
+      it('should handle badly formed json', () => {
+        response.addChunk('{ "a" : ');
         response.parse();
         expect(response.parsed).toBeFalsy();
         expect(response.result).toBeNull();
         expect(response.data).toBeNull();
       });
 
-      it('should not parse if status code is 204', () => {
-        response.contentType = 'plain/text';
-        response.statusCode = 204;
+      it('should not parse if not json', () => {
+        response.headers['content-type'] = 'plain/text';
         response.parse();
         expect(response.parsed).toBeFalsy();
         expect(response.result).toBeNull();
         expect(response.data).toBeNull();
       });
 
-      it('should handle badly formed json', () => {
-        response.addChunk('{ "a" : ');
-        response.parse();
+      it('should not parse if status code is 204', () => {
+        response.contentType = 'plain/text';
+        response.statusCode = 204;
+        const parsingReturnValue = response.parse();
+        expect(parsingReturnValue).toBeUndefined();
         expect(response.parsed).toBeFalsy();
         expect(response.result).toBeNull();
         expect(response.data).toBeNull();