add sendStatus support

jeremydaly · Dec 26, 2018 · 2990ada · 2990ada
1 parent 20303b2
commit 2990ada
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 3 deletions.
diff --git a/lib/response.js b/lib/response.js
@@ -372,7 +372,10 @@ class RESPONSE {
 
 
 
-  // TODO: sendStatus
+  // Convenience method for sending status codes
+  sendStatus(status) {
+    this.status(status).send(UTILS.statusLookup(status))
+  }
 
 
   // Convenience method for setting CORS headers
@@ -456,10 +459,10 @@ class RESPONSE {
         : { headers: UTILS.stringifyHeaders(this._headers) },
       {
         statusCode: this._statusCode,
-        statusDescription: this._request.interface === 'alb' ? `${this._statusCode} ${UTILS.statusLookup(this._statusCode)}` : undefined,
         body: this._request.method === 'HEAD' ? '' : UTILS.encodeBody(body,this._serializer),
         isBase64Encoded: this._isBase64
-      }
+      },
+      this._request.interface === 'alb' ? { statusDescription:  `${this._statusCode} ${UTILS.statusLookup(this._statusCode)}` } : {} 
     )
 
     // Trigger the callback function

diff --git a/test/responses.js b/test/responses.js
@@ -52,6 +52,14 @@ api.get('/testJSONPResponse', function(req,res) {
   res.jsonp({ foo: 'bar' })
 })
 
+api.get('/testSendStatus', function(req,res) {
+  res.sendStatus(200)
+})
+
+api.get('/testSendStatus403', function(req,res) {
+  res.sendStatus(403)
+})
+
 // Secondary route
 api2.get('/testJSONPResponse', function(req,res) {
   res.jsonp({ foo: 'bar' })
@@ -141,6 +149,18 @@ describe('Response Tests:', function() {
     expect(result).to.deep.equal({ multiValueHeaders: { 'content-type': ['application/json'] }, statusCode: 200, body: '', isBase64Encoded: false })
   }) // end it
 
+  it('sendStatus 200', async function() {
+    let _event = Object.assign({},event,{ path: '/testSendStatus'})
+    let result = await new Promise(r => api.run(_event,{},(e,res) => { r(res) }))
+    expect(result).to.deep.equal({ multiValueHeaders: { 'content-type': ['application/json'] }, statusCode: 200, body: 'OK', isBase64Encoded: false })
+  }) // end it
+
+  it('sendStatus 403', async function() {
+    let _event = Object.assign({},event,{ path: '/testSendStatus403'})
+    let result = await new Promise(r => api.run(_event,{},(e,res) => { r(res) }))
+    expect(result).to.deep.equal({ multiValueHeaders: { 'content-type': ['application/json'] }, statusCode: 403, body: 'Forbidden', isBase64Encoded: false })
+  }) // end it
+
   it('JSONP response (default callback)', async function() {
     let _event = Object.assign({},event,{ path: '/testJSONPResponse' })
     let result = await new Promise(r => api.run(_event,{},(e,res) => { r(res) }))