From 30952ca876bb04e291bd30dcd8bc9a0ef2e79eba Mon Sep 17 00:00:00 2001 From: Andreas Dangel Date: Thu, 5 Jan 2017 20:26:00 +0100 Subject: [PATCH] [test] Javadoc doclint fixes References #1516 [build] doclint error during build --- .../sourceforge/pmd/AbstractLanguageVersionTest.java | 6 +++--- .../net/sourceforge/pmd/ant/AbstractAntTestHelper.java | 2 +- .../main/java/net/sourceforge/pmd/cli/BaseCLITest.java | 2 +- .../sourceforge/pmd/testframework/PMDTestRunner.java | 10 ++++------ 4 files changed, 9 insertions(+), 11 deletions(-) diff --git a/pmd-test/src/main/java/net/sourceforge/pmd/AbstractLanguageVersionTest.java b/pmd-test/src/main/java/net/sourceforge/pmd/AbstractLanguageVersionTest.java index dc8f19baea..155de383d2 100644 --- a/pmd-test/src/main/java/net/sourceforge/pmd/AbstractLanguageVersionTest.java +++ b/pmd-test/src/main/java/net/sourceforge/pmd/AbstractLanguageVersionTest.java @@ -26,7 +26,7 @@ import net.sourceforge.pmd.util.ResourceLoader; * *
  * @Parameters
- *     public static Collection data() {
+ *     public static Collection<Object[]> data() {
  *       return Arrays.asList(new Object[][] {
  *            { MyLanguageModule.NAME, MyLanguageModule.TERSE_NAME, "1.1",
  *              LanguageRegistry.getLanguage(MyLanguageModule.NAME).getVersion("1.1") },
@@ -39,8 +39,8 @@ import net.sourceforge.pmd.util.ResourceLoader;
  *       });
  * 
* - * For the parameters, see the constructor - * {@link #AbstractLanguageVersionTest(String, String, String, LanguageVersion)}. + *

For the parameters, see the constructor + * {@link #AbstractLanguageVersionTest(String, String, String, LanguageVersion)}.

*/ @RunWith(Parameterized.class) public class AbstractLanguageVersionTest { diff --git a/pmd-test/src/main/java/net/sourceforge/pmd/ant/AbstractAntTestHelper.java b/pmd-test/src/main/java/net/sourceforge/pmd/ant/AbstractAntTestHelper.java index e33e9e030a..0a4ca4caf2 100644 --- a/pmd-test/src/main/java/net/sourceforge/pmd/ant/AbstractAntTestHelper.java +++ b/pmd-test/src/main/java/net/sourceforge/pmd/ant/AbstractAntTestHelper.java @@ -19,7 +19,7 @@ import org.junit.Rule; * there is a workaround that must be shared by both tests (PMD and CPD's) I * felt compelled to move it to a single classes. * - * @author Romain Pelisse + * @author Romain Pelisse <belaran@gmail.com> * */ public abstract class AbstractAntTestHelper { diff --git a/pmd-test/src/main/java/net/sourceforge/pmd/cli/BaseCLITest.java b/pmd-test/src/main/java/net/sourceforge/pmd/cli/BaseCLITest.java index 18b8274ec7..ef7fd48918 100644 --- a/pmd-test/src/main/java/net/sourceforge/pmd/cli/BaseCLITest.java +++ b/pmd-test/src/main/java/net/sourceforge/pmd/cli/BaseCLITest.java @@ -19,7 +19,7 @@ import org.junit.BeforeClass; import net.sourceforge.pmd.PMD; /** - * @author Romain Pelisse + * @author Romain Pelisse <belaran@gmail.com> * */ public abstract class BaseCLITest { diff --git a/pmd-test/src/main/java/net/sourceforge/pmd/testframework/PMDTestRunner.java b/pmd-test/src/main/java/net/sourceforge/pmd/testframework/PMDTestRunner.java index 33b331e11c..62f7ad7f1b 100644 --- a/pmd-test/src/main/java/net/sourceforge/pmd/testframework/PMDTestRunner.java +++ b/pmd-test/src/main/java/net/sourceforge/pmd/testframework/PMDTestRunner.java @@ -25,13 +25,12 @@ import org.junit.runners.model.TestClass; import net.sourceforge.pmd.Rule; /** - * A test runner for rule tests. Unlike - * {@link SimpleAggregatorTst.CustomXmlTestClassMethodsRunner} it also reports + * A test runner for rule tests. It reports * the successful executed tests and allows to selectively execute single test * cases (it is {@link Filterable}). - *

- * In order to use it, you'll need to subclass {@link SimpleAggregatorTst} and - * annotate your test class with RunWith: + * + *

In order to use it, you'll need to subclass {@link SimpleAggregatorTst} and + * annotate your test class with RunWith:

* *
  * {@code @}RunWith(PMDTestRunner.class)
@@ -39,7 +38,6 @@ import net.sourceforge.pmd.Rule;
  * ...
  * }
  * 
- *

*/ public class PMDTestRunner extends Runner implements Filterable { private final Description desc;