Polishing.

Reformat code, tweak visibility. See #2147

Author: mp911de

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/dialect/JdbcSqlServerDialectTest.java

@@ -15,11 +15,13 @@
  */
 package org.springframework.data.jdbc.core.dialect;
 
+import static org.assertj.core.api.Assertions.*;
+
 import java.time.Instant;
 import java.util.List;
 
-import org.assertj.core.api.Assertions;
 import org.junit.jupiter.api.Test;
+
 import org.springframework.data.jdbc.core.convert.JdbcCustomConversions;
 
 /**
@@ -29,14 +31,12 @@
  */
 class JdbcSqlServerDialectTest {
 
- @Test // GH-1873
- void testCustomConversions() {
+@Test // GH-1873
+void testCustomConversions() {
 
- JdbcCustomConversions jdbcCustomConversions = new JdbcCustomConversions(
- (List<?>) JdbcSqlServerDialect.INSTANCE.getConverters());
+JdbcCustomConversions conversions = JdbcCustomConversions.of(JdbcSqlServerDialect.INSTANCE, List.of());
 
- Assertions
- .assertThat(jdbcCustomConversions.hasCustomReadTarget(microsoft.sql.DateTimeOffset.class, Instant.class))
- .isTrue();
- }
-}
+assertThat(conversions.hasCustomReadTarget(microsoft.sql.DateTimeOffset.class, Instant.class))
+.isTrue();
+}
+}

spring-data-relational/src/test/java/org/springframework/data/relational/core/dialect/SqlServerDialectUnitTests.java

@@ -30,10 +30,10 @@
  * @author Mark Paluch
  * @author Myeonghyeon Lee
  */
-public class SqlServerDialectUnitTests {
+class SqlServerDialectUnitTests {
 
 @Test // DATAJDBC-278
-public void shouldNotSupportArrays() {
+void shouldNotSupportArrays() {
 
 ArrayColumns arrayColumns = SqlServerDialect.INSTANCE.getArraySupport();
 
@@ -42,7 +42,7 @@ public void shouldNotSupportArrays() {
 }
 
 @Test // DATAJDBC-278
-public void shouldRenderLimit() {
+void shouldRenderLimit() {
 
 LimitClause limit = SqlServerDialect.INSTANCE.limit();
 
@@ -51,23 +51,23 @@ public void shouldRenderLimit() {
 }
 
 @Test // DATAJDBC-278
-public void shouldRenderOffset() {
+void shouldRenderOffset() {
 
 LimitClause limit = SqlServerDialect.INSTANCE.limit();
 
 assertThat(limit.getOffset(10)).isEqualTo("OFFSET 10 ROWS");
 }
 
 @Test // DATAJDBC-278
-public void shouldRenderLimitOffset() {
+void shouldRenderLimitOffset() {
 
 LimitClause limit = SqlServerDialect.INSTANCE.limit();
 
 assertThat(limit.getLimitOffset(20, 10)).isEqualTo("OFFSET 10 ROWS FETCH NEXT 20 ROWS ONLY");
 }
 
 @Test // DATAJDBC-498
-public void shouldRenderLock() {
+void shouldRenderLock() {
 
 LockClause lock = SqlServerDialect.INSTANCE.lock();
 From from = mock(From.class);