Repository: maven-surefire Updated Branches: refs/heads/master 641f97171 -> f25c10d70
[SUREFIRE] fixed little checkstyle issues Project: http://git-wip-us.apache.org/repos/asf/maven-surefire/repo Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/commit/f25c10d7 Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/f25c10d7 Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/f25c10d7 Branch: refs/heads/master Commit: f25c10d70cf319207abaaa33da5438d41d9b7b6c Parents: 641f971 Author: Tibor17 <tibo...@lycos.com> Authored: Wed Jan 13 02:04:46 2016 +0100 Committer: Tibor17 <tibo...@lycos.com> Committed: Wed Jan 13 02:04:46 2016 +0100 ---------------------------------------------------------------------- .../java/org/apache/maven/surefire/common/junit4/Notifier.java | 2 ++ .../main/java/org/apache/maven/surefire/testng/TestNGExecutor.java | 2 ++ 2 files changed, 4 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/f25c10d7/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/Notifier.java ---------------------------------------------------------------------- diff --git a/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/Notifier.java b/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/Notifier.java index dfa3948..43df609 100644 --- a/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/Notifier.java +++ b/surefire-providers/common-junit4/src/main/java/org/apache/maven/surefire/common/junit4/Notifier.java @@ -91,6 +91,7 @@ public class Notifier return failFast; } + // CHECKSTYLE:OFF @Override @SuppressWarnings( "checkstyle:redundantthrowscheck" ) // checkstyle is wrong here, see super.fireTestStarted() public final void fireTestStarted( Description description ) throws StoppedByUserException @@ -103,6 +104,7 @@ public class Notifier testClassNames.remove( cutTestClassAndMethod( description ).getClazz() ); } } + // CHECKSTYLE:ON @Override public final void fireTestFailure( Failure failure ) http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/f25c10d7/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGExecutor.java ---------------------------------------------------------------------- diff --git a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGExecutor.java b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGExecutor.java index 3c78b60..4f2abcf 100644 --- a/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGExecutor.java +++ b/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGExecutor.java @@ -73,6 +73,7 @@ final class TestNGExecutor throw new IllegalStateException( "not instantiable constructor" ); } + // CHECKSTYLE:OFF @SuppressWarnings( "checkstyle:parameternumbercheck" ) static void run( Iterable<Class<?>> testClasses, String testSourceDirectory, Map<String, String> options, // string,string because TestNGMapConfigurator#configure() @@ -132,6 +133,7 @@ final class TestNGExecutor extractVerboseLevel( options ) ); testng.run(); } + // CHECKSTYLE:ON private static boolean isCliDebugOrShowErrors( List<CommandLineOption> mainCliOptions ) {
