From ab4b346f4f7e0e1d21f28cda0bf630066f9d2fce Mon Sep 17 00:00:00 2001 From: Joze Rozanec Date: Sat, 16 Jul 2022 10:40:21 +0200 Subject: [PATCH] Reinstantiate test for #499 --- src/test/java/com/cronutils/Issue499Test.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/test/java/com/cronutils/Issue499Test.java b/src/test/java/com/cronutils/Issue499Test.java index 5931bdd8..47727f73 100644 --- a/src/test/java/com/cronutils/Issue499Test.java +++ b/src/test/java/com/cronutils/Issue499Test.java @@ -9,17 +9,17 @@ import com.cronutils.parser.CronParser; import org.junit.jupiter.api.Disabled; -import org.junit.jupiter.api.Test; - +import org.junit.Test; class Issue499Test { /** - * We want to convert Unix cron expressions to Quartz cron expressions. This is - * a known format we expect. 9.1.3 Result: java.lang.IllegalArgumentException: + * We want to convert Unix cron expressions to Quartz cron expressions. We + * expect an exception: java.lang.IllegalArgumentException: * Failed to parse '12 1 * ? *'. Invalid expression: ? + * Given question marks are not supported at Unix crons. See: + * https://github.com/jmrozanec/cron-utils/issues/499 */ - @Test - @Disabled("as the comment states obviously broken since cronUtils version 9.1.3") + @Test(expected = IllegalArgumentException.class) void testCronExpressionForConversionToQuartz() { final CronParser unixParser = new CronParser(CronDefinitionBuilder.instanceDefinitionFor(CronType.UNIX)); final CronMapper unixToQuartz = CronMapper.fromUnixToQuartz();