Repository: maven-surefire Updated Branches: refs/heads/SUREFIRE-1366 5745ba135 -> 3857f5590 (forced update)
http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-method-pattern/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-method-pattern/pom.xml b/surefire-integration-tests/src/test/resources/testng-method-pattern/pom.xml index 4cb27c6..760211d 100644 --- a/surefire-integration-tests/src/test/resources/testng-method-pattern/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-method-pattern/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-multiple-method-patterns/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-multiple-method-patterns/pom.xml b/surefire-integration-tests/src/test/resources/testng-multiple-method-patterns/pom.xml index f1a554b..84378e8 100644 --- a/surefire-integration-tests/src/test/resources/testng-multiple-method-patterns/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-multiple-method-patterns/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <parent> <groupId>org.apache.maven.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-parallel-suites/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-parallel-suites/pom.xml b/surefire-integration-tests/src/test/resources/testng-parallel-suites/pom.xml index d9d0a31..c90c6e0 100644 --- a/surefire-integration-tests/src/test/resources/testng-parallel-suites/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-parallel-suites/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <parent> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-parallel-with-annotations/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-parallel-with-annotations/pom.xml b/surefire-integration-tests/src/test/resources/testng-parallel-with-annotations/pom.xml index 58c275c..ca93853 100644 --- a/surefire-integration-tests/src/test/resources/testng-parallel-with-annotations/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-parallel-with-annotations/pom.xml @@ -10,7 +10,7 @@ --> <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-path with spaces/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-path with spaces/pom.xml b/surefire-integration-tests/src/test/resources/testng-path with spaces/pom.xml index 88eb6da..5f2e8e7 100644 --- a/surefire-integration-tests/src/test/resources/testng-path with spaces/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-path with spaces/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-simple/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-simple/pom.xml b/surefire-integration-tests/src/test/resources/testng-simple/pom.xml index bad1178..2fcdfde 100644 --- a/surefire-integration-tests/src/test/resources/testng-simple/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-simple/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-single-method-5-14-9/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-single-method-5-14-9/pom.xml b/surefire-integration-tests/src/test/resources/testng-single-method-5-14-9/pom.xml index 7539000..1f84839 100644 --- a/surefire-integration-tests/src/test/resources/testng-single-method-5-14-9/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-single-method-5-14-9/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-single-method/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-single-method/pom.xml b/surefire-integration-tests/src/test/resources/testng-single-method/pom.xml index 965fc55..e2fbaa7 100644 --- a/surefire-integration-tests/src/test/resources/testng-single-method/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-single-method/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-succes-percentage/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-succes-percentage/pom.xml b/surefire-integration-tests/src/test/resources/testng-succes-percentage/pom.xml index 1d2cfbd..4e74bd8 100644 --- a/surefire-integration-tests/src/test/resources/testng-succes-percentage/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-succes-percentage/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <parent> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-suite-xml/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-suite-xml/pom.xml b/surefire-integration-tests/src/test/resources/testng-suite-xml/pom.xml index f51bb49..96438a0 100644 --- a/surefire-integration-tests/src/test/resources/testng-suite-xml/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-suite-xml/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/testng-twoTestCaseSuite/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/testng-twoTestCaseSuite/pom.xml b/surefire-integration-tests/src/test/resources/testng-twoTestCaseSuite/pom.xml index 37d77bf..abdf7a1 100644 --- a/surefire-integration-tests/src/test/resources/testng-twoTestCaseSuite/pom.xml +++ b/surefire-integration-tests/src/test/resources/testng-twoTestCaseSuite/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/timeout-forked/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/timeout-forked/pom.xml b/surefire-integration-tests/src/test/resources/timeout-forked/pom.xml index d1bf628..ef9ea09 100644 --- a/surefire-integration-tests/src/test/resources/timeout-forked/pom.xml +++ b/surefire-integration-tests/src/test/resources/timeout-forked/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/unicode-testnames/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/unicode-testnames/pom.xml b/surefire-integration-tests/src/test/resources/unicode-testnames/pom.xml index 1056c79..2a385b5 100644 --- a/surefire-integration-tests/src/test/resources/unicode-testnames/pom.xml +++ b/surefire-integration-tests/src/test/resources/unicode-testnames/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/webapp/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/webapp/pom.xml b/surefire-integration-tests/src/test/resources/webapp/pom.xml index a9b0c39..6777637 100644 --- a/surefire-integration-tests/src/test/resources/webapp/pom.xml +++ b/surefire-integration-tests/src/test/resources/webapp/pom.xml @@ -1,5 +1,5 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.surefire.its</groupId> <artifactId>webapp</artifactId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/working-directory-is-invalid-property/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/working-directory-is-invalid-property/pom.xml b/surefire-integration-tests/src/test/resources/working-directory-is-invalid-property/pom.xml index b079009..40366ab 100644 --- a/surefire-integration-tests/src/test/resources/working-directory-is-invalid-property/pom.xml +++ b/surefire-integration-tests/src/test/resources/working-directory-is-invalid-property/pom.xml @@ -19,7 +19,7 @@ --> <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>localhost</groupId> <artifactId>working-directory-test</artifactId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/working-directory-missing/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/working-directory-missing/pom.xml b/surefire-integration-tests/src/test/resources/working-directory-missing/pom.xml index 04c7490..677f6d3 100644 --- a/surefire-integration-tests/src/test/resources/working-directory-missing/pom.xml +++ b/surefire-integration-tests/src/test/resources/working-directory-missing/pom.xml @@ -19,7 +19,7 @@ --> <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>localhost</groupId> <artifactId>working-directory-test</artifactId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/working-directory/child/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/working-directory/child/pom.xml b/surefire-integration-tests/src/test/resources/working-directory/child/pom.xml index bca882c..bdfbed8 100644 --- a/surefire-integration-tests/src/test/resources/working-directory/child/pom.xml +++ b/surefire-integration-tests/src/test/resources/working-directory/child/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <parent> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-integration-tests/src/test/resources/working-directory/pom.xml ---------------------------------------------------------------------- diff --git a/surefire-integration-tests/src/test/resources/working-directory/pom.xml b/surefire-integration-tests/src/test/resources/working-directory/pom.xml index 80e97b3..88d7f46 100644 --- a/surefire-integration-tests/src/test/resources/working-directory/pom.xml +++ b/surefire-integration-tests/src/test/resources/working-directory/pom.xml @@ -20,7 +20,7 @@ <project xmlns="http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven.plugins.surefire</groupId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/ConsoleLogger.java ---------------------------------------------------------------------- diff --git a/surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/ConsoleLogger.java b/surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/ConsoleLogger.java index 6baa7e3..34a8f83 100644 --- a/surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/ConsoleLogger.java +++ b/surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/ConsoleLogger.java @@ -21,16 +21,13 @@ package org.apache.maven.plugin.surefire.log.api; /** * Allows providers to write console messages on the running maven process. - * <p/> + * <br> * This output is associated with the entire test run and not a specific * test, which means it just goes "straight" to the console "immediately". - * <p/> - * s* <p/> - * <p/> + * <br> * This interface is used in org.apache.maven.plugin.surefire.CommonReflector and reflected * via IsolatedClassLoader which can see classes from JRE only. This interface MUST use * JRE types in method signatures, e.g. {@link String} or {@link Throwable}, etc. - * <p/> */ public interface ConsoleLogger { http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/JUnit4RunListener.java ---------------------------------------------------------------------- diff --git a/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/JUnit4RunListener.java b/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/JUnit4RunListener.java index 06f0fca..2a17bcb 100644 --- a/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/JUnit4RunListener.java +++ b/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/JUnit4RunListener.java @@ -46,8 +46,9 @@ public class JUnit4RunListener protected final RunListener reporter; /** - * This flag is set after a failure has occurred so that a <code>testSucceeded</code> event is not fired. - * This is necessary because JUnit4 always fires a <code>testRunFinished</code> event-- even if there was a failure. + * This flag is set after a failure has occurred so that a {@link RunListener#testSucceeded} event is not fired. + * This is necessary because JUnit4 always fires a {@link org.junit.runner.notification.RunListener#testRunFinished} + * event-- even if there was a failure. */ private final ThreadLocal<Boolean> failureFlag = new InheritableThreadLocal<Boolean>(); http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java ---------------------------------------------------------------------- diff --git a/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java b/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java index 36096e4..228ca6d 100644 --- a/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java +++ b/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java @@ -45,7 +45,7 @@ public class FilterFactory } /** - * @return <tt>true</tt> if non-blank + * @return {@code true} if non-blank * {@link org.apache.maven.surefire.booter.ProviderParameterNames#TESTNG_GROUPS_PROP} and/or * {@link org.apache.maven.surefire.booter.ProviderParameterNames#TESTNG_EXCLUDEDGROUPS_PROP} exists. */ http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/JUnit46StackTraceWriter.java ---------------------------------------------------------------------- diff --git a/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/JUnit46StackTraceWriter.java b/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/JUnit46StackTraceWriter.java index a07caaa..4051e19 100644 --- a/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/JUnit46StackTraceWriter.java +++ b/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/JUnit46StackTraceWriter.java @@ -24,7 +24,7 @@ import org.junit.runner.notification.Failure; /** * A stacktrace writer that requires at least junit 4.6 to run. Note that we only use this for 4.8 and higher - * <p/> + * <br> * Writes out a specific {@link org.junit.runner.notification.Failure} for * surefire as a stacktrace. * http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentRunListener.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentRunListener.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentRunListener.java index 8095dbf..56e3e1a 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentRunListener.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentRunListener.java @@ -32,7 +32,7 @@ import static org.apache.maven.surefire.junitcore.TestMethod.getThreadTestMethod /** * Handles responses from concurrent junit - * <p/> + * <br> * Stuff to remember about JUnit threading: * parallel=classes; beforeClass/afterClass, constructor and all tests method run on same thread * parallel=methods; beforeClass/afterClass run on main thread, constructor + each test method run on same thread http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCore.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCore.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCore.java index 9935d9a..ef1d63f 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCore.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCore.java @@ -32,7 +32,7 @@ import org.junit.runner.notification.RunListener; * * @author <a href="mailto:tibordig...@apache.org";>Tibor Digana (tibor17)</a> * @since 2.19 - * @see https://github.com/junit-team/junit/issues/1186 + * @see <a href="https://github.com/junit-team/junit/issues/1186";>JUnit issue 1186</a> */ class JUnitCore { http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java index 6a7a248..52aec92 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java @@ -117,10 +117,11 @@ public final class JUnitCoreParameters } /** - * @deprecated Instead use the expression ( {@link #isParallelMethods()} && {@link #isParallelClasses()} ). + * @deprecated Instead use the expression isParallelMethods() && isParallelClasses(). + * @return {@code true} if classes and methods are both parallel */ @Deprecated - @SuppressWarnings( { "unused", "deprecation" } ) + @SuppressWarnings( "unused" ) public boolean isParallelBoth() { return isParallelMethods() && isParallelClasses(); http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreRunListener.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreRunListener.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreRunListener.java index 98b87ef..e32e32c 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreRunListener.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreRunListener.java @@ -32,7 +32,7 @@ import org.junit.runner.notification.Failure; /** * Noteworthy things about JUnit4 listening: - * <p/> + * <br> * A class that is annotated with @Ignore will have one invocation of "testSkipped" with source==name * A method that is annotated with @Ignore will have a invocation of testSkipped with source and name distinct * Methods annotated with @Ignore trigger no further events. http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/TestMethod.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/TestMethod.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/TestMethod.java index ef70027..7bba28c 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/TestMethod.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/TestMethod.java @@ -29,7 +29,7 @@ import java.util.concurrent.atomic.AtomicReference; /** * Represents the test-state of a single test method that is run. - * <p/> + * <br> * Notes about thread safety: This instance is serially confined to 1-3 threads (construction, test-run, reporting), * without any actual parallel access */ http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ThreadSafe.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ThreadSafe.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ThreadSafe.java index 4f28775..2eaf90a 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ThreadSafe.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ThreadSafe.java @@ -27,7 +27,7 @@ import java.lang.annotation.Target; /** * After compiling sources under endorsed junit:4.7, this annotation - * is relocated in {@link org.junit.runner.notification.RunListener.ThreadSafe}. + * is relocated in org.junit.runner.notification.RunListener.ThreadSafe. */ @Documented @Target( ElementType.TYPE ) http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Balancer.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Balancer.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Balancer.java index 873ccfd..ec93dba 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Balancer.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Balancer.java @@ -22,7 +22,7 @@ package org.apache.maven.surefire.junitcore.pc; /** * The Balancer controls the maximum of concurrent threads in the current Scheduler(s) and prevents * from own thread resources exhaustion if other group of schedulers share the same pool of threads. - * <p/> + * <br> * If a permit is available, {@link #acquirePermit()} simply returns and a new test is scheduled * by {@link Scheduler#schedule(Runnable)} in the current runner. Otherwise waiting for a release. * One permit is released as soon as the child thread has finished. @@ -36,7 +36,7 @@ public interface Balancer /** * Acquires a permit from this balancer, blocking until one is available. * - * @return <code>true</code> if current thread is <em>NOT</em> interrupted + * @return {@code true} if current thread is <b>NOT</b> interrupted * while waiting for a permit. */ boolean acquirePermit(); http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/BalancerFactory.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/BalancerFactory.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/BalancerFactory.java index e4c36e3..82a9a46 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/BalancerFactory.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/BalancerFactory.java @@ -32,6 +32,7 @@ public class BalancerFactory /** * Infinite permits. + * @return Balancer wih infinite permits */ public static Balancer createInfinitePermitsBalancer() { @@ -43,6 +44,7 @@ public class BalancerFactory * Fairness guarantees the waiting schedulers to wake up in order they acquired a permit. * * @param concurrency number of permits to acquire when maintaining concurrency on tests + * @return Balancer with given number of permits */ public static Balancer createBalancer( int concurrency ) { @@ -54,6 +56,7 @@ public class BalancerFactory * Fairness guarantees the waiting schedulers to wake up in order they acquired a permit. * * @param concurrency number of permits to acquire when maintaining concurrency on tests + * @return Balancer with given number of permits */ public static Balancer createBalancerWithFairness( int concurrency ) { http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ExecutionStatus.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ExecutionStatus.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ExecutionStatus.java index 2afc012..07c21c9 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ExecutionStatus.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ExecutionStatus.java @@ -20,7 +20,7 @@ package org.apache.maven.surefire.junitcore.pc; */ /** - * Status of {@link ParallelComputer ParallelComputer runtime}.<p/> + * Status of {@link ParallelComputer ParallelComputer runtime}.<br> * Used together with shutdown hook. * * @author <a href="mailto:tibordig...@apache.org";>Tibor Digana (tibor17)</a> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerBuilder.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerBuilder.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerBuilder.java index 0b35765..92ee18c 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerBuilder.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerBuilder.java @@ -56,6 +56,7 @@ import static org.apache.maven.surefire.junitcore.pc.Type.CLASSES; import static org.apache.maven.surefire.junitcore.pc.Type.METHODS; import static org.apache.maven.surefire.junitcore.pc.Type.SUITES; +@SuppressWarnings( { "javadoc", "checkstyle:javadoctype" } ) /** * Executing suites, classes and methods with defined concurrency. In this example the threads which completed * the suites and classes can be reused in parallel methods. @@ -70,9 +71,9 @@ import static org.apache.maven.surefire.junitcore.pc.Type.SUITES; * Note that the type has always at least one thread even if unspecified. The capacity in * {@link ParallelComputerBuilder#useOnePool(int)} must be greater than the number of concurrent suites and classes * altogether. - * <p/> + * <br> * The Computer can be stopped in a separate thread. Pending tests will be interrupted if the argument is - * <tt>true</tt>. + * {@code true}. * <pre> * computer.describeStopped(true); * </pre> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerUtil.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerUtil.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerUtil.java index d9e3467..e70fc6f 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerUtil.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ParallelComputerUtil.java @@ -30,7 +30,8 @@ import java.util.Iterator; /** * An algorithm which configures {@link ParallelComputer} with allocated thread resources by given * {@link org.apache.maven.surefire.junitcore.JUnitCoreParameters}. - * The <code>AbstractSurefireMojo</code> has to provide correct combinations of thread-counts and <em>parallel</em>. + * The {@code AbstractSurefireMojo} has to provide correct combinations of thread-counts and + * configuration parameter {@code parallel}. * * @author Tibor Digana (tibor17) * @see org.apache.maven.surefire.junitcore.pc.ParallelComputerBuilder http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Scheduler.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Scheduler.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Scheduler.java index 3c21409..cf2b7ac 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Scheduler.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Scheduler.java @@ -36,10 +36,10 @@ import java.util.concurrent.ThreadPoolExecutor; /** * Schedules tests, controls thread resources, awaiting tests and other schedulers finished, and * a master scheduler can shutdown slaves. - * <p/> + * <br> * The scheduler objects should be first created (and wired) and set in runners * {@link org.junit.runners.ParentRunner#setScheduler(org.junit.runners.model.RunnerScheduler)}. - * <p/> + * <br> * A new instance of scheduling strategy should be passed to the constructor of this scheduler. * * @author Tibor Digana (tibor17) @@ -68,9 +68,13 @@ public class Scheduler /** * Use e.g. parallel classes have own non-shared thread pool, and methods another pool. - * <p/> + * <br> * You can use it with one infinite thread pool shared in strategies across all * suites, class runners, etc. + * + * @param logger console logger + * @param description JUnit description of class + * @param strategy scheduling strategy */ public Scheduler( ConsoleStream logger, Description description, SchedulingStrategy strategy ) { @@ -79,10 +83,10 @@ public class Scheduler /** * Should be used if schedulers in parallel children and parent use one instance of bounded thread pool. - * <p/> + * <br> * Set this scheduler in a e.g. one suite of classes, then every individual class runner should reference - * {@link Scheduler(ConsoleStream, org.junit.runner.Description, Scheduler, SchedulingStrategy)} - * or {@link Scheduler(ConsoleStream, org.junit.runner.Description, Scheduler, SchedulingStrategy, int)}. + * {@link #Scheduler(ConsoleStream, org.junit.runner.Description, Scheduler, SchedulingStrategy)} + * or {@link #Scheduler(ConsoleStream, org.junit.runner.Description, Scheduler, SchedulingStrategy, int)}. * * @param logger current logger implementation * @param description description of current runner @@ -98,9 +102,9 @@ public class Scheduler /** * New instances should be used by schedulers with limited concurrency by <tt>balancer</tt> * against other groups of schedulers. The schedulers share one pool. - * <p/> - * Unlike in {@link Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy, int)} which was - * limiting the <tt>concurrency</tt> of children of a runner where this scheduler was set, <em>this</em> + * <br> + * Unlike in {@link #Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy, int)} which was + * limiting the <tt>concurrency</tt> of children of a runner where this scheduler was set, {@code this} * <tt>balancer</tt> is limiting the concurrency of all children in runners having schedulers created by this * constructor. * @@ -140,11 +144,16 @@ public class Scheduler } /** + * @param logger console logger + * @param description JUnit description of class * @param masterScheduler a reference to - * {@link Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy, int)} - * or {@link Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy)} - * @see Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy) - * @see Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy, int) + * {@link #Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy, int)} + * or {@link #Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy)} + * @param strategy scheduling strategy + * @param concurrency determines maximum concurrent children scheduled a time via {@link #schedule(Runnable)} + * + * @see #Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy) + * @see #Scheduler(ConsoleStream, org.junit.runner.Description, SchedulingStrategy, int) */ public Scheduler( ConsoleStream logger, Description description, Scheduler masterScheduler, SchedulingStrategy strategy, int concurrency ) @@ -157,8 +166,13 @@ public class Scheduler /** * Should be used with individual pools on suites, classes and methods, see * {@link org.apache.maven.surefire.junitcore.pc.ParallelComputerBuilder#useSeparatePools()}. - * <p/> + * <br> * Cached thread pool is infinite and can be always shared. + * + * @param logger console logger + * @param description JUnit description of class + * @param masterScheduler parent scheduler + * @param strategy scheduling strategy */ public Scheduler( ConsoleStream logger, Description description, Scheduler masterScheduler, SchedulingStrategy strategy ) @@ -177,7 +191,7 @@ public class Scheduler /** * @param slave a slave scheduler to register - * @return <tt>true</tt> if successfully registered the <tt>slave</tt>. + * @return {@code true} if successfully registered the <tt>slave</tt>. */ private boolean register( Scheduler slave ) { @@ -196,7 +210,7 @@ public class Scheduler } /** - * @return <tt>true</tt> if new tasks can be scheduled. + * @return {@code true} if new tasks can be scheduled. */ private boolean canSchedule() { @@ -226,11 +240,11 @@ public class Scheduler /** * Attempts to stop all actively executing tasks and immediately returns a collection * of descriptions of those tasks which have started prior to this call. - * <p/> + * <br> * This scheduler and other registered schedulers will stop, see {@link #register(Scheduler)}. * If <tt>shutdownNow</tt> is set, waiting methods will be interrupted via {@link Thread#interrupt}. * - * @param stopNow if <tt>true</tt> interrupts waiting test methods + * @param stopNow if {@code true} interrupts waiting test methods * @return collection of descriptions started before shutting down */ protected ShutdownResult describeStopped( boolean stopNow ) @@ -252,7 +266,7 @@ public class Scheduler * is finished in * {@link org.junit.runners.Suite#run(org.junit.runner.notification.RunNotifier)} * all the thread-pools created by {@link ParallelComputerBuilder.PC} are already dead. - * See the unit test <em>ParallelComputerBuilder#timeoutAndForcedShutdown()</em>. + * See the unit test {@code ParallelComputerBuilder#timeoutAndForcedShutdown()}. * @param stopNow Interrupting tests by {@link java.util.concurrent.ExecutorService#shutdownNow()} or * {@link java.util.concurrent.Future#cancel(boolean) Future#cancel(true)} or * {@link Thread#interrupt()}. @@ -428,7 +442,7 @@ public class Scheduler } /** - * @return <tt>true</tt> if new children can be scheduled. + * @return {@code true} if new children can be scheduled. */ boolean canSchedule() { http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategies.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategies.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategies.java index 520c312..b3f80ba 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategies.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategies.java @@ -67,7 +67,7 @@ public class SchedulingStrategies /** * The <tt>threadPool</tt> passed to this strategy can be shared in other strategies. - * <p/> + * <br> * The call {@link SchedulingStrategy#finished()} is waiting until own tasks have finished. * New tasks will not be scheduled by this call in this strategy. This strategy is not * waiting for other strategies to finish. The {@link org.junit.runners.model.RunnerScheduler#finished()} may http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategy.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategy.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategy.java index ce337bc..d310628 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategy.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategy.java @@ -27,10 +27,10 @@ import java.util.concurrent.atomic.AtomicBoolean; /** * Specifies the strategy of scheduling whether sequential, or parallel. - * The strategy may use a thread pool <em>shared</em> with other strategies. - * <p/> + * The strategy may use a thread pool <b>shared</b> with other strategies. + * <br> * One instance of strategy can be used just by one {@link Scheduler}. - * <p/> + * <br> * The strategy is scheduling tasks in {@link #schedule(Runnable)} and awaiting them * completed in {@link #finished()}. Both methods should be used in one thread. * @@ -66,8 +66,8 @@ public abstract class SchedulingStrategy * Waiting for scheduled tasks to finish. * New tasks will not be scheduled by calling this method. * - * @return <tt>true</tt> if successfully stopped the scheduler, else - * <tt>false</tt> if already stopped (a <em>shared</em> thread + * @return {@code true} if successfully stopped the scheduler, else + * {@code false} if already stopped (a <b>shared</b> thread * pool was shutdown externally). * @throws InterruptedException if interrupted while waiting * for scheduled tasks to finish @@ -78,24 +78,24 @@ public abstract class SchedulingStrategy /** * Stops scheduling new tasks (e.g. by {@link java.util.concurrent.ExecutorService#shutdown()} - * on a private thread pool which cannot be <em>shared</em> with other strategy). + * on a private thread pool which cannot be <b>shared</b> with other strategy). * - * @return <tt>true</tt> if successfully stopped the scheduler, else - * <tt>false</tt> if already stopped (a <em>shared</em> thread + * @return {@code true} if successfully stopped the scheduler, else + * {@code false} if already stopped (a <b>shared</b> thread * pool was shutdown externally). * @see java.util.concurrent.ExecutorService#shutdown() */ protected abstract boolean stop(); /** - * Stops scheduling new tasks and <em>interrupts</em> running tasks + * Stops scheduling new tasks and {@code interrupts} running tasks * (e.g. by {@link java.util.concurrent.ExecutorService#shutdownNow()} on a private thread pool - * which cannot be <em>shared</em> with other strategy). - * <p/> + * which cannot be <b>shared</b> with other strategy). + * <br> * This method calls {@link #stop()} by default. * - * @return <tt>true</tt> if successfully stopped the scheduler, else - * <tt>false</tt> if already stopped (a <em>shared</em> thread + * @return {@code true} if successfully stopped the scheduler, else + * {@code false} if already stopped (a <b>shared</b> thread * pool was shutdown externally). * @see java.util.concurrent.ExecutorService#shutdownNow() */ @@ -105,7 +105,7 @@ public abstract class SchedulingStrategy } /** - * Persistently disables this strategy. Atomically ignores {@link Balancer} to acquire a new permit.<p/> + * Persistently disables this strategy. Atomically ignores {@link Balancer} to acquire a new permit.<br> * The method {@link #canSchedule()} atomically returns {@code false}. * @return {@code true} if {@link #canSchedule()} has return {@code true} on the beginning of this method call. */ @@ -119,13 +119,13 @@ public abstract class SchedulingStrategy } /** - * @return <tt>true</tt> if a thread pool associated with this strategy + * @return {@code true} if a thread pool associated with this strategy * can be shared with other strategies. */ protected abstract boolean hasSharedThreadPool(); /** - * @return <tt>true</tt> unless stopped, finished or disabled. + * @return {@code true} unless stopped, finished or disabled. */ protected boolean canSchedule() { http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SingleThreadScheduler.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SingleThreadScheduler.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SingleThreadScheduler.java index a0f5c2d..8e61fa3 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SingleThreadScheduler.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SingleThreadScheduler.java @@ -33,8 +33,8 @@ import java.util.concurrent.ThreadPoolExecutor; import java.util.concurrent.TimeUnit; /** - * Used to execute tests annotated with {@link net.jcip.annotations.NotThreadSafe}. - * <p/> + * Used to execute tests annotated with net.jcip.annotations.NotThreadSafe. + * <br> * * @author <a href="mailto:tibordig...@apache.org";>Tibor Digana (tibor17)</a> * @see ParallelComputerBuilder http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ThreadResourcesBalancer.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ThreadResourcesBalancer.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ThreadResourcesBalancer.java index 070c457..13f49bf 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ThreadResourcesBalancer.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/ThreadResourcesBalancer.java @@ -48,7 +48,7 @@ final class ThreadResourcesBalancer /** * @param numPermits number of permits to acquire when maintaining concurrency on tests. * Must be >0 and < {@link Integer#MAX_VALUE}. - * @param fair <tt>true</tt> guarantees the waiting schedulers to wake up in order they acquired a permit + * @param fair {@code true} guarantees the waiting schedulers to wake up in order they acquired a permit * @throws IllegalArgumentException if <tt>numPermits</tt> is not positive number */ ThreadResourcesBalancer( int numPermits, boolean fair ) @@ -65,7 +65,7 @@ final class ThreadResourcesBalancer /** * Acquires a permit from this balancer, blocking until one is available. * - * @return <code>true</code> if current thread is <em>NOT</em> interrupted + * @return {@code true} if current thread is <b>NOT</b> interrupted * while waiting for a permit. */ @Override http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/WrappedRunners.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/WrappedRunners.java b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/WrappedRunners.java index ba45e74..707cc35 100644 --- a/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/WrappedRunners.java +++ b/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/WrappedRunners.java @@ -23,7 +23,7 @@ import org.junit.runners.ParentRunner; /** * We need to wrap runners in a suite and count children of these runners. - * <p/> + * <br> * Old JUnit versions do not cache children after the first call of * {@link org.junit.runners.ParentRunner#getChildren()}. * Due to performance reasons, the children have to be observed just once. http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit47RunnerTest.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit47RunnerTest.java b/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit47RunnerTest.java index b883d09..3e139e1 100644 --- a/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit47RunnerTest.java +++ b/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit47RunnerTest.java @@ -27,15 +27,15 @@ import org.junit.runner.notification.Failure; * {@code * TestCase that expose "No tests were executed!" on Test failure using Maven Surefire 2.6-SNAPSHOT * and the JUnit 4.7 Runner. - * <p/> + * <br> * ------------------------------------------------------- * T E S T S * ------------------------------------------------------- - * <p/> + * <br> * Results: - * <p/> + * <br> * Tests run: 0, Failures: 0, Errors: 0, Skipped: 0 - * <p/> + * <br> * [INFO] ------------------------------------------------------------------------ * [INFO] BUILD FAILURE * [INFO] ------------------------------------------------------------------------ @@ -46,15 +46,15 @@ import org.junit.runner.notification.Failure; * [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.5:test * (default-test) on project xxxxxx: No tests were executed! (Set -DfailIfNoTests=false to * ignore this error.) -> [Help 1] - * <p/> - * <p/> + * <br> + * <br> * <dependency> * <groupId>junit</groupId> * <artifactId>junit</artifactId> * <version>4.8.1</version> * <scope>test</scope> * </dependency> - * <p/> + * <br> * <dependency> * <groupId>org.apache.maven.surefire</groupId> * <artifactId>surefire-booter</artifactId> @@ -90,7 +90,7 @@ public class MavenSurefireJUnit47RunnerTest * JUnits ParentRunner is responsible for adding the BeforeClass/AfterClass statements to the * statement execution chain. After BeforeClass is executed, a Statement that delegates to the * abstract method: runChild(T child, RunNotifier notifier) is called. As the JavaDoc explains: - * "Subclasses are responsible for making sure that relevant test events are reported through {@code notifier}". + * "Subclasses are responsible for making sure that relevant test events are reported through <b>notifier</b>". * When a @BeforeClass fail, the child that should handle the relevant test events(Started, Failed, Finished) * is never executed. * http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit48RunnerTest.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit48RunnerTest.java b/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit48RunnerTest.java index 08e1695..1faa782 100644 --- a/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit48RunnerTest.java +++ b/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/MavenSurefireJUnit48RunnerTest.java @@ -27,15 +27,15 @@ import org.junit.runner.notification.Failure; * TestCase that expose "No tests were executed!" on Test failure using Maven Surefire 2.6-SNAPSHOT * and the JUnit 4.8 Runner. * {@code - * * <p/> + * * <br> * ------------------------------------------------------- * T E S T S * ------------------------------------------------------- - * <p/> + * <br> * Results: - * <p/> + * <br> * Tests run: 0, Failures: 0, Errors: 0, Skipped: 0 - * <p/> + * <br> * [INFO] ------------------------------------------------------------------------ * [INFO] BUILD FAILURE * [INFO] ------------------------------------------------------------------------ @@ -46,15 +46,15 @@ import org.junit.runner.notification.Failure; * [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.5:test * (default-test) on project xxxxxx: No tests were executed! (Set -DfailIfNoTests=false to * ignore this error.) -> [Help 1] - * <p/> - * <p/> + * <br> + * <br> * <dependency> * <groupId>junit</groupId> * <artifactId>junit</artifactId> * <version>4.8.1</version> * <scope>test</scope> * </dependency> - * <p/> + * <br> * <dependency> * <groupId>org.apache.maven.surefire</groupId> * <artifactId>surefire-booter</artifactId> @@ -90,7 +90,7 @@ public class MavenSurefireJUnit48RunnerTest * JUnits ParentRunner is responsible for adding the BeforeClass/AfterClass statements to the * statement execution chain. After BeforeClass is executed, a Statement that delegates to the * abstract method: runChild(T child, RunNotifier notifier) is called. As the JavaDoc explains: - * "Subclasses are responsible for making sure that relevant test events are reported through {@code notifier}". + * "Subclasses are responsible for making sure that relevant test events are reported through <b>notifier</b>". * When a @BeforeClass fail, the child that should handle the relevant test events(Started, Failed, Finished) * is never executed. * http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java b/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java index 3e9ca93..eb5534e 100644 --- a/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java +++ b/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java @@ -54,7 +54,7 @@ import static junit.framework.Assert.assertEquals; * <version>4.8.1</version> * <scope>test</scope> * </dependency> - * <p/> + * <br> * <dependency> * <groupId>org.apache.maven.surefire</groupId> * <artifactId>surefire-booter</artifactId> http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategiesTest.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategiesTest.java b/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategiesTest.java index 812355f..0f1ac1a 100644 --- a/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategiesTest.java +++ b/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategiesTest.java @@ -33,13 +33,13 @@ import static org.junit.Assert.assertTrue; /** * Tests the factories in SchedulingStrategy. - * <p/> + * <br> * Th point of these tests is to check {@link Task#result} if changed - * from <code>false</code> to <code>true</code> after all scheduled tasks + * from {@code false} to {@code true} after all scheduled tasks * have finished. * The call {@link SchedulingStrategy#finished()} is waiting until the * strategy has finished. - * Then {@link Task#result} should be asserted that is <code>true</code>. + * Then {@link Task#result} should be asserted that is {@code true}. * * @author Tibor Digana (tibor17) * @see SchedulingStrategy http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/MethodSelector.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/MethodSelector.java b/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/MethodSelector.java index d55c079..2cd2f48 100644 --- a/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/MethodSelector.java +++ b/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/MethodSelector.java @@ -30,7 +30,6 @@ import org.testng.ITestNGMethod; * For internal use only * * @author Olivier Lamy - * @noinspection UnusedDeclaration * @since 2.7.3 */ public class MethodSelector http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/Stoppable.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/Stoppable.java b/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/Stoppable.java index dd172d2..e4fe304 100644 --- a/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/Stoppable.java +++ b/surefire-providers/surefire-testng-utils/src/main/java/org/apache/maven/surefire/testng/utils/Stoppable.java @@ -20,7 +20,7 @@ package org.apache.maven.surefire.testng.utils; */ /** - * Covers configuration parameter <em>skipAfterFailureCount</em>. + * Covers configuration parameter {@code skipAfterFailureCount}. * * @author <a href="mailto:tibordig...@apache.org";>Tibor Digana (tibor17)</a> * @since 2.19 http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/ConfigurationAwareTestNGReporter.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/ConfigurationAwareTestNGReporter.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/ConfigurationAwareTestNGReporter.java index 956a9d9..dd646b6 100644 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/ConfigurationAwareTestNGReporter.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/ConfigurationAwareTestNGReporter.java @@ -28,7 +28,6 @@ import org.testng.internal.IResultListener; * Just like TestNGReporter, but explicitly implements IResultListener; this interface is new in TestNG 5.5 * * @author Dan Fabulich - * @noinspection UnusedDeclaration */ public class ConfigurationAwareTestNGReporter extends TestNGReporter http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGProvider.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGProvider.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGProvider.java index a8a3837..14e103a 100644 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGProvider.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGProvider.java @@ -51,7 +51,6 @@ import static org.apache.maven.surefire.util.TestsToRun.fromClass; /** * @author Kristian Rosenvold - * @noinspection UnusedDeclaration */ public class TestNGProvider extends AbstractProvider http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGReporter.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGReporter.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGReporter.java index d7d5ff9..19cd22d 100644 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGReporter.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGReporter.java @@ -40,7 +40,6 @@ import static org.apache.maven.surefire.report.SimpleReportEntry.withException; * {@link org.apache.maven.surefire.report.RunListener}. * * @author jkuhnert - * @noinspection ThrowableResultOfMethodCallIgnored */ public class TestNGReporter implements ITestListener, ISuiteListener @@ -50,8 +49,8 @@ public class TestNGReporter /** * Constructs a new instance that will listen to * test updates from a {@link org.testng.TestNG} class instance. - * <p/> - * <p/>It is assumed that the requisite {@link org.testng.TestNG#addListener(ITestListener)} + * <br> + * <br>It is assumed that the requisite {@link org.testng.TestNG#addListener(ITestListener)} * method call has already associated with this instance <i>before</i> the test * suite is run. * http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java index 081b532..9955f44 100644 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java @@ -23,14 +23,14 @@ import org.apache.maven.surefire.booter.ProviderParameterNames; /** * TestNG 4.7 and 5.1 configurator. - * <p/> + * <br> * Allowed options: * -groups * -excludedgroups * -junit (boolean) * -threadcount (int) * -parallel (boolean) - * <p/> + * <br> * * @author <a href='mailto:the[dot]mindstorm[at]gmail[dot]com'>Alex Popescu</a> */ http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG513Configurator.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG513Configurator.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG513Configurator.java index b003789..ff164fb 100644 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG513Configurator.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG513Configurator.java @@ -22,7 +22,7 @@ package org.apache.maven.surefire.testng.conf; import org.apache.maven.surefire.testset.TestSetFailedException; /** - * TestNG 5.13 configurator. Changed: "reporterslist" need String instead of List<ReporterConfig>. + * TestNG 5.13 configurator. Changed: "reporterslist" need String instead of List<ReporterConfig>. */ public class TestNG513Configurator extends TestNG510Configurator http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG5143Configurator.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG5143Configurator.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG5143Configurator.java index 5575ccf..15a46a8 100644 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG5143Configurator.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG5143Configurator.java @@ -24,7 +24,7 @@ import org.apache.maven.surefire.testset.TestSetFailedException; /** * TestNG 5.14.3 configurator. Changed: "reporterslist" replaced by "reporter", - * and "listener" can use String instead of List<Class>. + * and "listener" can use String instead of List<Class>. */ public class TestNG5143Configurator extends TestNG5141Configurator http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java index 28c802f..44da341 100644 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java @@ -23,14 +23,14 @@ import org.apache.maven.surefire.booter.ProviderParameterNames; /** * TestNG 5.2 configurator. - * <p/> + * <br> * Allowed options: * -groups * -excludedgroups * -junit (boolean) * -threadcount (int) * -parallel (String) - * <p/> + * <br> * * @author <a href='mailto:the[dot]mindstorm[at]gmail[dot]com'>Alex Popescu</a> */ http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/3857f559/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java index b819298..c51aca6 100755 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java @@ -35,13 +35,13 @@ import static org.apache.maven.surefire.testng.conf.AbstractDirectConfigurator.l /** * TestNG configurator for 5.3+ versions. TestNG exposes a {@link org.testng.TestNG#configure(java.util.Map)} method. - * All suppported TestNG options are passed in String format, except - * <code>TestNGCommandLineArgs.LISTENER_COMMAND_OPT</code> which is <code>List>Class<</code>, - * <code>TestNGCommandLineArgs.JUNIT_DEF_OPT</code> which is a <code>Boolean</code>, - * <code>TestNGCommandLineArgs.SKIP_FAILED_INVOCATION_COUNT_OPT</code> which is a <code>Boolean</code>, - * <code>TestNGCommandLineArgs.OBJECT_FACTORY_COMMAND_OPT</code> which is a <code>Class</code>, - * <code>TestNGCommandLineArgs.REPORTERS_LIST</code> which is a <code>List>ReporterConfig<</code>. - * <p/> + * All supported TestNG options are passed in String format, except + * {@link org.testng.TestNGCommandLineArgs#LISTENER_COMMAND_OPT} which is {@link java.util.List List>Class<}, + * {@link org.testng.TestNGCommandLineArgs#JUNIT_DEF_OPT} which is a {@link Boolean}, + * {@link org.testng.TestNGCommandLineArgs#SKIP_FAILED_INVOCATION_COUNT_OPT} which is a {@link Boolean}, + * {@link org.testng.TestNGCommandLineArgs#OBJECT_FACTORY_COMMAND_OPT} which is a {@link Class}, + * {@link org.testng.TestNGCommandLineArgs#REPORTERS_LIST} which is a {@link java.util.List List>ReporterConfig<}. + * <br> * Test classes and/or suite files are not passed along as options parameters, but configured separately. * * @author <a href='mailto:the[dot]mindstorm[at]gmail[dot]com'>Alex Popescu</a>
