Update stage and region handling (#3)

* Add tests

* Update stage and region handling

Author: sbstjn

README.md

@@ -75,7 +75,7 @@ package:
 
 custom:
   output:
-    handler: scripts/output.handle
+    handler: scripts/output.handler
     file: .build/stack.toml
 
 provider:

src/plugin.js

@@ -24,7 +24,7 @@ class Plugin {
   }
 
   get stackName () {
-    return this.serverless.service.service + '-' + this.options.stage
+    return this.serverless.service.service + '-' + this.serverless.getProvider('aws').getStage()
   }
 
   hasConfig (key) {
@@ -71,8 +71,8 @@ class Plugin {
       {
         StackName: this.stackName
       },
-      this.options.stage,
-      this.options.region
+      this.serverless.getProvider('aws').getStage(),
+      this.serverless.getProvider('aws').getRegion()
     )
   }
 

test/plugin.spec.js

@@ -25,6 +25,9 @@ describe('Plugin', () => {
   describe('Configuration', () => {
     it('hasHandler', () => {
       const config = {
+        config: {
+          servicePath: ''
+        },
         cli: { log: () => {} },
         region: 'us-east-1',
         service: {
@@ -44,12 +47,17 @@ describe('Plugin', () => {
 
       expect(test.hasHandler()).toBe(true)
       expect(test.hasFile()).toBe(false)
+
+      expect(test.handler).toContain('foo/bar.baz')
     })
   })
 
   describe('Configuration', () => {
     it('hasFile', () => {
       const config = {
+        config: {
+          servicePath: ''
+        },
         cli: { log: () => {} },
         region: 'us-east-1',
         service: {
@@ -58,7 +66,7 @@ describe('Plugin', () => {
           },
           custom: {
             output: {
-              file: './foo/bar.toml'
+              file: 'foo/bar.toml'
             }
           }
         },
@@ -69,6 +77,8 @@ describe('Plugin', () => {
 
       expect(test.hasHandler()).toBe(false)
       expect(test.hasFile()).toBe(true)
+
+      expect(test.file).toContain('foo/bar.toml')
     })
   })
 })