From 72895f081026df7e0b34807729d9cdea6c7ff4ec Mon Sep 17 00:00:00 2001 From: Arjen Poutsma Date: Fri, 24 Jul 2020 12:37:17 +0200 Subject: [PATCH] Add test for combination of fixed date fields Added test for a fixed day-of-week and day-of-month combination. With the new CronExpression in place, this failure does not occur anymore. Closes gh-13621 --- .../scheduling/support/CronExpressionTests.java | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/spring-context/src/test/java/org/springframework/scheduling/support/CronExpressionTests.java b/spring-context/src/test/java/org/springframework/scheduling/support/CronExpressionTests.java index b1e2b54e7d..b55c3b7fd0 100644 --- a/spring-context/src/test/java/org/springframework/scheduling/support/CronExpressionTests.java +++ b/spring-context/src/test/java/org/springframework/scheduling/support/CronExpressionTests.java @@ -431,4 +431,17 @@ class CronExpressionTests { assertThat(expression.next(last)).isEqualTo(expected); } + @Test + public void fixedDays() { + CronExpression expression = CronExpression.parse("0 0 0 29 2 WED"); + + LocalDateTime last = LocalDateTime.of(2012, 2, 29, 1, 0); + assertThat(last.getDayOfWeek()).isEqualTo(WEDNESDAY); + + LocalDateTime actual = expression.next(last); + assertThat(actual).isNotNull(); + assertThat(actual.getDayOfMonth()).isEqualTo(29); + assertThat(actual.getDayOfWeek()).isEqualTo(WEDNESDAY); + } + }