AAE-24320 Remove condition to check if date difference is negative

Activiti · Jul 25, 2024 · 6196864 · 6196864
1 parent 022f8d4
commit 6196864
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.../src/main/java/org/activiti/cloud/services/audit/jpa/service/AuditEventsAdminService.java b/.../src/main/java/org/activiti/cloud/services/audit/jpa/service/AuditEventsAdminService.java
@@ -37,8 +37,8 @@ public Collection<AuditEventEntity> findAuditsBetweenDates(LocalDate fromDate, L
 
         long daysBetween = ChronoUnit.DAYS.between(fromDate, toDate);
 
-        if (daysBetween > 31 || daysBetween < 0) {
-            throw new IllegalArgumentException("Difference between dates cannot be more than 31 days or negative");
+        if (daysBetween > 31) {
+            throw new IllegalArgumentException("Difference between dates cannot be more than 31 days");
         }
 
         Long startDateTime = fromDate.atStartOfDay().toEpochSecond(ZoneOffset.UTC);

diff --git a/.../test/java/org/activiti/cloud/services/audit/jpa/service/AuditEventsAdminServiceTest.java b/.../test/java/org/activiti/cloud/services/audit/jpa/service/AuditEventsAdminServiceTest.java
@@ -67,7 +67,7 @@ void should_throw_exception_when_difference_between_dates_is_more_than_31_days()
         );
 
         // then
-        assertEquals("Difference between dates cannot be more than 31 days or negative", thrown.getMessage());
+        assertEquals("Difference between dates cannot be more than 31 days", thrown.getMessage());
     }
 
     @Test
-Original file line number
+Diff line change
@@ Expand Up @@
             );
             // then
-            assertEquals("Difference between dates cannot be more than 31 days or negative", thrown.getMessage());
+            assertEquals("Difference between dates cannot be more than 31 days", thrown.getMessage());
         }
         @Test
@@ Expand Down @@