Documented deprecation of the JUnit4 testing approach

# Conflicts:
#	maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/AbstractMojoTestCase.java

Author: aamotharald

maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/AbstractMojoTestCase.java

@@ -79,8 +79,22 @@
 import org.codehaus.plexus.util.xml.Xpp3DomBuilder;
 
 /**
+ * TODO: add a way to use the plugin POM for the lookup so that the user doesn't have to provide the a:g:v:goal
+ * as the role hint for the mojo lookup.
+ * TODO: standardize the execution of the mojo and looking at the results, but could simply have a template method
+ * for verifying the state of the mojo post execution
+ * TODO: need a way to look at the state of the mojo without adding getters, this could be where we finally specify
+ * the expressions which extract values from the mojo.
+ * TODO: create a standard directory structure for picking up POMs to make this even easier, we really just need a testing
+ * descriptor and make this entirely declarative!
+ *
+ * @deprected As of version 3.4.0, it is advised to work with JUnit5 tests which do not
+ * use this class but {@link org.apache.maven.plugin.testing.junit5.MojoTest}
+ * instead.
+ *
  * @author jesse
  */
+@Deprecated
 public abstract class AbstractMojoTestCase extends PlexusTestCase {
     private static final DefaultArtifactVersion MAVEN_VERSION;
 

maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/MojoLogWrapper.java

@@ -25,7 +25,13 @@
 
 /**
  * @author jdcasey
+ *
+ * @deprected As of version 3.4.0, it is advised to work with JUnit5 tests which do not
+ * use this class but {@link  javax.inject.Inject} to inject a Log instance
+ * instead.
+ *
  */
+@Deprecated
 public class MojoLogWrapper implements Log {
     private final Logger logger;
 

maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/MojoParameters.java

@@ -23,8 +23,13 @@
 /**
  * Static helpers to create and manipulate mojo execution configuration parameters
  *
+ * @deprected As of version 3.4.0, it is advised to work with JUnit5 tests which do not
+ * use this class but {@link org.apache.maven.plugin.testing.junit5.MojoParameters}
+ * instead.
+ *
  * @since 3.2.0
  */
+@Deprecated
 public class MojoParameters {
     public static Xpp3Dom newParameter(String name, String value) {
         Xpp3Dom child = new Xpp3Dom(name);

maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/MojoRule.java

@@ -52,9 +52,14 @@
  * exhibited as {@code public} in the rule. You may annotate single tests methods with
  * {@link WithoutMojo} to prevent the rule from firing.
  *
+ * @deprected As of version 3.4.0, it is advised to work with JUnit5 tests which do not
+ * use rules but extensions {@link org.apache.maven.plugin.testing.junit5.MojoExtension}
+ * instead.
+ *
  * @author Mirko Friedenhagen
  * @since 2.2
  */
+@Deprecated
 public class MojoRule implements TestRule {
     private final AbstractMojoTestCase testCase;
 

maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/WithoutMojo.java

@@ -29,8 +29,13 @@
  *
  * An annotation for test methods that do not require the {@link MojoRule} to create and tear down the instance.
  *
+ * @deprected As of version 3.4.0, it is advised to work with JUnit5 tests which do not
+ * use rules but extensions {@link org.apache.maven.plugin.testing.junit5.MojoExtension}
+ * instead.
+ *
  * @author Mirko Friedenhagen
  */
+@Deprecated
 @Retention(RUNTIME)
 @Documented
 @Target(METHOD)

maven-plugin-testing-harness/src/main/java/org/apache/maven/plugin/testing/resources/TestResources.java

@@ -34,8 +34,13 @@
 /**
  * Junit4 test {@link Rule} to extract and assert test resources.
  *
+ * @deprected As of version 3.4.0, it is advised to work with JUnit5 tests which do not
+ * use rules but extensions {@link org.apache.maven.plugin.testing.junit5.MojoExtension}
+ * instead.
+ *
  * @since 3.1.0
  */
+@Deprecated
 public class TestResources extends TestWatcher {
 
     private final String projectsDir;