[#1724] Attempt to fix JDBCTimeZoneZonedTest sporadic failures probably due lack of database Timestamp precision

Author: dreab8

hibernate-reactive-core/src/test/java/org/hibernate/reactive/timezones/JDBCTimeZoneZonedTest.java

@@ -10,10 +10,15 @@
 import java.time.ZoneId;
 import java.time.ZoneOffset;
 import java.time.ZonedDateTime;
+import java.time.temporal.ChronoField;
+import java.time.temporal.ChronoUnit;
 import java.util.Collection;
 import java.util.List;
 
 import org.hibernate.cfg.Configuration;
+import org.hibernate.dialect.Dialect;
+import org.hibernate.dialect.MySQLDialect;
+import org.hibernate.dialect.SybaseDialect;
 import org.hibernate.reactive.BaseReactiveTest;
 import org.hibernate.reactive.annotations.DisabledFor;
 
@@ -31,7 +36,7 @@
 import static org.hibernate.cfg.AvailableSettings.TIMEZONE_DEFAULT_STORAGE;
 import static org.hibernate.reactive.containers.DatabaseConfiguration.DBType.DB2;
 import static org.hibernate.reactive.testing.ReactiveAssertions.assertWithTruncationThat;
-import static org.hibernate.type.descriptor.DateTimeUtils.roundToDefaultPrecision;
+import static org.hibernate.type.descriptor.DateTimeUtils.adjustToDefaultPrecision;
 
 @Timeout(value = 10, timeUnit = MINUTES)
 @DisabledFor(value = DB2, reason = "Exception: IllegalStateException: Needed to have 6 in buffer but only had 0")
@@ -51,8 +56,24 @@ protected void setProperties(Configuration configuration) {
 
 	@Test
 	public void test(VertxTestContext context) {
-		ZonedDateTime nowZoned = ZonedDateTime.now().withZoneSameInstant( ZoneId.of( "CET" ) );
-		OffsetDateTime nowOffset = OffsetDateTime.now().withOffsetSameInstant( ZoneOffset.ofHours( 3 ) );
+		final ZonedDateTime nowZoned;
+		final OffsetDateTime nowOffset;
+		final Dialect dialect = getDialect();
+		if ( dialect instanceof SybaseDialect || dialect instanceof MySQLDialect ) {
+			// Sybase has 1/300th sec precision
+			nowZoned = ZonedDateTime.now().withZoneSameInstant( ZoneId.of("CET") )
+					.with( ChronoField.NANO_OF_SECOND, 0L );
+			nowOffset = OffsetDateTime.now().withOffsetSameInstant( ZoneOffset.ofHours(3) )
+					.with( ChronoField.NANO_OF_SECOND, 0L );
+		}
+		else if ( dialect.getDefaultTimestampPrecision() == 6 ) {
+			nowZoned = ZonedDateTime.now().withZoneSameInstant( ZoneId.of("CET") ).truncatedTo( ChronoUnit.MICROS );
+			nowOffset = OffsetDateTime.now().withOffsetSameInstant( ZoneOffset.ofHours(3) ).truncatedTo( ChronoUnit.MICROS );
+		}
+		else {
+			nowZoned = ZonedDateTime.now().withZoneSameInstant( ZoneId.of("CET") );
+			nowOffset = OffsetDateTime.now().withOffsetSameInstant( ZoneOffset.ofHours(3) );
+		}
 		test( context, getSessionFactory()
 				.withTransaction( s -> {
 					Zoned z = new Zoned();
@@ -63,11 +84,11 @@ public void test(VertxTestContext context) {
 				.thenCompose( zid -> openSession()
 						.thenCompose( s -> s.find( Zoned.class, zid )
 								.thenAccept( z -> {
-									assertWithTruncationThat( roundToDefaultPrecision( z.zonedDateTime.toInstant(), getDialect() ) )
-											.isEqualTo( roundToDefaultPrecision( nowZoned.toInstant(), getDialect() ) );
+									assertWithTruncationThat( adjustToDefaultPrecision( z.zonedDateTime.toInstant(), getDialect() ) )
+											.isEqualTo( adjustToDefaultPrecision( nowZoned.toInstant(), getDialect() ) );
 
-									assertWithTruncationThat( roundToDefaultPrecision( z.offsetDateTime.toInstant(), getDialect() ) )
-											.isEqualTo( roundToDefaultPrecision( nowOffset.toInstant(), getDialect() ) );
+									assertWithTruncationThat( adjustToDefaultPrecision( z.offsetDateTime.toInstant(), getDialect() ) )
+											.isEqualTo( adjustToDefaultPrecision( nowOffset.toInstant(), getDialect() ) );
 
 									ZoneId systemZone = ZoneId.systemDefault();
 									ZoneOffset systemOffset = systemZone.getRules().getOffset( Instant.now() );