DATAJDBC-484 - Fixes a JDK dependent test failure.

schauder · schauder · commit 5ba898fe3441 · 2020-02-07T15:27:51.000+01:00
Beginning from JDK 10 `Instant` uses nanosecond precision which doesn't make it through the database.
This caused the test to fail.
This fix fixes only the test by making the assertion more lenient.
Actually storing and retrieving nanosecond precision time values is a separate issue.
diff --git a/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/repository/JdbcRepositoryIntegrationTests.java b/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/repository/JdbcRepositoryIntegrationTests.java
@@ -278,7 +278,9 @@ public void executeQueryWithParameterRequiringConversion() {
 
 		repository.saveAll(asList(first, second));
 
-		assertThat(repository.after(now)).containsExactly(second);
+		assertThat(repository.after(now)) //
+				.extracting(DummyEntity::getName) //
+				.containsExactly("second");
 	}
 
 	@Test // DATAJDBC-234

Original file line number	Diff line number	Diff line change
`@@ -278,7 +278,9 @@ public void executeQueryWithParameterRequiringConversion() {`
`278`	`278`
`279`	`279`	`repository.saveAll(asList(first, second));`
`280`	`280`
`281`		`- assertThat(repository.after(now)).containsExactly(second);`
	`281`	`+ assertThat(repository.after(now)) //`
	`282`	`+ .extracting(DummyEntity::getName) //`
	`283`	`+ .containsExactly("second");`
`282`	`284`	`}`
`283`	`285`
`284`	`286`	`@Test // DATAJDBC-234`