setHeader should return this (#268)

eugef · Mar 13, 2023 · 7665e06 · 7665e06
1 parent ee5fdab
commit 7665e06
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/mockResponse.js b/lib/mockResponse.js
@@ -638,7 +638,7 @@ function createResponse(options) {
      */
     mockResponse.setHeader = function(name, value) {
         mockResponse._headers[name.toLowerCase()] = value;
-        return value;
+        return this;
     };
 
     /**

diff --git a/test/lib/mockResponse.spec.js b/test/lib/mockResponse.spec.js
@@ -967,6 +967,10 @@ describe('mockResponse', function () {
         expect(response.setHeader).to.throw;
       });
 
+      it('should return this', function() {
+        expect(response.setHeader('name', 'value')).to.be(response);
+      })
+
     });
 
     describe('.getHeader()', function () {