From 08d4e283d422f723bf73e39531e3db905f82136f Mon Sep 17 00:00:00 2001
From: Mark Paluch <mpaluch@vmware.com>
Date: Tue, 14 Sep 2021 12:06:13 +0200
Subject: [PATCH] Disable parameter names for test compilation.

We now no longer use compiled parameter names for tests to avoid interference with Eclipselink's mechanism of using these improperly for stored procedure calls. Especially the fact that a parameter is named "in" lets the database consider using an IN clause instead of the parameter name.

When configuring the compile plugin, we also need to ensure that the AspectJ plugin post-processes the compiled classes and doesn't run before the Maven compiler plugin.

Closes: #2298
See also https://github.com/spring-projects/spring-data-build/issues/1513
---
 pom.xml | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/pom.xml b/pom.xml
index 453c09790..f1605939a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -382,6 +382,22 @@
 				</executions>
 			</plugin>
 
+			<plugin>
+				<artifactId>maven-compiler-plugin</artifactId>
+				<executions>
+					<execution>
+						<id>java-test-compile</id>
+						<phase>test-compile</phase>
+						<goals>
+							<goal>testCompile</goal>
+						</goals>
+						<configuration>
+							<parameters>false</parameters>
+						</configuration>
+					</execution>
+				</executions>
+			</plugin>
+
 			<plugin>
 				<groupId>io.starter</groupId>
 				<artifactId>aspectj-maven-plugin</artifactId>
@@ -403,6 +419,7 @@
 						<goals>
 							<goal>compile</goal>
 						</goals>
+						<phase>process-classes</phase>
 					</execution>
 				</executions>
 				<configuration>