Merge pull request #414 from melonhead901/issue413

Fix interval notation in period error message. #413

Author: jmrozanec

src/main/java/com/cronutils/model/field/expression/visitor/ValidationFieldExpressionVisitor.java

@@ -167,7 +167,7 @@ protected void isPeriodInRange(final FieldValue<?> fieldValue) {
             final int value = ((IntegerFieldValue) fieldValue).getValue();
             if (!constraints.isPeriodInRange(value)) {
                 throw new IllegalArgumentException(
-                        String.format("Period %s not in range (%s, %s]", value, constraints.getStartRange(), constraints.getEndRange()));
+                        String.format("Period %s not in range [%s, %s)", value, constraints.getStartRange(), constraints.getEndRange()));
             }
         }
     }

src/test/java/com/cronutils/Issue413Test.java

@@ -0,0 +1,24 @@
+package com.cronutils;
+
+import com.cronutils.model.CronType;
+import com.cronutils.model.definition.CronDefinitionBuilder;
+import com.cronutils.parser.CronParser;
+import org.junit.Test;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.fail;
+
+public class Issue413Test {
+
+    @Test
+    public void testFridayToSaturdayQuartz() {
+        final CronParser parser = new CronParser(CronDefinitionBuilder.instanceDefinitionFor(CronType.UNIX));
+        try {
+            parser.parse("* * * */12 *");
+            // expected to fail.
+            fail();
+        } catch (IllegalArgumentException expected) {
+            assertEquals("Failed to parse '* * * */12 *'. Period 12 not in range [1, 12)", expected.getMessage());
+        }
+    }
+}