From 3f127347ceeb042f6624c9ce956cc31555bc376d Mon Sep 17 00:00:00 2001
From: Alexander Kriegisch <Alexander@Kriegisch.name>
Date: Wed, 28 Apr 2021 13:54:33 +0700
Subject: Fix missing dependencies in module 'run-all-junit-tests'

Some runtime dependencies are reported as unused in Maven Dependency
Plugin goal 'dependency:analyze', but actually they are needed. I
noticed by chance when running RunTheseBeforeYouCommitTests in IntelliJ
IDEA for the first time after a while and dependency modules could not
find classes.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
---
 run-all-junit-tests/pom.xml | 54 +++++++++++++++++++++++++++++++++++++++++----
 1 file changed, 50 insertions(+), 4 deletions(-)

(limited to 'run-all-junit-tests')

diff --git a/run-all-junit-tests/pom.xml b/run-all-junit-tests/pom.xml
index fe7d15dfb..fb21ee685 100644
--- a/run-all-junit-tests/pom.xml
+++ b/run-all-junit-tests/pom.xml
@@ -12,10 +12,7 @@
 	<artifactId>run-all-junit-tests</artifactId>
 
 	<dependencies>
-		<dependency>
-			<groupId>org.aspectj</groupId>
-			<artifactId>asm-renamed</artifactId>
-		</dependency>
+
 		<dependency>
 			<groupId>org.aspectj</groupId>
 			<artifactId>util</artifactId>
@@ -142,6 +139,41 @@
 			<type>test-jar</type>
 			<scope>test</scope>
 		</dependency>
+
+		<!--
+			The tests need these during runtime, even though no direct usage is in our classes.
+			See also 'usedDependencies' in maven-dependency-plugin configuration.
+		-->
+		<dependency>
+			<groupId>ant</groupId>
+			<artifactId>ant-launcher</artifactId>
+			<version>${lib.ant.version}</version>
+			<scope>test</scope>
+		</dependency>
+		<dependency>
+			<groupId>org.aspectj</groupId>
+			<artifactId>asm-renamed</artifactId>
+			<scope>test</scope>
+		</dependency>
+		<dependency>
+			<groupId>org.aspectj</groupId>
+			<artifactId>ajde</artifactId>
+			<version>${project.version}</version>
+			<scope>test</scope>
+		</dependency>
+		<dependency>
+			<groupId>org.aspectj</groupId>
+			<artifactId>build</artifactId>
+			<version>${project.version}</version>
+			<scope>test</scope>
+		</dependency>
+		<dependency>
+			<groupId>org.aspectj</groupId>
+			<artifactId>tests</artifactId>
+			<version>${project.version}</version>
+			<scope>test</scope>
+		</dependency>
+
 	</dependencies>
 
 	<profiles>
@@ -200,6 +232,20 @@
 					</execution>
 				</executions>
 			</plugin>
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-dependency-plugin</artifactId>
+				<configuration>
+					<usedDependencies>
+						<!-- The tests need these during runtime, even though no direct usage is in our classes -->
+						<usedDependency>ant:ant-launcher</usedDependency>
+						<usedDependency>org.aspectj:asm-renamed</usedDependency>
+						<usedDependency>org.aspectj:ajde</usedDependency>
+						<usedDependency>org.aspectj:build</usedDependency>
+						<usedDependency>org.aspectj:tests</usedDependency>
+					</usedDependencies>
+				</configuration>
+			</plugin>
 		</plugins>
 	</build>
 
-- 
cgit v1.2.3