Merge pull request #8 from iliasbhal/master

default header sets Allow-Access-Origin to '*' by default now

Author: iliasbhal

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "serverless-express",
-  "version": "2.0.4",
+  "version": "2.0.5",
   "description": "Run express apps on AWS λ easily.",
   "main": "index.js",
   "scripts": {
@@ -30,7 +30,8 @@
     "jest": "^22.2.2",
     "lodash": "^4.17.5",
     "sinon": "^4.3.0",
-    "sinon-chai": "^2.14.0"
+    "sinon-chai": "^2.14.0",
+    "supertest": "^3.1.0"
   },
   "dependencies": {
     "aws-serverless-express": "^3.1.3",

package.json

@@ -7,6 +7,7 @@ let tdd = require('./_tdd')
 let express = require('express')
 let sls_express = require('serverless-express/express')
 let mock = require('./src/mock')
+var request = require('supertest');
 
 let initPlugin = require('./src/init-plugin')
 let aws_middle = require('../src/aws/aws_middleware')
@@ -73,4 +74,36 @@ describe("serverless-express express", function() {
     })
 
   })
+
+  it('should Access-Control-Allow-Origin to everyone by defualt', function(done){
+    
+    let testIsCompleted = false
+    let testRunned = 0
+    tdd.supported_providers.forEach((provider)=>{
+
+      initPlugin({provider: provider})
+      let app = sls_express()
+          app.get('*', (req, res)=>{
+            res.json(res)
+           }) // if we don't register a route the middleware stack is not exposed
+
+           request(app).get('/')
+                       .then((res)=>{
+                          testRunned += 1
+                          console.log(res)
+                          expect(res.headers['access-control-allow-origin']).to.equal('*')
+
+                          if( testRunned == tdd.supported_providers.length - 1){
+                            done()
+                          }
+
+                       })
+
+
+      // let injected = app._router.stack.filter( layer => layer && layer.handle == aws_middle ).length == 1
+      // expect(injected).to.equal( provider == 'aws')
+
+    })
+
+  })
 })