Update testcases

Author: kanoshiou

server/src/main/java/org/elasticsearch/common/time/DateUtils.java

@@ -374,7 +374,7 @@ public static long roundMonthOfYear(final long utcMillis) {
      */
     public static long roundIntervalMonthOfYear(final long utcMillis, final int monthInterval) {
         if (monthInterval <= 0) {
-            throw new IllegalArgumentException("month interval must be strictly positive, got  [" + monthInterval + "]");
+            throw new IllegalArgumentException("month interval must be strictly positive, got [" + monthInterval + "]");
         }
         int year = getYear(utcMillis);
         int month = getMonthOfYear(utcMillis, year);
@@ -400,7 +400,7 @@ public static long roundYear(final long utcMillis) {
      */
     public static long roundYearInterval(final long utcMillis, final int yearInterval) {
         if (yearInterval <= 0) {
-            throw new IllegalArgumentException("year interval [" + yearInterval + "] must be positive");
+            throw new IllegalArgumentException("year interval must be strictly positive, got [" + yearInterval + "]");
         }
         int year = getYear(utcMillis);
         return utcMillisAtStartOfYear(((year - 1) / yearInterval) * yearInterval + 1);
@@ -425,7 +425,7 @@ public static long roundWeekOfWeekYear(final long utcMillis) {
      */
     public static long roundWeekIntervalOfWeekYear(final long utcMillis, final int weekInterval) {
         if (weekInterval <= 0) {
-            throw new IllegalArgumentException("week interval [" + weekInterval + "] must be positive");
+            throw new IllegalArgumentException("week interval must be strictly positive, got [" + weekInterval + "]");
         }
         return roundFloor(utcMillis + 3 * 86400 * 1000L, 604800000L * weekInterval) - 3 * 86400 * 1000L;
     }

server/src/test/java/org/elasticsearch/common/time/DateUtilsTests.java

@@ -230,7 +230,7 @@ public void testRoundMonthOfYear() {
         assertThat(DateUtils.roundMonthOfYear(-1), is(dec1969));
 
         IllegalArgumentException exc = expectThrows(IllegalArgumentException.class, () -> DateUtils.roundIntervalMonthOfYear(0, -1));
-        assertThat(exc.getMessage(), is("month interval [-1] must be positive"));
+        assertThat(exc.getMessage(), is("month interval must be strictly positive, got [-1]"));
         assertThat(DateUtils.roundIntervalMonthOfYear(1, 5), is(0L));
         long epochMilli = LocalDate.of(1969, 7, 1).atStartOfDay().toInstant(ZoneOffset.UTC).toEpochMilli();
         assertThat(DateUtils.roundIntervalMonthOfYear(-1, 13), is(epochMilli));
@@ -251,7 +251,7 @@ public void testRoundYear() {
         assertThat(DateUtils.roundYear(endOf1996), is(startOf1996));
 
         IllegalArgumentException exc = expectThrows(IllegalArgumentException.class, () -> DateUtils.roundYearInterval(0, -1));
-        assertThat(exc.getMessage(), is("year interval [-1] must be positive"));
+        assertThat(exc.getMessage(), is("year interval must be strictly positive, got [-1]"));
         assertThat(DateUtils.roundYearInterval(0, 2), is(startOf1969));
         long startOf1968 = Year.of(1968).atDay(1).atStartOfDay().toInstant(ZoneOffset.UTC).toEpochMilli();
         assertThat(DateUtils.roundYearInterval(0, 7), is(startOf1968));
@@ -270,7 +270,7 @@ public void testRoundWeek() {
         assertThat(DateUtils.roundWeekOfWeekYear(-1), is(epochMilli));
 
         IllegalArgumentException exc = expectThrows(IllegalArgumentException.class, () -> DateUtils.roundWeekIntervalOfWeekYear(0, -1));
-        assertThat(exc.getMessage(), is("week interval [-1] must be positive"));
+        assertThat(exc.getMessage(), is("week interval must be strictly positive, got [-1]"));
         assertThat(DateUtils.roundWeekIntervalOfWeekYear(0, 3), is(epochMilli));
         assertThat(DateUtils.roundWeekIntervalOfWeekYear(1, 3), is(epochMilli));
         assertThat(DateUtils.roundWeekIntervalOfWeekYear(-1, 2), is(epochMilli));