Polishing.

Introduce factory methods to create mapping contexts that use quoting and that use plain identifiers for easier creation of the correct mapping context.

See #1993
Original pull request #2066

Author: mp911de

spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/mapping/JdbcMappingContext.java

@@ -54,6 +54,61 @@ public JdbcMappingContext(NamingStrategy namingStrategy) {
 		setSimpleTypeHolder(JdbcSimpleTypes.HOLDER);
 	}
 
+	/**
+	 * Create a new {@code JdbcMappingContext} using {@link #setForceQuote(boolean) plain identifiers}. Plain
+	 * {@link org.springframework.data.relational.core.sql.SqlIdentifier identifiers} (i.e. table and column names) are
+	 * typically not case-sensitive (case-sensitivity can be still enforced by specific database configurations).
+	 *
+	 * @return a new {@code JdbcMappingContext} using plain identifiers.
+	 * @since 4.0
+	 */
+	public static JdbcMappingContext forPlainIdentifiers() {
+		JdbcMappingContext context = forQuotedIdentifiers();
+		context.setForceQuote(false);
+		return context;
+	}
+
+	/**
+	 * Create a new {@code JdbcMappingContext} using {@link #setForceQuote(boolean) plain identifiers} and the given
+	 * {@link NamingStrategy}. Plain {@link org.springframework.data.relational.core.sql.SqlIdentifier identifiers} (i.e.
+	 * table and column names) are typically not case-sensitive (case-sensitivity can be still enforced by specific
+	 * database configurations).
+	 *
+	 * @param namingStrategy must not be {@literal null}.
+	 * @return a new {@code JdbcMappingContext} using plain identifiers.
+	 * @since 4.0
+	 */
+	public static JdbcMappingContext forPlainIdentifiers(NamingStrategy namingStrategy) {
+		JdbcMappingContext context = forQuotedIdentifiers(namingStrategy);
+		context.setForceQuote(false);
+		return context;
+	}
+
+	/**
+	 * Create a new {@code JdbcMappingContext} using {@link #setForceQuote(boolean) quoted identifiers} (default
+	 * behavior). Quoted {@link org.springframework.data.relational.core.sql.SqlIdentifier identifiers} (i.e. table and
+	 * column names) are typically case-sensitive.
+	 *
+	 * @return a new {@code JdbcMappingContext} using quoted identifiers.
+	 * @since 4.0
+	 */
+	public static JdbcMappingContext forQuotedIdentifiers() {
+		return new JdbcMappingContext();
+	}
+
+	/**
+	 * Create a new {@code JdbcMappingContext} using {@link #setForceQuote(boolean) quoted identifiers} (default behavior)
+	 * and the given {@link NamingStrategy}. Quoted {@link org.springframework.data.relational.core.sql.SqlIdentifier
+	 * identifiers} (i.e. table and column names) are typically case-sensitive.
+	 *
+	 * @param namingStrategy must not be {@literal null}.
+	 * @return a new {@code JdbcMappingContext} using quoted identifiers.
+	 * @since 4.0
+	 */
+	public static JdbcMappingContext forQuotedIdentifiers(NamingStrategy namingStrategy) {
+		return new JdbcMappingContext(namingStrategy);
+	}
+
 	@Override
 	protected RelationalPersistentProperty createPersistentProperty(Property property,
 			RelationalPersistentEntity<?> owner, SimpleTypeHolder simpleTypeHolder) {

spring-data-jdbc/src/main/java/org/springframework/data/jdbc/repository/config/AbstractJdbcConfiguration.java

@@ -116,7 +116,8 @@ public RelationalManagedTypes jdbcManagedTypes() throws ClassNotFoundException {
 	public JdbcMappingContext jdbcMappingContext(Optional<NamingStrategy> namingStrategy,
 			JdbcCustomConversions customConversions, RelationalManagedTypes jdbcManagedTypes) {
 
-		JdbcMappingContext mappingContext = new JdbcMappingContext(namingStrategy.orElse(DefaultNamingStrategy.INSTANCE));
+		JdbcMappingContext mappingContext = JdbcMappingContext
+				.forQuotedIdentifiers(namingStrategy.orElse(DefaultNamingStrategy.INSTANCE));
 		mappingContext.setSimpleTypeHolder(customConversions.getSimpleTypeHolder());
 		mappingContext.setManagedTypes(jdbcManagedTypes);
 

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/EntityRowMapperUnitTests.java

@@ -1032,7 +1032,7 @@ private <T> EntityRowMapper<T> createRowMapper(Class<T> type) {
 	@SuppressWarnings("unchecked")
 	private <T> EntityRowMapper<T> createRowMapper(Class<T> type, NamingStrategy namingStrategy) {
 
-		RelationalMappingContext context = new JdbcMappingContext(namingStrategy);
+		RelationalMappingContext context = JdbcMappingContext.forQuotedIdentifiers(namingStrategy);
 
 		DataAccessStrategy accessStrategy = mock(DataAccessStrategy.class);
 

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/JdbcIdentifierBuilderUnitTests.java

@@ -40,7 +40,7 @@
  */
 class JdbcIdentifierBuilderUnitTests {
 
-	JdbcMappingContext context = new JdbcMappingContext();
+	JdbcMappingContext context = JdbcMappingContext.forQuotedIdentifiers();
 	JdbcConverter converter = new MappingJdbcConverter(context, (identifier, path) -> {
 		throw new UnsupportedOperationException();
 	});

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/MappingJdbcConverterUnitTests.java

@@ -165,7 +165,7 @@ void accessesCorrectValuesForOneToOneRelationshipWithIdenticallyNamedIdPropertie
 	@Test // GH-1750
 	void readByteArrayToNestedUuidWithCustomConverter() {
 
-		JdbcMappingContext context = new JdbcMappingContext();
+		JdbcMappingContext context = JdbcMappingContext.forQuotedIdentifiers();
 		StubbedJdbcTypeFactory typeFactory = new StubbedJdbcTypeFactory();
 		Converter<byte[], UUID> customConverter = new ByteArrayToUuid();
 		MappingJdbcConverter converter = new MappingJdbcConverter( //

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/RowDocumentResultSetExtractorUnitTests.java

@@ -48,7 +48,7 @@
  */
 public class RowDocumentResultSetExtractorUnitTests {
 
-	RelationalMappingContext context = new JdbcMappingContext(new DefaultNamingStrategy());
+	RelationalMappingContext context = JdbcMappingContext.forQuotedIdentifiers(new DefaultNamingStrategy());
 
 	private final PathToColumnMapping column = new PathToColumnMapping() {
 		@Override

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/SqlGeneratorContextBasedNamingStrategyUnitTests.java

@@ -212,7 +212,7 @@ private void threadedTest(String user, CountDownLatch latch, Consumer<String> te
 	 */
 	private SqlGenerator configureSqlGenerator(NamingStrategy namingStrategy) {
 
-		RelationalMappingContext context = new JdbcMappingContext(namingStrategy);
+		RelationalMappingContext context = JdbcMappingContext.forQuotedIdentifiers(namingStrategy);
 		JdbcConverter converter = new MappingJdbcConverter(context, (identifier, path) -> {
 			throw new UnsupportedOperationException();
 		});

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/SqlGeneratorEmbeddedUnitTests.java

@@ -46,15 +46,14 @@
  */
 class SqlGeneratorEmbeddedUnitTests {
 
-	private RelationalMappingContext context = new JdbcMappingContext();
+	private RelationalMappingContext context = JdbcMappingContext.forPlainIdentifiers();
 	private JdbcConverter converter = new MappingJdbcConverter(context, (identifier, path) -> {
 		throw new UnsupportedOperationException();
 	});
 	private SqlGenerator sqlGenerator;
 
 	@BeforeEach
 	void setUp() {
-		this.context.setForceQuote(false);
 		this.sqlGenerator = createSqlGenerator(DummyEntity.class);
 	}
 

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/SqlGeneratorFixedNamingStrategyUnitTests.java

@@ -196,7 +196,7 @@ private PersistentPropertyPath<RelationalPersistentProperty> getPath(String path
 	 */
 	private SqlGenerator configureSqlGenerator(NamingStrategy namingStrategy) {
 
-		context = new JdbcMappingContext(namingStrategy);
+		context = JdbcMappingContext.forQuotedIdentifiers(namingStrategy);
 		JdbcConverter converter = new MappingJdbcConverter(context, (identifier, path) -> {
 			throw new UnsupportedOperationException();
 		});

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/SqlGeneratorUnitTests.java

@@ -15,17 +15,11 @@
  */
 package org.springframework.data.jdbc.core.convert;
 
-import static java.util.Collections.emptySet;
-import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
-import static org.assertj.core.api.Assertions.assertThatThrownBy;
-import static org.assertj.core.api.Assertions.entry;
-import static org.assertj.core.api.SoftAssertions.assertSoftly;
-import static org.springframework.data.relational.core.mapping.ForeignKeyNaming.APPLY_RENAMING;
-import static org.springframework.data.relational.core.mapping.ForeignKeyNaming.IGNORE_RENAMING;
-import static org.springframework.data.relational.core.sql.SqlIdentifier.EMPTY;
-import static org.springframework.data.relational.core.sql.SqlIdentifier.quoted;
-import static org.springframework.data.relational.core.sql.SqlIdentifier.unquoted;
+import static java.util.Collections.*;
+import static org.assertj.core.api.Assertions.*;
+import static org.assertj.core.api.SoftAssertions.*;
+import static org.springframework.data.relational.core.mapping.ForeignKeyNaming.*;
+import static org.springframework.data.relational.core.sql.SqlIdentifier.*;
 
 import java.util.Map;
 import java.util.Set;
@@ -87,7 +81,7 @@ class SqlGeneratorUnitTests {
 	private static final Identifier BACKREF = Identifier.of(unquoted("backref"), "some-value", String.class);
 
 	private final PrefixingNamingStrategy namingStrategy = new PrefixingNamingStrategy();
-	private RelationalMappingContext context = new JdbcMappingContext(namingStrategy);
+	private RelationalMappingContext context = JdbcMappingContext.forQuotedIdentifiers(namingStrategy);
 	private final JdbcConverter converter = new MappingJdbcConverter(context, (identifier, path) -> {
 		throw new UnsupportedOperationException();
 	});
@@ -977,7 +971,7 @@ void selectByQueryPaginationValidTest() {
 	void backReferenceShouldConsiderRenamedParent() {
 
 		namingStrategy.setForeignKeyNaming(APPLY_RENAMING);
-		context = new JdbcMappingContext(namingStrategy);
+		context = JdbcMappingContext.forQuotedIdentifiers(namingStrategy);
 
 		String sql = sqlGenerator.createDeleteInByPath(getPath("ref", RenamedDummy.class));
 
@@ -988,7 +982,7 @@ void backReferenceShouldConsiderRenamedParent() {
 	void backReferenceShouldIgnoreRenamedParent() {
 
 		namingStrategy.setForeignKeyNaming(IGNORE_RENAMING);
-		context = new JdbcMappingContext(namingStrategy);
+		context = JdbcMappingContext.forQuotedIdentifiers(namingStrategy);
 
 		String sql = sqlGenerator.createDeleteInByPath(getPath("ref", RenamedDummy.class));
 
@@ -999,7 +993,7 @@ void backReferenceShouldIgnoreRenamedParent() {
 	void keyColumnShouldConsiderRenamedParent() {
 
 		namingStrategy.setForeignKeyNaming(APPLY_RENAMING);
-		context = new JdbcMappingContext(namingStrategy);
+		context = JdbcMappingContext.forQuotedIdentifiers(namingStrategy);
 
 		SqlGenerator sqlGenerator = createSqlGenerator(ReferencedEntity.class);
 		String sql = sqlGenerator.getFindAllByProperty(Identifier.of(unquoted("parentId"), 23, RenamedDummy.class),
@@ -1012,7 +1006,7 @@ void keyColumnShouldConsiderRenamedParent() {
 	void keyColumnShouldIgnoreRenamedParent() {
 
 		namingStrategy.setForeignKeyNaming(IGNORE_RENAMING);
-		context = new JdbcMappingContext(namingStrategy);
+		context = JdbcMappingContext.forQuotedIdentifiers(namingStrategy);
 
 		SqlGenerator sqlGenerator = createSqlGenerator(ReferencedEntity.class);
 		String sql = sqlGenerator.getFindAllByProperty(Identifier.of(unquoted("parentId"), 23, RenamedDummy.class),