From: Alexander Kriegisch <Alexander@Kriegisch.name>
Date: Mon, 22 Mar 2021 19:22:08 +0000 (+0700)
Subject: Deactivate test run in 'run-all-unit-tests' module by default
X-Git-Tag: java16-add-opens~2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=acdf4ffd9d4c6ee0b8d8595a112c5331974e08cb;p=aspectj.git

Deactivate test run in 'run-all-unit-tests' module by default

There is a new Maven profile 'repeat-all-unit-tests', and the name
already implies what a comment in the Maven module explains like this:
ATTENTION: This profile is inactive by default, because when active and
running a full reactor build, it makes almost all tests run 2x, doubling
the build time without any added value. This Maven module only exists
for convenience: As a developer, your IDE can detect and run
'RunTheseBeforeYouCommitTests'. That way, you do not have to use Maven
and get the test results reported within the IDE's JUnit user interface.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
---

diff --git a/run-all-junit-tests/pom.xml b/run-all-junit-tests/pom.xml
index 97347e3eb..bcb21ca4e 100644
--- a/run-all-junit-tests/pom.xml
+++ b/run-all-junit-tests/pom.xml
@@ -269,12 +269,62 @@
 		</dependency>
 	</dependencies>
 
+	<profiles>
+		<profile>
+			<!--
+				ATTENTION: This profile is inactive by default, because when active and running a full reactor build, it makes
+				almost all tests run 2x, doubling the build time without any added value. This Maven module only exists for
+				convenience: As a developer, your IDE can detect and run 'RunTheseBeforeYouCommitTests'. That way, you do not
+				have to use Maven and get the test results reported within the IDE's JUnit user interface.
+			-->
+			<id>repeat-all-unit-tests</id>
+			<activation>
+				<activeByDefault>false</activeByDefault>
+			</activation>
+			<build>
+				<plugins>
+					<plugin>
+						<groupId>org.apache.maven.plugins</groupId>
+						<artifactId>maven-surefire-plugin</artifactId>
+						<configuration>
+							<testFailureIgnore>true</testFailureIgnore>
+						</configuration>
+						<executions>
+							<execution>
+								<id>extra-test-run</id>
+								<phase>test</phase>
+								<goals>
+									<goal>test</goal>
+								</goals>
+							</execution>
+						</executions>
+					</plugin>
+				</plugins>
+			</build>
+		</profile>
+	</profiles>
+
+	<build>
+		<plugins>
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-surefire-plugin</artifactId>
+				<configuration>
+					<testFailureIgnore>true</testFailureIgnore>
+				</configuration>
+				<executions>
+					<execution>
+						<id>default-test</id>
+						<!-- By default, do not run any tests -->
+						<phase>none</phase>
+						<goals>
+							<goal>test</goal>
+						</goals>
+					</execution>
+				</executions>
+			</plugin>
+		</plugins>
+	</build>
 
-	<!-- <build> <pluginManagement> <plugins> <plugin> <groupId>org.apache.maven.plugins</groupId> 
-		<artifactId>maven-compiler-plugin</artifactId> <configuration> <source>1.8</source> 
-		<target>1.8</target> </configuration> </plugin> <plugin> <groupId>org.apache.maven.plugins</groupId> 
-		<artifactId>maven-surefire-plugin</artifactId> <configuration> <testFailureIgnore>true</testFailureIgnore> 
-		<includes> <include>**/RunTheseBeforeYouCommitTests*</include> </includes> 
-		</configuration> </plugin> </plugins> </pluginManagement> </build> -->
 </project>