renaming command to sls validate stepf

Author: Christophe Bougère

README.md

@@ -260,10 +260,10 @@ $ sls invoke stepf --name <stepfunctionname> --data '{"foo":"bar"}'
 - --data or -d String data to be passed as an event to your step function.
 - --path or -p The path to a json file with input data to be passed to the invoked step function.
 
-### validate-state-machines
+### validate
 This command allows you to validate the definition of your state machine(s). Validation is done by [asl-validator](https://www.npmjs.com/package/asl-validator).
 ```
-$ sls validate-state-machines [--name <stepfunctionname>]
+$ sls validate stepf [--name <stepfunctionname>]
 ```
 
 #### options

lib/index.js

@@ -82,15 +82,19 @@ class ServerlessStepFunctions {
           },
         },
       },
-      'validate-state-machines': {
-        usage: 'Validate state machine definitions',
-        lifecycleEvents: [
-          'validate',
-        ],
-        options: {
-          name: {
-            usage: 'The StateMachine name (validate all state machines if not specified)',
-            shortcut: 'n',
+      validate: {
+        commands: {
+          stepf: {
+            usage: 'Validate step function state machine definitions',
+            lifecycleEvents: [
+              'validate',
+            ],
+            options: {
+              name: {
+                usage: 'The StateMachine name (validate all state machines if not specified)',
+                shortcut: 'n',
+              },
+            },
           },
         },
       },
@@ -100,9 +104,9 @@ class ServerlessStepFunctions {
       'invoke:stepf:invoke': () => BbPromise.bind(this)
         .then(this.yamlParse)
         .then(this.invoke),
-      'validate-state-machines:validate': () => BbPromise.bind(this)
+      'validate:stepf:validate': () => BbPromise.bind(this)
         .then(this.yamlParse)
-        .then(this.validateStateMachines),
+        .then(this.validate),
       'package:initialize': () => BbPromise.bind(this)
         .then(this.yamlParse),
       'package:compileFunctions': () => BbPromise.bind(this)
@@ -152,7 +156,7 @@ class ServerlessStepFunctions {
     });
   }
 
-  validateStateMachines() {
+  validate() {
     this.serverless.cli.consoleLog('Validating state machines definition...');
     const stateMachines = (this.options.name && [this.options.name]) ||
       Object.keys(this.serverless.service.stepFunctions.stateMachines);

lib/index.test.js

@@ -369,7 +369,7 @@ describe('#index', () => {
     });
   });
 
-  describe('#validateStateMachines()', () => {
+  describe('#validate()', () => {
     let consoleLogStub;
 
     beforeEach(() => {
@@ -397,7 +397,7 @@ describe('#index', () => {
           },
         },
       };
-      serverlessStepFunctions.validateStateMachines();
+      serverlessStepFunctions.validate();
       // would have thrown if definition was invalid
       expect(consoleLogStub.calledTwice).to.equal(true);
     });
@@ -419,7 +419,7 @@ describe('#index', () => {
           },
         },
       };
-      expect(() => serverlessStepFunctions.validateStateMachines())
+      expect(() => serverlessStepFunctions.validate())
         .to.throw(serverlessStepFunctions.serverless.classes.Error, expectedErrorMessage);
       expect(consoleLogStub.calledTwice).to.equal(true);
     });
@@ -431,7 +431,7 @@ describe('#index', () => {
       serverlessStepFunctions.serverless.service.stepFunctions = {
         stateMachines: {},
       };
-      expect(() => serverlessStepFunctions.validateStateMachines())
+      expect(() => serverlessStepFunctions.validate())
         .to.throw(serverlessStepFunctions.serverless.classes.Error, expectedErrorMessage);
       expect(consoleLogStub.calledOnce).to.equal(true);
     });