Merge pull request #167 from lukehedger/master

fix: add request property to EventBridge schema

Author: lopburny

lib/apiGateway/schema.js

@@ -308,7 +308,8 @@ const proxiesSchemas = {
       eventBusName: stringOrRef.required(),
       source: eventBridgeSource.required(),
       detailType: eventBridgeDetailType,
-      detail: eventBridgeDetail
+      detail: eventBridgeDetail,
+      request
     })
   })
 }

lib/apiGateway/validate.test.js

@@ -2272,4 +2272,27 @@ describe('#validateServiceProxies()', () => {
       expect(() => serverlessApigatewayServiceProxy.validateServiceProxies()).to.not.throw()
     })
   })
+
+  describe('eventbridge', () => {
+    it('should throw error if eventbridge request is missing the template property', () => {
+      serverlessApigatewayServiceProxy.serverless.service.custom = {
+        apiGatewayServiceProxies: [
+          {
+            eventbridge: {
+              eventBusName: 'yourBus',
+              source: 'proxy.apigw',
+              path: 'eventbridge',
+              method: 'post',
+              request: { xxx: { 'application/json': 'mappingTemplate' } }
+            }
+          }
+        ]
+      }
+
+      expect(() => serverlessApigatewayServiceProxy.validateServiceProxies()).to.throw(
+        serverless.classes.Error,
+        'child "eventbridge" fails because [child "request" fails because [child "template" fails because ["template" is required]]]'
+      )
+    })
+  })
 })