fix: change to new joi syntax

Author: kuhiga

lib/deploy/stepFunctions/compileAlarms.js

@@ -2,7 +2,6 @@
 
 const _ = require('lodash');
 const BbPromise = require('bluebird');
-const Joi = require('joi');
 const schema = require('./compileAlarms.schema');
 const logger = require('../../utils/logger');
 
@@ -142,17 +141,17 @@ function validateConfig(serverless, stateMachineName, alarmsObj) {
     return false;
   }
 
-  Joi.validate(alarmsObj, schema, { allowUnknown: false }, (err) => {
-    if (err) {
-      logger.log(
-        `State machine [${stateMachineName}] : alarms config is malformed. `
-        + 'Please see https://github.com/horike37/serverless-step-functions for examples. '
-        + `${err}`,
-      );
-      return false;
-    }
-    return true;
-  });
+  const { error } = schema.validate(alarmsObj, { allowUnknown: false });
+
+  if (error) {
+    logger.log(
+      `State machine [${stateMachineName}] : alarms config is malformed. `
+      + 'Please see https://github.com/horike37/serverless-step-functions for examples. '
+      + `${error}`,
+    );
+    return false;
+  }
+
   return true;
 }
 

lib/deploy/stepFunctions/compileAlarms.schema.js

@@ -15,15 +15,15 @@ const arn = Joi.alternatives().try(
     ),
   }),
   Joi.object().keys({
-    'Fn::Join': Joi.array().items([
+    'Fn::Join': Joi.array().items(
       Joi.string(),
-      Joi.array().items([
+      Joi.array().items(
         Joi.string(),
         Joi.object().keys({
           Ref: Joi.string(),
         }),
-      ]),
-    ]),
+      ),
+    ),
   }),
 );
 

lib/deploy/stepFunctions/compileNotifications.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const _ = require('lodash');
-const Joi = require('joi');
 const crypto = require('crypto');
 const BbPromise = require('bluebird');
 const schema = require('./compileNotifications.schema');
@@ -326,18 +325,18 @@ function validateConfig(serverless, stateMachineName, stateMachineObj, notificat
     return false;
   }
 
-  Joi.validate(notificationsObj, schema, { allowUnknown: false }, (err) => {
-    if (err) {
-      logger.log(
-        `State machine [${stateMachineName}] : notifications config is malformed. `
-        + 'Please see https://github.com/horike37/serverless-step-functions for examples. '
-        + `${err}`,
-      );
-      return false;
-    }
+  const { error } = schema.validate(
+    notificationsObj, { allowUnknown: false },
+  );
 
-    return true;
-  });
+  if (error) {
+    logger.log(
+      `State machine [${stateMachineName}] : notifications config is malformed. `
+      + 'Please see https://github.com/horike37/serverless-step-functions for examples. '
+      + `${error}`,
+    );
+    return false;
+  }
 
   return true;
 }

lib/deploy/stepFunctions/compileNotifications.schema.js

@@ -9,15 +9,15 @@ const arn = Joi.alternatives().try(
     'Fn::GetAtt': Joi.array().items(Joi.string()),
   }),
   Joi.object().keys({
-    'Fn::Join': Joi.array().items([
+    'Fn::Join': Joi.array().items(
       Joi.string(),
-      Joi.array().items([
+      Joi.array().items(
         Joi.string(),
         Joi.object().keys({
           Ref: Joi.string(),
         }),
-      ]),
-    ]),
+      ),
+    ),
   }),
 );
 

lib/deploy/stepFunctions/compileStateMachines.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const _ = require('lodash');
-const Joi = require('joi');
 const aslValidator = require('asl-validator');
 const BbPromise = require('bluebird');
 const crypto = require('crypto');
@@ -107,17 +106,13 @@ module.exports = {
         } else {
           Tags = toTags(this.serverless.service.provider.tags);
         }
-        let validationValue;
-        Joi.validate(stateMachineObj, schema, { allowUnknown: false }, (err, value) => {
-          if (err) {
-            const errorMessage = `State machine [${stateMachineName}] is malformed. `
+        const { error, value } = schema.validate(stateMachineObj, { allowUnknown: false });
+        if (error) {
+          const errorMessage = `State machine [${stateMachineName}] is malformed. `
             + 'Please check the README for more info. '
-            + `${err}`;
-            throw new this.serverless.classes.Error(errorMessage);
-          } else {
-            validationValue = value;
-          }
-        });
+            + `${error}`;
+          throw new this.serverless.classes.Error(errorMessage);
+        }
 
         if (stateMachineObj.definition) {
           if (this.serverless.service.stepFunctions.validate) {
@@ -204,23 +199,23 @@ module.exports = {
           _.forEach(stateMachineTags, tag => Tags.push(tag));
         }
 
-        if (validationValue.loggingConfig) {
-          const Destinations = (validationValue.loggingConfig.destinations || [])
+        if (value.loggingConfig) {
+          const Destinations = (value.loggingConfig.destinations || [])
             .map(arn => ({
               CloudWatchLogsLogGroup: {
                 LogGroupArn: arn,
               },
             }));
           LoggingConfiguration = {
-            Level: validationValue.loggingConfig.level,
-            IncludeExecutionData: validationValue.loggingConfig.includeExecutionData,
+            Level: value.loggingConfig.level,
+            IncludeExecutionData: value.loggingConfig.includeExecutionData,
             Destinations,
           };
         }
 
-        if (validationValue.tracingConfig) {
+        if (value.tracingConfig) {
           TracingConfiguration = {
-            Enabled: validationValue.tracingConfig.enabled,
+            Enabled: value.tracingConfig.enabled,
           };
         }
 

lib/deploy/stepFunctions/compileStateMachines.schema.js

@@ -15,15 +15,15 @@ const arn = Joi.alternatives().try(
     ),
   }),
   Joi.object().keys({
-    'Fn::Join': Joi.array().items([
+    'Fn::Join': Joi.array().items(
       Joi.string(),
-      Joi.array().items([
+      Joi.array().items(
         Joi.string(),
         Joi.object().keys({
           Ref: Joi.string(),
         }),
-      ]),
-    ]),
+      ),
+    ),
   }),
 );