From dc24d2ceddf2d3dfb906ef864b4f74b337b880db Mon Sep 17 00:00:00 2001 From: Jens Schauder Date: Wed, 14 Feb 2018 12:38:53 +0100 Subject: [PATCH] DATACMNS-1259 - Polishing. Refactored lambdas to method references. Simplified type parameter. Added comment to test to indicate related issue. Original pull request: #273. --- .../data/auditing/DefaultAuditableBeanWrapperFactory.java | 6 +++--- .../MappingAuditableBeanWrapperFactoryUnitTests.java | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/springframework/data/auditing/DefaultAuditableBeanWrapperFactory.java b/src/main/java/org/springframework/data/auditing/DefaultAuditableBeanWrapperFactory.java index 8ecb3fbaa..e30d007c4 100644 --- a/src/main/java/org/springframework/data/auditing/DefaultAuditableBeanWrapperFactory.java +++ b/src/main/java/org/springframework/data/auditing/DefaultAuditableBeanWrapperFactory.java @@ -113,7 +113,7 @@ class DefaultAuditableBeanWrapperFactory implements AuditableBeanWrapperFactory public TemporalAccessor setCreatedDate(TemporalAccessor value) { auditable.setCreatedDate( - getAsTemporalAccessor(Optional.of(value), type).orElseThrow(() -> new IllegalStateException())); + getAsTemporalAccessor(Optional.of(value), type).orElseThrow(IllegalStateException::new)); return value; } @@ -147,7 +147,7 @@ class DefaultAuditableBeanWrapperFactory implements AuditableBeanWrapperFactory public TemporalAccessor setLastModifiedDate(TemporalAccessor value) { auditable.setLastModifiedDate( - getAsTemporalAccessor(Optional.of(value), type).orElseThrow(() -> new IllegalStateException())); + getAsTemporalAccessor(Optional.of(value), type).orElseThrow(IllegalStateException::new)); return value; } @@ -220,7 +220,7 @@ class DefaultAuditableBeanWrapperFactory implements AuditableBeanWrapperFactory * @return */ @SuppressWarnings("unchecked") - protected Optional getAsTemporalAccessor(Optional source, + protected Optional getAsTemporalAccessor(Optional source, Class target) { return source.map(it -> { diff --git a/src/test/java/org/springframework/data/auditing/MappingAuditableBeanWrapperFactoryUnitTests.java b/src/test/java/org/springframework/data/auditing/MappingAuditableBeanWrapperFactoryUnitTests.java index ebd84a293..33d9b28b6 100755 --- a/src/test/java/org/springframework/data/auditing/MappingAuditableBeanWrapperFactoryUnitTests.java +++ b/src/test/java/org/springframework/data/auditing/MappingAuditableBeanWrapperFactoryUnitTests.java @@ -163,7 +163,7 @@ public class MappingAuditableBeanWrapperFactoryUnitTests { ThreeTenBackPortConverters.LocalDateTimeToJsr310LocalDateTimeConverter.INSTANCE.convert(reference)); } - @Test + @Test // DATACMNS-1109 public void exposesInstantAsModificationDate() { SampleWithInstant sample = new SampleWithInstant();