feat: add s3 proxy to index

Author: theburningmonk

lib/index.js

@@ -21,6 +21,11 @@ const compileMethodsToSqs = require('./package/sqs/compileMethodsToSqs')
 const compileIamRoleToSqs = require('./package/sqs/compileIamRoleToSqs')
 const validateSqsServiceProxy = require('./package/sqs/validateSqsServiceProxy')
 const compileSqsServiceProxy = require('./package/sqs/compileSqsServiceProxy')
+// S3
+const compileMethodsToS3 = require('./package/s3/compileMethodsToS3')
+const compileIamRoleToS3 = require('./package/s3/compileIamRoleToS3')
+const validateS3ServiceProxy = require('./package/s3/validateS3ServiceProxy')
+const compileS3ServiceProxy = require('./package/s3/compileS3ServiceProxy')
 
 class ServerlessApigatewayServiceProxy {
   constructor(serverless, options) {
@@ -45,6 +50,10 @@ class ServerlessApigatewayServiceProxy {
       compileIamRoleToSqs,
       compileSqsServiceProxy,
       validateSqsServiceProxy,
+      compileMethodsToS3,
+      compileIamRoleToS3,
+      compileS3ServiceProxy,
+      validateS3ServiceProxy,
       getStackInfo,
       validate,
       methods,
@@ -66,6 +75,9 @@ class ServerlessApigatewayServiceProxy {
           // SQS getProxy
           await this.compileSqsServiceProxy()
 
+          // S3 getProxy
+          await this.compileS3ServiceProxy()
+
           await this.mergeDeployment()
         }
       },

lib/index.test.js

@@ -64,6 +64,12 @@ describe('#index()', () => {
               path: '/sqs',
               method: 'post'
             }
+          },
+          {
+            s3: {
+              path: '/s3',
+              method: 'post'
+            }
           }
         ]
       }
@@ -86,6 +92,9 @@ describe('#index()', () => {
       const compileSqsServiceProxyStub = sinon
         .stub(serverlessApigatewayServiceProxy, 'compileSqsServiceProxy')
         .returns(BbPromise.resolve())
+      const compileS3ServiceProxyStub = sinon
+        .stub(serverlessApigatewayServiceProxy, 'compileS3ServiceProxy')
+        .returns(BbPromise.resolve())
       const mergeDeploymentStub = sinon
         .stub(serverlessApigatewayServiceProxy, 'mergeDeployment')
         .returns(BbPromise.resolve())
@@ -99,6 +108,7 @@ describe('#index()', () => {
         expect(compileCorsStub.calledOnce).to.be.equal(true)
         expect(compileKinesisServiceProxyStub.calledOnce).to.be.equal(true)
         expect(compileSqsServiceProxyStub.calledOnce).to.be.equal(true)
+        expect(compileS3ServiceProxyStub.calledOnce).to.be.equal(true)
         expect(mergeDeploymentStub.calledOnce).to.be.equal(true)
       })
 
@@ -108,6 +118,7 @@ describe('#index()', () => {
       serverlessApigatewayServiceProxy.compileCors.restore()
       serverlessApigatewayServiceProxy.compileKinesisServiceProxy.restore()
       serverlessApigatewayServiceProxy.compileSqsServiceProxy.restore()
+      serverlessApigatewayServiceProxy.compileS3ServiceProxy.restore()
       serverlessApigatewayServiceProxy.mergeDeployment.restore()
     })
 

lib/package/s3/compileS3ServiceProxy.js

@@ -0,0 +1,9 @@
+'use strict'
+
+module.exports = {
+  async compileS3ServiceProxy() {
+    this.validateS3ServiceProxy()
+    this.compileIamRoleToS3()
+    this.compileMethodsToS3()
+  }
+}

package-lock.json

@@ -17,6 +17,7 @@
     ]
   },
   "dependencies": {
+    "@hapi/joi": "^15.1.0",
     "serverless": "^1.48.2"
   },
   "author": "horike37",