From 2f9d4b2c8354b54ee5d3a8e5b03d76a315083df9 Mon Sep 17 00:00:00 2001 From: Jens Schauder Date: Thu, 30 Jan 2020 09:27:00 +0100 Subject: [PATCH] DATAJDBC-481 - Improved test for optimistic locking version increase. --- .../jdbc/core/JdbcAggregateTemplateIntegrationTests.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/JdbcAggregateTemplateIntegrationTests.java b/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/JdbcAggregateTemplateIntegrationTests.java index af89e71e..8a2a69ac 100644 --- a/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/JdbcAggregateTemplateIntegrationTests.java +++ b/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/JdbcAggregateTemplateIntegrationTests.java @@ -76,6 +76,7 @@ import org.springframework.transaction.annotation.Transactional; * @author Myeonghyeon Lee * @author Tom Hombergs * @author Tyler Van Gorder + * @author Clemens Hahn */ @ContextConfiguration @Transactional @@ -700,6 +701,7 @@ public class JdbcAggregateTemplateIntegrationTests { AggregateWithImmutableVersion aggregate = new AggregateWithImmutableVersion(null, null); aggregate = template.save(aggregate); + assertThat(aggregate.version).isEqualTo(1L); Long id = aggregate.getId(); @@ -707,9 +709,12 @@ public class JdbcAggregateTemplateIntegrationTests { assertThat(reloadedAggregate.getVersion()).describedAs("version field should initially have the value 1") .isEqualTo(1L); - template.save(reloadedAggregate); - + AggregateWithImmutableVersion saved = template.save(reloadedAggregate); AggregateWithImmutableVersion updatedAggregate = template.findById(id, aggregate.getClass()); + + assertThat(saved.version) + .describedAs("returned by save(): "+ saved + " vs. returned by findById(): " + updatedAggregate) + .isEqualTo(updatedAggregate.version); assertThat(updatedAggregate.getVersion()).describedAs("version field should increment by one with each save") .isEqualTo(2L);