diff --git a/src/main/java/org/springframework/data/r2dbc/core/DefaultReactiveDataAccessStrategy.java b/src/main/java/org/springframework/data/r2dbc/core/DefaultReactiveDataAccessStrategy.java index fe393e6..75186ad 100644 --- a/src/main/java/org/springframework/data/r2dbc/core/DefaultReactiveDataAccessStrategy.java +++ b/src/main/java/org/springframework/data/r2dbc/core/DefaultReactiveDataAccessStrategy.java @@ -54,6 +54,7 @@ import org.springframework.util.CollectionUtils; * Default {@link ReactiveDataAccessStrategy} implementation. * * @author Mark Paluch + * @author Louis Morgan */ public class DefaultReactiveDataAccessStrategy implements ReactiveDataAccessStrategy { diff --git a/src/test/java/org/springframework/data/r2dbc/core/ReactiveDataAccessStrategyTestSupport.java b/src/test/java/org/springframework/data/r2dbc/core/ReactiveDataAccessStrategyTestSupport.java index a86e353..a91ab91 100644 --- a/src/test/java/org/springframework/data/r2dbc/core/ReactiveDataAccessStrategyTestSupport.java +++ b/src/test/java/org/springframework/data/r2dbc/core/ReactiveDataAccessStrategyTestSupport.java @@ -45,6 +45,7 @@ import org.springframework.data.relational.core.sql.SqlIdentifier; * Abstract base class for {@link R2dbcDialect}-aware {@link DefaultReactiveDataAccessStrategy} tests. * * @author Mark Paluch + * @author Louis Morgan */ public abstract class ReactiveDataAccessStrategyTestSupport { @@ -180,12 +181,14 @@ public abstract class ReactiveDataAccessStrategyTestSupport { @Test // gh-354 public void shouldNotWriteReadOnlyFields() { + TypeWithReadOnlyFields toSave = new TypeWithReadOnlyFields(); + toSave.setWritableField("writable"); toSave.setReadOnlyField("readonly"); toSave.setReadOnlyArrayField("readonly_array".getBytes()); - assertThat(getStrategy().getOutboundRow(toSave)) - .containsOnlyKeys(SqlIdentifier.unquoted("writable_field")); + + assertThat(getStrategy().getOutboundRow(toSave)).containsOnlyKeys(SqlIdentifier.unquoted("writable_field")); } private void testType(BiConsumer setter, Function getter, T testValue, @@ -250,9 +253,7 @@ public abstract class ReactiveDataAccessStrategyTestSupport { @Data static class TypeWithReadOnlyFields { String writableField; - @ReadOnlyProperty - String readOnlyField; - @ReadOnlyProperty - byte[] readOnlyArrayField; + @ReadOnlyProperty String readOnlyField; + @ReadOnlyProperty byte[] readOnlyArrayField; } }