throw error in case of step value is NaN

node-cron · May 18, 2020 · d740612 · d740612
1 parent 2a39703
commit d740612
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/convert-expression/step-values-conversion.js b/src/convert-expression/step-values-conversion.js
@@ -2,14 +2,18 @@
 
 module.exports = (() => {
     function convertSteps(expressions){
-        var stepValuePattern = /^(.+)\/(\d+)$/;
+        var stepValuePattern = /^(.+)\/(\w+)$/;
         for(var i = 0; i < expressions.length; i++){
             var match = stepValuePattern.exec(expressions[i]);
             var isStepValue = match !== null && match.length > 0;
             if(isStepValue){
+                var baseDivider = match[2];
+                if(isNaN(baseDivider)){
+                    throw baseDivider + 'is not a valid step value';
+                }
                 var values = match[1].split(',');
                 var setpValues = [];
-                var divider = parseInt(match[2], 10);
+                var divider = parseInt(baseDivider, 10);
                 for(var j = 0; j <= values.length; j++){
                     var value = parseInt(values[j], 10);
                     if(value % divider === 0){