You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ti...@apache.org on 2016/01/15 06:10:28 UTC
[16/30] maven-surefire git commit: [SUREFIRE] fixed little checkstyle
issues
[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/3.0-rc1
Commit: 9875d2a260a9fb81b6496bc8ed62722181b47142
Parents: f25c10d
Author: Tibor17 <ti...@lycos.com>
Authored: Wed Jan 13 02:35:08 2016 +0100
Committer: Tibor17 <ti...@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 )
{