maven-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mfriedenha...@apache.org
Subject [08/15] maven-surefire git commit: [SUREFIRE] fixed little checkstyle issues
Date Wed, 13 Jan 2016 06:06:08 GMT
[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 <tibor17@lycos.com>
Authored: Wed Jan 13 02:35:08 2016 +0100
Committer: Tibor17 <tibor17@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
)
     {


Mime
View raw message