Merge pull request #156 from derekpitt/empty-response

fix(http-response-message): handle empty responses
aurelia · Nov 8, 2016 · f44c814 · f44c814
2 parents 27a84bb + bae5188
commit f44c814
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/http-response-message.js b/src/http-response-message.js
@@ -101,7 +101,7 @@ export class HttpResponseMessage {
         return this._content;
       }
 
-      if (this.response === undefined || this.response === null) {
+      if (this.response === undefined || this.response === null || this.response == '') {
         this._content = this.response;
         return this._content;
       }

diff --git a/test/http-response-message.spec.js b/test/http-response-message.spec.js
@@ -109,6 +109,11 @@ describe("HttpResponseMessage", () => {
       expect(httpResponse.content).toBeNull();
     });
 
+    it("will return a blank string if response is blank", () => {
+      let httpResponse = new HttpResponseMessage(null, {response: '',responseText:''});
+      expect(httpResponse.content).toBe('');
+    });
+
     it("will JSON.parse if the response type is 'json'", () => {
       let response = {}, reviver = {}, content = {};
       let parseSpy = spyOn(JSON, 'parse').and.returnValue(content);