Update all var keywords to let, or const - It depends on variable update

node-cron · Sep 16, 2020 · 27e1140 · 27e1140
1 parent 4f26623
commit 27e1140
Show file tree

Hide file tree

Showing 21 changed files with 73 additions and 78 deletions.
diff --git a/src/convert-expression/index.js b/src/convert-expression/index.js
@@ -1,10 +1,10 @@
 'use strict';
 
-var monthNamesConversion = require('./month-names-conversion');
-var weekDayNamesConversion = require('./week-day-names-conversion');
-var convertAsterisksToRanges = require('./asterisk-to-range-conversion');
-var convertRanges = require('./range-conversion');
-var convertSteps = require('./step-values-conversion');
+const monthNamesConversion = require('./month-names-conversion');
+const weekDayNamesConversion = require('./week-day-names-conversion');
+const convertAsterisksToRanges = require('./asterisk-to-range-conversion');
+const convertRanges = require('./range-conversion');
+const convertSteps = require('./step-values-conversion');
 
 module.exports = (() => {
 
@@ -21,9 +21,9 @@ module.exports = (() => {
 
     // Function that takes care of normalization.
     function normalizeIntegers(expressions) {
-        for (var i=0; i < expressions.length; i++){
-            var numbers = expressions[i].split(',');
-            for (var j=0; j<numbers.length; j++){
+        for (let i=0; i < expressions.length; i++){
+            const numbers = expressions[i].split(',');
+            for (let j=0; j<numbers.length; j++){
                 numbers[j] = parseInt(numbers[j]);
             }
             expressions[i] = numbers;
@@ -49,7 +49,7 @@ module.exports = (() => {
    *  - Will be translated to 1,2,3,4,5 * * * *
    */
     function interprete(expression){
-        var expressions = removeSpaces(expression).split(' ');
+        let expressions = removeSpaces(expression).split(' ');
         expressions = appendSeccondExpression(expressions);
         expressions[4] = monthNamesConversion(expressions[4]);
         expressions[5] = weekDayNamesConversion(expressions[5]);

diff --git a/src/convert-expression/month-names-conversion.js b/src/convert-expression/month-names-conversion.js
@@ -1,12 +1,12 @@
 'use strict';
 module.exports = (() => {
-    var months = ['january','february','march','april','may','june','july',
+    const months = ['january','february','march','april','may','june','july',
         'august','september','october','november','december'];
-    var shortMonths = ['jan', 'feb', 'mar', 'apr', 'may', 'jun', 'jul', 'aug',
+    const shortMonths = ['jan', 'feb', 'mar', 'apr', 'may', 'jun', 'jul', 'aug',
         'sep', 'oct', 'nov', 'dec'];
 
     function convertMonthName(expression, items){
-        for(var i = 0; i < items.length; i++){
+        for(let i = 0; i < items.length; i++){
             expression = expression.replace(new RegExp(items[i], 'gi'), parseInt(i, 10) + 1);
         }
         return expression;

diff --git a/src/convert-expression/range-conversion.js b/src/convert-expression/range-conversion.js
@@ -2,25 +2,25 @@
 module.exports = ( () => {
     function replaceWithRange(expression, text, init, end) {
 
-        var numbers = [];
-        var last = parseInt(end);
-        var first = parseInt(init);
+        const numbers = [];
+        let last = parseInt(end);
+        let first = parseInt(init);
 
         if(first > last){
             last = parseInt(init);
             first = parseInt(end);
         }
 
-        for(var i = first; i <= last; i++) {
+        for(let i = first; i <= last; i++) {
             numbers.push(i);
         }
 
         return expression.replace(new RegExp(text, 'gi'), numbers.join());
     }
 
     function convertRange(expression){
-        var rangeRegEx = /(\d+)-(\d+)/;
-        var match = rangeRegEx.exec(expression);
+        const rangeRegEx = /(\d+)-(\d+)/;
+        let match = rangeRegEx.exec(expression);
         while(match !== null && match.length > 0){
             expression = replaceWithRange(expression, match[0], match[1], match[2]);
             match = rangeRegEx.exec(expression);
@@ -29,14 +29,11 @@ module.exports = ( () => {
     }
 
     function convertAllRanges(expressions){
-        for(var i = 0; i < expressions.length; i++){
+        for(let i = 0; i < expressions.length; i++){
             expressions[i] = convertRange(expressions[i]);
         }
         return expressions;
     }
 
     return convertAllRanges;
 })();
-
-
-
diff --git a/src/convert-expression/step-values-conversion.js b/src/convert-expression/step-values-conversion.js
@@ -2,26 +2,25 @@
 
 module.exports = (() => {
     function convertSteps(expressions){
-        var stepValuePattern = /^(.+)\/(\d+)$/;
-        for(var i = 0; i < expressions.length; i++){
-            var match = stepValuePattern.exec(expressions[i]);
-            var isStepValue = match !== null && match.length > 0;
+        const stepValuePattern = /^(.+)\/(\d+)$/;
+        for(let i = 0; i < expressions.length; i++){
+            const match = stepValuePattern.exec(expressions[i]);
+            const isStepValue = match !== null && match.length > 0;
             if(isStepValue){
-                var values = match[1].split(',');
-                var setpValues = [];
-                var divider = parseInt(match[2], 10);
-                for(var j = 0; j <= values.length; j++){
-                    var value = parseInt(values[j], 10);
+                const values = match[1].split(',');
+                const stepValues = [];
+                const divider = parseInt(match[2], 10);
+                for(let j = 0; j <= values.length; j++){
+                    const value = parseInt(values[j], 10);
                     if(value % divider === 0){
-                        setpValues.push(value);
+                        stepValues.push(value);
                     }
                 }
-                expressions[i] = setpValues.join(',');
+                expressions[i] = stepValues.join(',');
             }
         }
         return expressions;
     }
 
     return convertSteps;
 })();
-
diff --git a/src/convert-expression/week-day-names-conversion.js b/src/convert-expression/week-day-names-conversion.js
@@ -1,11 +1,11 @@
 'use strict';
 module.exports = (() => {
-    var weekDays = ['sunday', 'monday', 'tuesday', 'wednesday', 'thursday',
+    const weekDays = ['sunday', 'monday', 'tuesday', 'wednesday', 'thursday',
         'friday', 'saturday'];
-    var shortWeekDays = ['sun', 'mon', 'tue', 'wed', 'thu', 'fri', 'sat'];
+    const shortWeekDays = ['sun', 'mon', 'tue', 'wed', 'thu', 'fri', 'sat'];
 
     function convertWeekDayName(expression, items){
-        for(var i = 0; i < items.length; i++){
+        for(let i = 0; i < items.length; i++){
             expression = expression.replace(new RegExp(items[i], 'gi'), parseInt(i, 10));
         }
         return expression;

diff --git a/src/node-cron.js b/src/node-cron.js
@@ -35,9 +35,8 @@ module.exports = (() => {
     function createTask(expression, func, options){
         if(typeof func === 'string'){
             return new BackgroundScheduledTask(expression, func, options);
-        } else {
-            return new ScheduledTask(expression, func, options);
         }
+        return new ScheduledTask(expression, func, options);
     }
 
     /**

diff --git a/src/pattern-validation.js b/src/pattern-validation.js
@@ -1,15 +1,15 @@
 'use strict';
 
-var convertExpression = require('./convert-expression');
+const convertExpression = require('./convert-expression');
 
 
 module.exports = ( () => {
     function isValidExpression(expression, min, max){
-        var options = expression.split(',');
-        var regexValidation = /^\d+$|^\*$|^\*\/\d+$/;
-        for(var i = 0; i < options.length; i++){
-            var option = options[i];
-            var optionAsInt = parseInt(options[i], 10);
+        const options = expression.split(',');
+        const regexValidation = /^\d+$|^\*$|^\*\/\d+$/;
+        for(let i = 0; i < options.length; i++){
+            const option = options[i];
+            const optionAsInt = parseInt(options[i], 10);
             if(optionAsInt < min || optionAsInt > max || !regexValidation.test(option)) {
                 return false;
             }
@@ -72,9 +72,9 @@ module.exports = ( () => {
             throw 'pattern must be a string!';
         }
 
-        var patterns = pattern.split(' ');
-        var executablePattern = convertExpression(pattern);
-        var executablePatterns = executablePattern.split(' ');
+        let patterns = pattern.split(' ');
+        const executablePattern = convertExpression(pattern);
+        const executablePatterns = executablePattern.split(' ');
 
         if(patterns.length === 5){
             patterns = ['0'].concat(patterns);

diff --git a/test/convert-expression/asterisk-to-range-conversion-test.js b/test/convert-expression/asterisk-to-range-conversion-test.js
@@ -1,12 +1,12 @@
 'use strict';
 
 const { expect } = require('chai');
-var conversion = require('../../src/convert-expression/asterisk-to-range-conversion');
+const conversion = require('../../src/convert-expression/asterisk-to-range-conversion');
 
 describe('asterisk-to-range-conversion.js', () => {
     it('shuld convert * to ranges', () => {
-        var expressions = '* * * * * *'.split(' ');
-        var expression = conversion(expressions).join(' ');
+        const expressions = '* * * * * *'.split(' ');
+        const expression = conversion(expressions).join(' ');
         expect(expression).to.equal('0-59 0-59 0-23 1-31 1-12 0-6');
     });
 });
diff --git a/test/convert-expression/convert-expression-test.js b/test/convert-expression/convert-expression-test.js
@@ -1,18 +1,18 @@
 'use strict';
 
 const { expect } = require('chai');
-var conversion = require('../../src/convert-expression');
+const conversion = require('../../src/convert-expression');
 
 describe('month-names-conversion.js', () => {
     it('shuld convert month names', () => {
-        var expression = conversion('* * * * January,February *');
-        var expressions = expression.split(' ');
+        const expression = conversion('* * * * January,February *');
+        const expressions = expression.split(' ');
         expect(expressions[4]).to.equal('1,2');
     });
 
     it('shuld convert week day names', () => {
-        var expression = conversion('* * * * * Mon,Sun');
-        var expressions = expression.split(' ');
+        const expression = conversion('* * * * * Mon,Sun');
+        const expressions = expression.split(' ');
         expect(expressions[5]).to.equal('1,0');
     });
 });
diff --git a/test/convert-expression/month-names-conversion-test.js b/test/convert-expression/month-names-conversion-test.js
@@ -1,16 +1,16 @@
 'use strict';
 
 const { expect } = require('chai');
-var conversion = require('../../src/convert-expression/month-names-conversion');
+const conversion = require('../../src/convert-expression/month-names-conversion');
 
 describe('month-names-conversion.js', () => {
     it('shuld convert month names', () => {
-        var months = conversion('January,February,March,April,May,June,July,August,September,October,November,December');
+        const months = conversion('January,February,March,April,May,June,July,August,September,October,November,December');
         expect(months).to.equal('1,2,3,4,5,6,7,8,9,10,11,12');
     });
 
     it('shuld convert month names', () => {
-        var months = conversion('Jan,Feb,Mar,Apr,May,Jun,Jul,Aug,Sep,Oct,Nov,Dec');
+        const months = conversion('Jan,Feb,Mar,Apr,May,Jun,Jul,Aug,Sep,Oct,Nov,Dec');
         expect(months).to.equal('1,2,3,4,5,6,7,8,9,10,11,12');
     });
 });
diff --git a/test/convert-expression/range-conversion-test.js b/test/convert-expression/range-conversion-test.js
@@ -1,18 +1,18 @@
 'use strict';
 
 const { expect } = require('chai');
-var conversion = require('../../src/convert-expression/range-conversion');
+const conversion = require('../../src/convert-expression/range-conversion');
 
 describe('range-conversion.js', () => {
     it('shuld convert ranges to numbers', () => {
-        var expressions = '0-3 0-3 0-2 1-3 1-2 0-3'.split(' ');
-        var expression = conversion(expressions).join(' ');
+        const expressions = '0-3 0-3 0-2 1-3 1-2 0-3'.split(' ');
+        const expression = conversion(expressions).join(' ');
         expect(expression).to.equal('0,1,2,3 0,1,2,3 0,1,2 1,2,3 1,2 0,1,2,3');
     });
 
     it('shuld convert ranges to numbers', () => {
-        var expressions = '0-3 0-3 8-10 1-3 1-2 0-3'.split(' ');
-        var expression = conversion(expressions).join(' ');
+        const expressions = '0-3 0-3 8-10 1-3 1-2 0-3'.split(' ');
+        const expression = conversion(expressions).join(' ');
         expect(expression).to.equal('0,1,2,3 0,1,2,3 8,9,10 1,2,3 1,2 0,1,2,3');
     });
 });
diff --git a/test/convert-expression/step-values-conversion-test.js b/test/convert-expression/step-values-conversion-test.js
@@ -1,11 +1,11 @@
 'use strict';
 
 const { expect } = require('chai');
-var conversion = require('../../src/convert-expression/step-values-conversion');
+const conversion = require('../../src/convert-expression/step-values-conversion');
 
 describe('step-values-conversion.js', () => {
     it('shuld convert step values', () => {
-        var expressions = '1,2,3,4,5,6,7,8,9,10/2 0,1,2,3,4,5,6,7,8,9/5 * * * *'.split(' ');
+        let expressions = '1,2,3,4,5,6,7,8,9,10/2 0,1,2,3,4,5,6,7,8,9/5 * * * *'.split(' ');
         expressions = conversion(expressions);
         expect(expressions[0]).to.equal('2,4,6,8,10');
         expect(expressions[1]).to.equal('0,5');

diff --git a/test/convert-expression/week-day-names-conversion-test.js b/test/convert-expression/week-day-names-conversion-test.js
@@ -1,21 +1,21 @@
 'use strict';
 
 const { expect } = require('chai');
-var conversion = require('../../src/convert-expression/week-day-names-conversion');
+const conversion = require('../../src/convert-expression/week-day-names-conversion');
 
 describe('week-day-names-conversion.js', () => {
     it('shuld convert week day names names', () => {
-        var weekDays = conversion('Monday,Tuesday,Wednesday,Thursday,Friday,Saturday,Sunday');
+        const weekDays = conversion('Monday,Tuesday,Wednesday,Thursday,Friday,Saturday,Sunday');
         expect(weekDays).to.equal('1,2,3,4,5,6,0');
     });
 
     it('shuld convert short week day names names', () => {
-        var weekDays = conversion('Mon,Tue,Wed,Thu,Fri,Sat,Sun');
+        const weekDays = conversion('Mon,Tue,Wed,Thu,Fri,Sat,Sun');
         expect(weekDays).to.equal('1,2,3,4,5,6,0');
     });
 
     it('shuld convert 7 to 0', () => {
-        var weekDays = conversion('7');
+        const weekDays = conversion('7');
         expect(weekDays).to.equal('0');
     });
 });
diff --git a/test/pattern-validation/validate-day-test.js b/test/pattern-validation/validate-day-test.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const { expect } = require('chai');
-var validate = require('../../src/pattern-validation');
+const validate = require('../../src/pattern-validation');
 
 describe('pattern-validation.js', () => {
     describe('validate day of month', () => {

diff --git a/test/pattern-validation/validate-hours-test.js b/test/pattern-validation/validate-hours-test.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const { expect } = require('chai');
-var validate = require('../../src/pattern-validation');
+const validate = require('../../src/pattern-validation');
 
 describe('pattern-validation.js', () => {
     describe('validate hour', () => {

diff --git a/test/pattern-validation/validate-minute-test.js b/test/pattern-validation/validate-minute-test.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const { expect } = require('chai');
-var validate = require('../../src/pattern-validation');
+const validate = require('../../src/pattern-validation');
 
 describe('pattern-validation.js', () => {
     describe('validate minutes', () => {

diff --git a/test/pattern-validation/validate-month-test.js b/test/pattern-validation/validate-month-test.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const { expect } = require('chai');
-var validate = require('../../src/pattern-validation');
+const validate = require('../../src/pattern-validation');
 
 describe('pattern-validation.js',  () => {
     describe('validate month',  () => {

diff --git a/test/pattern-validation/validate-pattern-type.js b/test/pattern-validation/validate-pattern-type.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const { expect } = require('chai');
-var Task = require('../../src/task');
+const Task = require('../../src/task');
 
 describe('Task', () => {
     it('should accept a function', () => {

diff --git a/test/pattern-validation/validate-second-test.js b/test/pattern-validation/validate-second-test.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const { expect } = require('chai');
-var validate = require('../../src/pattern-validation');
+const validate = require('../../src/pattern-validation');
 
 describe('pattern-validation.js', () => {
     describe('validate seconds', () => {

diff --git a/test/pattern-validation/validate-test.js b/test/pattern-validation/validate-test.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const { expect } = require('chai');
-var validate = require('../../src/pattern-validation');
+const validate = require('../../src/pattern-validation');
 
 describe('pattern-validation', () => {
     it('should succeed with a valid expression', () =>  {

diff --git a/test/pattern-validation/validate-week-day-test.js b/test/pattern-validation/validate-week-day-test.js
@@ -1,7 +1,7 @@
 'use strict';
 
 const { expect } = require('chai');
-var validate = require('../../src/pattern-validation');
+const validate = require('../../src/pattern-validation');
 
 describe('pattern-validation.js', () => {
     describe('validate week day', () => {