Hi Martin,

Thank you very much for the heads-up. I have filed a bug for the same: 
https://bugs.openjdk.java.net/browse/JDK-8190259

Thank you for the patch as well. This will be integrated into JDK along with 
the tzdata2017c.

 

Regards,

Ramanand.

 

From: Martin Buchholz [mailto:marti...@google.com] 
Sent: Tuesday, October 24, 2017 2:05 AM
To: i18n-...@openjdk.java.net; core-libs-dev <core-libs-dev@openjdk.java.net>; 
Ramanand Patil <ramanand.pa...@oracle.com>; Stephen Colebourne 
<scolebou...@joda.org>
Subject: tzdata2017c is out

 

tzdata2017c came out today, and the elves at Google are working hard to 
incorporate changes in hours or days instead of weeks or quarters.  One thing 
we noticed is that one of the java.time tck tests was broken by tzdata rewrite 
of history.  Here's the fix we're applying (s/1879/1892/g):

 

@@ -941,21 +941,21 @@

     }

 

     public void test_Apia_jumpForwardOverInternationalDateLine_P12_to_M12() {

-        // transition occurred at 1879-07-04T00:00+12:33:04

+        // transition occurred at 1892-07-04T00:00+12:33:04

         ZoneRules test = pacificApia();

-        Instant instantBefore = LocalDate.of(1879, 7, 
2).atStartOfDay(ZoneOffset.UTC).toInstant();

+        Instant instantBefore = LocalDate.of(1892, 7, 
2).atStartOfDay(ZoneOffset.UTC).toInstant();

         ZoneOffsetTransition trans = test.nextTransition(instantBefore);

-        assertEquals(trans.getDateTimeBefore(), LocalDateTime.of(1879, 7, 5, 
0, 0));

-        assertEquals(trans.getDateTimeAfter(), LocalDateTime.of(1879, 7, 4, 0, 
0));

+        assertEquals(trans.getDateTimeBefore(), LocalDateTime.of(1892, 7, 5, 
0, 0));

+        assertEquals(trans.getDateTimeAfter(), LocalDateTime.of(1892, 7, 4, 0, 
0));

         assertEquals(trans.isGap(), false);

         assertEquals(trans.isOverlap(), true);

         assertEquals(trans.isValidOffset(ZoneOffset.ofHoursMinutesSeconds(+12, 
33, 4)), true);

         assertEquals(trans.isValidOffset(ZoneOffset.ofHoursMinutesSeconds(-11, 
-26, -56)), true);

         assertEquals(trans.getDuration(), Duration.ofHours(-24));

-        assertEquals(trans.getInstant(), LocalDateTime.of(1879, 7, 4, 0, 
0).toInstant(ZoneOffset.ofHoursMinutesSeconds(-11, -26, -56)));

+        assertEquals(trans.getInstant(), LocalDateTime.of(1892, 7, 4, 0, 
0).toInstant(ZoneOffset.ofHoursMinutesSeconds(-11, -26, -56)));

 

-        ZonedDateTime zdt = ZonedDateTime.of(1879, 7, 4, 23, 0, 0, 0, 
ZoneId.of("Pacific/Apia"));

-        assertEquals(zdt.plusHours(2).toLocalDateTime(), 
LocalDateTime.of(1879, 7, 4, 1, 0, 0));

+        ZonedDateTime zdt = ZonedDateTime.of(1892, 7, 4, 23, 0, 0, 0, 
ZoneId.of("Pacific/Apia"));

+        assertEquals(zdt.plusHours(2).toLocalDateTime(), 
LocalDateTime.of(1892, 7, 4, 1, 0, 0));

     }

 

     //-------------------------------------------------------------------------

 

Reply via email to