diff options
| author | Alexander Kriegisch <Alexander@Kriegisch.name> | 2021-03-23 01:25:22 +0700 |
|---|---|---|
| committer | Alexander Kriegisch <Alexander@Kriegisch.name> | 2021-03-23 17:03:16 +0700 |
| commit | eac253e74682f126147f1c80ed818e171feea22d (patch) | |
| tree | f7c8f2c50c16c78aed0e055170d09f35719933a3 /run-all-junit-tests | |
| parent | 37cd358d32af3fd8a1b1e09057ae007d2419282b (diff) | |
| download | aspectj-eac253e74682f126147f1c80ed818e171feea22d.tar.gz aspectj-eac253e74682f126147f1c80ed818e171feea22d.zip | |
Revert "Remove module 'run-all-junit-tests' from root POM -> speed up the build"
This reverts commit a1867b05ba6443d32abc4049c26b92fc226d6f78.
Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
Diffstat (limited to 'run-all-junit-tests')
| -rw-r--r-- | run-all-junit-tests/pom.xml | 15 | ||||
| -rw-r--r-- | run-all-junit-tests/src/test/java/RunTheseBeforeYouCommitTests.java | 3 |
2 files changed, 5 insertions, 13 deletions
diff --git a/run-all-junit-tests/pom.xml b/run-all-junit-tests/pom.xml index b55f3b273..97347e3eb 100644 --- a/run-all-junit-tests/pom.xml +++ b/run-all-junit-tests/pom.xml @@ -9,11 +9,6 @@ <version>1.9.7.BUILD-SNAPSHOT</version> </parent> - <!-- - These are redundant, making almost all tests run 2x. You can still open and run the module manually for now. - But why not just run 'mvn test' on top level? - TODO: After feedback from Andy Clement, probably just remove the whole module. - --> <artifactId>run-all-junit-tests</artifactId> <packaging>jar</packaging> <name>run-all-junit-tests</name> @@ -275,11 +270,11 @@ </dependencies> - <!-- <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> + <!-- <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> diff --git a/run-all-junit-tests/src/test/java/RunTheseBeforeYouCommitTests.java b/run-all-junit-tests/src/test/java/RunTheseBeforeYouCommitTests.java index 6a55f133d..4515959c9 100644 --- a/run-all-junit-tests/src/test/java/RunTheseBeforeYouCommitTests.java +++ b/run-all-junit-tests/src/test/java/RunTheseBeforeYouCommitTests.java @@ -30,11 +30,8 @@ import junit.framework.Test; import junit.framework.TestSuite; /** - * TODO: Remove this class along with the whole module. Run 'mvn test' at the project root instead. - * * @author Andy Clement */ -@Deprecated public class RunTheseBeforeYouCommitTests { public static Test suite() { |