Merge pull request #38 from JohnnyEstilles/hotfix/response-statuscode…

…-default Setting mockResponse.statusCode default to 200
eugef · Mar 5, 2015 · 5619386 · 5619386
2 parents e61617d + 91efe55
commit 5619386
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/lib/mockResponse.js b/lib/mockResponse.js
@@ -50,7 +50,7 @@ function createResponse(options) {
 
     var mockResponse = {};
 
-    mockResponse.statusCode = -1;
+    mockResponse.statusCode = 200;
     mockResponse.cookies = {};
 
     mockResponse.cookie = function(name, value, options) {
@@ -194,11 +194,6 @@ function createResponse(options) {
     mockResponse.json = function(a, b) {
 
         mockResponse.setHeader('Content-Type', 'application/json');
-        // Sets default status code if none has been specify.
-        if(mockResponse.statusCode === -1) {
-            mockResponse.statusCode = 200;
-        }
-
         if (a) {
             if (typeof a === 'number') {
                 mockResponse.statusCode = a;

diff --git a/test/test-mockResponse.js b/test/test-mockResponse.js
@@ -20,7 +20,7 @@ exports['object - Simple verification'] = function (test) {
 
 exports['object - Data Initialization'] = function (test) {
   var response = httpMocks.createResponse();
-  test.equal(-1, response.statusCode);
+  test.equal(200, response.statusCode);
   test.equal("", response._getData());
   test.ok(!response._isUTF8());
   test.ok(!response._isEndCalled());