[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/9875d2a2 Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/9875d2a2 Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/9875d2a2 Branch: refs/heads/SUREFIRE-1217 Commit: 9875d2a260a9fb81b6496bc8ed62722181b47142 Parents: f25c10d Author: Tibor17 <tibo...@lycos.com> Authored: Wed Jan 13 02:35:08 2016 +0100 Committer: Tibor17 <tibo...@lycos.com> Committed: Wed Jan 13 02:35:08 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 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/9875d2a2/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 43df609..dfa3948 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,7 +91,6 @@ 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 @@ -104,7 +103,6 @@ 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/9875d2a2/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 4f2abcf..3c78b60 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,7 +73,6 @@ 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() @@ -133,7 +132,6 @@ final class TestNGExecutor extractVerboseLevel( options ) ); testng.run(); } - // CHECKSTYLE:ON private static boolean isCliDebugOrShowErrors( List<CommandLineOption> mainCliOptions ) {
