feat: add 500 response mapping

theburningmonk · theburningmonk · commit 923b40ddf9e7 · 2019-08-17T09:53:50.000+02:00
diff --git a/lib/apiGateway/methods.js b/lib/apiGateway/methods.js
@@ -14,6 +14,11 @@ module.exports = {
             ResponseParameters: {},
             ResponseModels: {},
             StatusCode: 400
+          },
+          {
+            ResponseParameters: {},
+            ResponseModels: {},
+            StatusCode: 500
           }
         ]
       }
diff --git a/lib/package/kinesis/compileMethodsToKinesis.test.js b/lib/package/kinesis/compileMethodsToKinesis.test.js
@@ -118,7 +118,8 @@ describe('#compileMethodsToKinesis()', () => {
           },
           MethodResponses: [
             { ResponseParameters: {}, ResponseModels: {}, StatusCode: 200 },
-            { ResponseParameters: {}, ResponseModels: {}, StatusCode: 400 }
+            { ResponseParameters: {}, ResponseModels: {}, StatusCode: 400 },
+            { ResponseParameters: {}, ResponseModels: {}, StatusCode: 500 }
           ]
         }
       }
@@ -238,6 +239,11 @@ describe('#compileMethodsToKinesis()', () => {
               ResponseParameters: { 'method.response.header.Access-Control-Allow-Origin': "'*'" },
               ResponseModels: {},
               StatusCode: 400
+            },
+            {
+              ResponseParameters: { 'method.response.header.Access-Control-Allow-Origin': "'*'" },
+              ResponseModels: {},
+              StatusCode: 500
             }
           ]
         }
diff --git a/lib/package/sqs/compileMethodsToSqs.test.js b/lib/package/sqs/compileMethodsToSqs.test.js
@@ -8,7 +8,7 @@ const ServerlessApigatewayServiceProxy = require('./../../index')
 chai.use(require('chai-as-promised'))
 const expect = require('chai').expect
 
-describe('#compileIamRoleToSqs()', () => {
+describe('#compileMethodsToSqs()', () => {
   let serverless
   let serverlessApigatewayServiceProxy
 
@@ -96,7 +96,8 @@ describe('#compileIamRoleToSqs()', () => {
           },
           MethodResponses: [
             { ResponseParameters: {}, ResponseModels: {}, StatusCode: 200 },
-            { ResponseParameters: {}, ResponseModels: {}, StatusCode: 400 }
+            { ResponseParameters: {}, ResponseModels: {}, StatusCode: 400 },
+            { ResponseParameters: {}, ResponseModels: {}, StatusCode: 500 }
           ]
         }
       }
@@ -196,6 +197,11 @@ describe('#compileIamRoleToSqs()', () => {
               ResponseParameters: { 'method.response.header.Access-Control-Allow-Origin': "'*'" },
               ResponseModels: {},
               StatusCode: 400
+            },
+            {
+              ResponseParameters: { 'method.response.header.Access-Control-Allow-Origin': "'*'" },
+              ResponseModels: {},
+              StatusCode: 500
             }
           ]
         }

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,11 @@ module.exports = {`
`14`	`14`	`ResponseParameters: {},`
`15`	`15`	`ResponseModels: {},`
`16`	`16`	`StatusCode: 400`
	`17`	`+ },`
	`18`	`+ {`
	`19`	`+ ResponseParameters: {},`
	`20`	`+ ResponseModels: {},`
	`21`	`+ StatusCode: 500`
`17`	`22`	`}`
`18`	`23`	`]`
`19`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,8 @@ describe('#compileMethodsToKinesis()', () => {`
`118`	`118`	`},`
`119`	`119`	`MethodResponses: [`
`120`	`120`	`{ ResponseParameters: {}, ResponseModels: {}, StatusCode: 200 },`
`121`		`- { ResponseParameters: {}, ResponseModels: {}, StatusCode: 400 }`
	`121`	`+ { ResponseParameters: {}, ResponseModels: {}, StatusCode: 400 },`
	`122`	`+ { ResponseParameters: {}, ResponseModels: {}, StatusCode: 500 }`
`122`	`123`	`]`
`123`	`124`	`}`
`124`	`125`	`}`
`@@ -238,6 +239,11 @@ describe('#compileMethodsToKinesis()', () => {`
`238`	`239`	`ResponseParameters: { 'method.response.header.Access-Control-Allow-Origin': "'*'" },`
`239`	`240`	`ResponseModels: {},`
`240`	`241`	`StatusCode: 400`
	`242`	`+ },`
	`243`	`+ {`
	`244`	`+ ResponseParameters: { 'method.response.header.Access-Control-Allow-Origin': "'*'" },`
	`245`	`+ ResponseModels: {},`
	`246`	`+ StatusCode: 500`
`241`	`247`	`}`
`242`	`248`	`]`
`243`	`249`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ const ServerlessApigatewayServiceProxy = require('./../../index')`
`8`	`8`	`chai.use(require('chai-as-promised'))`
`9`	`9`	`const expect = require('chai').expect`
`10`	`10`
`11`		`-describe('#compileIamRoleToSqs()', () => {`
	`11`	`+describe('#compileMethodsToSqs()', () => {`
`12`	`12`	`let serverless`
`13`	`13`	`let serverlessApigatewayServiceProxy`
`14`	`14`
`@@ -96,7 +96,8 @@ describe('#compileIamRoleToSqs()', () => {`
`96`	`96`	`},`
`97`	`97`	`MethodResponses: [`
`98`	`98`	`{ ResponseParameters: {}, ResponseModels: {}, StatusCode: 200 },`
`99`		`- { ResponseParameters: {}, ResponseModels: {}, StatusCode: 400 }`
	`99`	`+ { ResponseParameters: {}, ResponseModels: {}, StatusCode: 400 },`
	`100`	`+ { ResponseParameters: {}, ResponseModels: {}, StatusCode: 500 }`
`100`	`101`	`]`
`101`	`102`	`}`
`102`	`103`	`}`
`@@ -196,6 +197,11 @@ describe('#compileIamRoleToSqs()', () => {`
`196`	`197`	`ResponseParameters: { 'method.response.header.Access-Control-Allow-Origin': "'*'" },`
`197`	`198`	`ResponseModels: {},`
`198`	`199`	`StatusCode: 400`
	`200`	`+ },`
	`201`	`+ {`
	`202`	`+ ResponseParameters: { 'method.response.header.Access-Control-Allow-Origin': "'*'" },`
	`203`	`+ ResponseModels: {},`
	`204`	`+ StatusCode: 500`
`199`	`205`	`}`
`200`	`206`	`]`
`201`	`207`	`}`