Allow the same value for range start and end value (#291)

harrisiirak · Nov 9, 2022 · 7d8aad6 · 7d8aad6
1 parent fefd0a2
commit 7d8aad6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/lib/expression.js b/lib/expression.js
@@ -324,7 +324,7 @@ CronExpression._parseField = function _parseField (field, value, constraints) {
           ' expected range ' +
           constraints.min + '-' + constraints.max
         );
-      } else if (min >= max) {
+      } else if (min > max) {
         throw new Error('Invalid range: ' + val);
       }
 
@@ -343,7 +343,6 @@ CronExpression._parseField = function _parseField (field, value, constraints) {
           repeatIndex++;
         }
       }
-
       return stack;
     }
 

diff --git a/test/expression.js b/test/expression.js
@@ -372,6 +372,17 @@ test('incremental range test with iterator', function(t) {
   t.end();
 });
 
+test('range with the same start and end value', function(t) {
+  try {
+    var interval = CronExpression.parse('*/10 2-2 * * *');
+    t.ok(interval, 'Interval parsed');
+  } catch (err) {
+    t.error(err, 'Interval parse error');
+  }
+
+  t.end();
+});
+
 test('predefined expression', function(t) {
   try {
     var interval = CronExpression.parse('@yearly');