Polishing.

Use consistently domain type instead of introducing a new terminology to repository infrastructure.

Rename Kotlin variant of ParameterUnitTests to KParameterUnitTests to avoid duplicate classes.

See #2770
Original pull request: #2771

Author: mp911de

src/main/java/org/springframework/data/repository/query/Parameter.java

@@ -79,19 +79,20 @@ protected Parameter(MethodParameter parameter) {
 	}
 
 	/**
-	 * Creates a new {@link Parameter} for the given {@link MethodParameter} and aggregate {@link TypeInformation}.
+	 * Creates a new {@link Parameter} for the given {@link MethodParameter} and domain {@link TypeInformation}.
 	 *
 	 * @param parameter must not be {@literal null}.
-	 * @param aggregateType must not be {@literal null}.
+	 * @param domainType must not be {@literal null}.
+	 * @since 3.0.2
 	 */
-	protected Parameter(MethodParameter parameter, TypeInformation<?> aggregateType) {
+	protected Parameter(MethodParameter parameter, TypeInformation<?> domainType) {
 
 		Assert.notNull(parameter, "MethodParameter must not be null");
-		Assert.notNull(aggregateType, "TypeInformation must not be null!");
+		Assert.notNull(domainType, "TypeInformation must not be null!");
 
 		this.parameter = parameter;
 		this.parameterType = potentiallyUnwrapParameterType(parameter);
-		this.isDynamicProjectionParameter = isDynamicProjectionParameter(parameter, aggregateType);
+		this.isDynamicProjectionParameter = isDynamicProjectionParameter(parameter, domainType);
 		this.name = isSpecialParameterType(parameter.getParameterType()) ? Lazy.of(Optional.empty()) : Lazy.of(() -> {
 			Param annotation = parameter.getParameterAnnotation(Param.class);
 			return Optional.ofNullable(annotation == null ? parameter.getParameterName() : annotation.value());
@@ -218,10 +219,10 @@ boolean isSort() {
 	 * </code>
 	 *
 	 * @param parameter must not be {@literal null}.
-	 * @param aggregateType the reference aggregate type, must not be {@literal null}.
+	 * @param domainType the reference domain type, must not be {@literal null}.
 	 * @return
 	 */
-	private static boolean isDynamicProjectionParameter(MethodParameter parameter, TypeInformation<?> aggregateType) {
+	private static boolean isDynamicProjectionParameter(MethodParameter parameter, TypeInformation<?> domainType) {
 
 		if (!parameter.getParameterType().equals(Class.class)) {
 			return false;
@@ -241,7 +242,7 @@ private static boolean isDynamicProjectionParameter(MethodParameter parameter, T
 		var unwrapped = QueryExecutionConverters.unwrapWrapperTypes(returnType);
 		var reactiveUnwrapped = ReactiveWrapperConverters.unwrapWrapperTypes(unwrapped);
 
-		if (aggregateType.isAssignableFrom(reactiveUnwrapped)) {
+		if (domainType.isAssignableFrom(reactiveUnwrapped)) {
 			return false;
 		}
 

src/main/java/org/springframework/data/repository/query/Parameters.java

@@ -77,6 +77,7 @@ public Parameters(Method method) {
 	 *
 	 * @param method must not be {@literal null}.
 	 * @param parameterFactory must not be {@literal null}.
+	 * @since 3.0.2
 	 */
 	protected Parameters(Method method, Function<MethodParameter, T> parameterFactory) {
 
@@ -176,7 +177,7 @@ private S getBindable() {
 	 *
 	 * @param parameter will never be {@literal null}.
 	 * @return
-	 * @deprecated since 3.1, in your extension, call {@link #Parameters(Method, ParameterFactory)} instead.
+	 * @deprecated since 3.1, in your extension, call {@link #Parameters(Method, Function)} instead.
 	 */
 	@SuppressWarnings("unchecked")
 	@Deprecated(since = "3.1", forRemoval = true)

src/main/java/org/springframework/data/repository/query/QueryMethod.java

@@ -69,12 +69,10 @@ public QueryMethod(Method method, RepositoryMetadata metadata, ProjectionFactory
 		Assert.notNull(metadata, "Repository metadata must not be null");
 		Assert.notNull(factory, "ProjectionFactory must not be null");
 
-		Parameters.TYPES.stream()
-				.filter(type -> getNumberOfOccurrences(method, type) > 1)
-				.findFirst().ifPresent(type -> {
-					throw new IllegalStateException(
-							String.format("Method must have only one argument of type %s; Offending method: %s",
-									type.getSimpleName(), method));
+		Parameters.TYPES.stream() //
+				.filter(type -> getNumberOfOccurrences(method, type) > 1).findFirst().ifPresent(type -> {
+					throw new IllegalStateException(String.format(
+							"Method must have only one argument of type %s; Offending method: %s", type.getSimpleName(), method));
 				});
 
 		this.method = method;
@@ -132,11 +130,12 @@ public QueryMethod(Method method, RepositoryMetadata metadata, ProjectionFactory
 	 * Creates a {@link Parameters} instance.
 	 *
 	 * @param method must not be {@literal null}.
-	 * @param aggregateType must not be {@literal null}.
+	 * @param domainType must not be {@literal null}.
 	 * @return must not return {@literal null}.
+	 * @since 3.0.2
 	 */
-	protected Parameters<?, ?> createParameters(Method method, TypeInformation<?> aggregateType) {
-		return new DefaultParameters(method, aggregateType);
+	protected Parameters<?, ?> createParameters(Method method, TypeInformation<?> domainType) {
+		return new DefaultParameters(method, domainType);
 	}
 
 	/**

src/test/java/org/springframework/data/repository/query/ParameterUnitTests.java

@@ -20,6 +20,7 @@
 import java.util.Collections;
 import java.util.List;
 import java.util.Optional;
+import java.util.function.Function;
 import java.util.stream.Stream;
 
 import org.jetbrains.annotations.NotNull;
@@ -34,6 +35,7 @@
  * Unit tests for {@link Parameter}.
  *
  * @author Jens Schauder
+ * @author Oliver Drotbohm
  */
 class ParameterUnitTests {
 
@@ -69,9 +71,9 @@ Stream<DynamicTest> doesNotConsiderClassParametersDynamicProjectionOnes() {
 				"staticReturnNonDynamicBindWildcard", //
 				"staticReturnNonDynamicBindWildcardExtends");
 
-		return DynamicTest.stream(methods, it -> it, it -> {
-			assertThat(new Parameter(getMethodParameter(it), TypeInformation.of(User.class))
-					.isDynamicProjectionParameter()).isFalse();
+		return DynamicTest.stream(methods, Function.identity(), it -> {
+			Parameter parameter = new Parameter(getMethodParameter(it), TypeInformation.of(User.class));
+			assertThat(parameter.isDynamicProjectionParameter()).isFalse();
 		});
 	}
 

src/test/kotlin/org/springframework/data/repository/query/ParameterUnitTests.kt renamed to src/test/kotlin/org/springframework/data/repository/query/KParameterUnitTests.kt

@@ -26,17 +26,17 @@ import kotlin.reflect.jvm.javaMethod
  *
  * @author Mark Paluch
  */
-class ParameterUnitTests {
+class KParameterUnitTests {
 
-	@Test // DATACMNS-1508
-	fun `should consider Continuation a special parameter`() {
+    @Test // DATACMNS-1508
+    fun `should consider Continuation a special parameter`() {
 
-		val methodParameter =
-			MethodParameter(MyCoroutineRepository::hello.javaMethod!!, 0)
-		methodParameter.initParameterNameDiscovery(DefaultParameterNameDiscoverer())
-		val parameter = Parameter(methodParameter)
+        val methodParameter =
+            MethodParameter(MyCoroutineRepository::hello.javaMethod!!, 0)
+        methodParameter.initParameterNameDiscovery(DefaultParameterNameDiscoverer())
+        val parameter = Parameter(methodParameter)
 
-		assertThat(parameter.name).isEmpty()
+        assertThat(parameter.name).isEmpty()
 		assertThat(parameter.isBindable).isFalse()
 	}