You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by kr...@apache.org on 2011/12/05 20:15:18 UTC
svn commit: r1210583 - in /maven/surefire/trunk:
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/
surefire-api/src/main/java/org/apache/maven/surefire/booter/
surefire-providers/common-junit48/src/main/java/org/apache/maven/surefir...
Author: krosenvold
Date: Mon Dec 5 19:15:17 2011
New Revision: 1210583
URL: http://svn.apache.org/viewvc?rev=1210583&view=rev
Log:
o Extracted ProviderParameterNames to contain names of providers
Added:
maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/ProviderParameterNames.java
- copied, changed from r1210559, maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java
Modified:
maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/BaseProviderFactory.java
maven/surefire/trunk/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java
maven/surefire/trunk/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java
maven/surefire/trunk/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java
maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/AbstractDirectConfigurator.java
maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java
maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java
maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java
Modified: maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java (original)
+++ maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java Mon Dec 5 19:15:17 2011
@@ -54,6 +54,7 @@ import org.apache.maven.surefire.booter.
import org.apache.maven.surefire.booter.Classpath;
import org.apache.maven.surefire.booter.ClasspathConfiguration;
import org.apache.maven.surefire.booter.ProviderConfiguration;
+import org.apache.maven.surefire.booter.ProviderParameterNames;
import org.apache.maven.surefire.booter.StartupConfiguration;
import org.apache.maven.surefire.booter.StartupReportConfiguration;
import org.apache.maven.surefire.booter.SurefireBooterForkException;
@@ -253,13 +254,13 @@ public abstract class AbstractSurefireMo
if ( this.getParallel() != null )
{
- getProperties().setProperty( "parallel", this.getParallel() );
+ getProperties().setProperty( ProviderParameterNames.PARALLEL_PROP, this.getParallel() );
}
convertGroupParameters();
if ( this.getThreadCount() > 0 )
{
- getProperties().setProperty( "threadcount", Integer.toString( this.getThreadCount() ) );
+ getProperties().setProperty( ProviderParameterNames.THREADCOUNT_PROP, Integer.toString( this.getThreadCount() ) );
}
if ( this.getObjectFactory() != null )
{
@@ -281,11 +282,11 @@ public abstract class AbstractSurefireMo
}
if ( this.getExcludedGroups() != null )
{
- getProperties().setProperty( "excludedgroups", this.getExcludedGroups() );
+ getProperties().setProperty( ProviderParameterNames.TESTNG_EXCLUDEDGROUPS_PROP, this.getExcludedGroups() );
}
if ( this.getGroups() != null )
{
- getProperties().setProperty( "groups", this.getGroups() );
+ getProperties().setProperty( ProviderParameterNames.TESTNG_GROUPS_PROP, this.getGroups() );
}
}
@@ -312,11 +313,11 @@ public abstract class AbstractSurefireMo
if ( this.getParallel() != null )
{
- getProperties().setProperty( "parallel", this.getParallel() );
+ getProperties().setProperty( ProviderParameterNames.PARALLEL_PROP, this.getParallel() );
}
if ( this.getThreadCount() > 0 )
{
- getProperties().setProperty( "threadCount", Integer.toString( this.getThreadCount() ) );
+ getProperties().setProperty( ProviderParameterNames.THREADCOUNT_PROP, Integer.toString( this.getThreadCount() ) );
}
getProperties().setProperty( "perCoreThreadCount", Boolean.toString( getPerCoreThreadCount() ) );
getProperties().setProperty( "useUnlimitedThreads", Boolean.toString( getUseUnlimitedThreads() ) );
Modified: maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/BaseProviderFactory.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/BaseProviderFactory.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/BaseProviderFactory.java (original)
+++ maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/BaseProviderFactory.java Mon Dec 5 19:15:17 2011
@@ -83,7 +83,7 @@ public class BaseProviderFactory
private int getThreadCount()
{
- final String threadcount = (String) providerProperties.get( "threadcount" );
+ final String threadcount = (String) providerProperties.get( ProviderParameterNames.THREADCOUNT_PROP );
return threadcount == null ? 1 : Integer.parseInt( threadcount );
}
Copied: maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/ProviderParameterNames.java (from r1210559, maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java)
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/ProviderParameterNames.java?p2=maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/ProviderParameterNames.java&p1=maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java&r1=1210559&r2=1210583&rev=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java (original)
+++ maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/booter/ProviderParameterNames.java Mon Dec 5 19:15:17 2011
@@ -1,5 +1,4 @@
-package org.apache.maven.surefire.testng.conf;
-
+package org.apache.maven.surefire.booter;
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
@@ -20,23 +19,13 @@ package org.apache.maven.surefire.testng
*/
/**
- * TestNG 5.2 configurator.
- * <p/>
- * Allowed options:
- * -groups
- * -excludedgroups
- * -junit (boolean)
- * -threadcount (int)
- * -parallel (String)
- * <p/>
- *
- * @author <a href='mailto:the[dot]mindstorm[at]gmail[dot]com'>Alex Popescu</a>
+ * @author Kristian Rosenvold
*/
-public class TestNG52Configurator
- extends AbstractDirectConfigurator
+public class ProviderParameterNames
{
- public TestNG52Configurator()
- {
- setters.put( "parallel", new Setter( "setParallel", String.class ) );
- }
-}
\ No newline at end of file
+ public static final String TESTNG_EXCLUDEDGROUPS_PROP = "excludegroups";
+ public static final String TESTNG_GROUPS_PROP = "groups";
+ public static final String THREADCOUNT_PROP = "threadcount";
+ public static final String PARALLEL_PROP = "parallel";
+
+}
Modified: maven/surefire/trunk/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java (original)
+++ maven/surefire/trunk/surefire-providers/common-junit48/src/main/java/org/apache/maven/surefire/common/junit48/FilterFactory.java Mon Dec 5 19:15:17 2011
@@ -22,6 +22,7 @@ package org.apache.maven.surefire.common
import java.util.ArrayList;
import java.util.List;
import java.util.Properties;
+import org.apache.maven.surefire.booter.ProviderParameterNames;
import org.codehaus.plexus.util.SelectorUtils;
import org.junit.experimental.categories.Categories;
@@ -42,8 +43,8 @@ public class FilterFactory
public Filter createGroupFilter( Properties providerProperties )
{
- String groups = providerProperties.getProperty( "groups" );
- String excludedGroups = providerProperties.getProperty( "excludedgroups" );
+ String groups = providerProperties.getProperty( ProviderParameterNames.TESTNG_GROUPS_PROP );
+ String excludedGroups = providerProperties.getProperty( ProviderParameterNames.TESTNG_EXCLUDEDGROUPS_PROP );
List<Filter> included = commaSeparatedListToFilters( groups );
List<Filter> excluded = commaSeparatedListToFilters( excludedGroups );
return new CombinedCategoryFilter( included, excluded );
Modified: maven/surefire/trunk/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java (original)
+++ maven/surefire/trunk/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreParameters.java Mon Dec 5 19:15:17 2011
@@ -20,6 +20,7 @@ package org.apache.maven.surefire.junitc
*/
import java.util.Properties;
+import org.apache.maven.surefire.booter.ProviderParameterNames;
/**
* @author Kristian Rosenvold
@@ -34,11 +35,11 @@ class JUnitCoreParameters
private final Boolean useUnlimitedThreads;
- public static final String PARALLEL_KEY = "parallel";
+ public static final String PARALLEL_KEY = ProviderParameterNames.PARALLEL_PROP;
public static final String PERCORETHREADCOUNT_KEY = "perCoreThreadCount";
- public static final String THREADCOUNT_KEY = "threadCount";
+ public static final String THREADCOUNT_KEY = ProviderParameterNames.THREADCOUNT_PROP;
public static final String USEUNLIMITEDTHREADS_KEY = "useUnlimitedThreads";
Modified: maven/surefire/trunk/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java (original)
+++ maven/surefire/trunk/surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Surefire746Test.java Mon Dec 5 19:15:17 2011
@@ -25,6 +25,7 @@ import java.util.Properties;
import java.util.concurrent.ConcurrentHashMap;
import org.apache.maven.plugin.surefire.report.FileReporterFactory;
import org.apache.maven.surefire.booter.BaseProviderFactory;
+import org.apache.maven.surefire.booter.ProviderParameterNames;
import org.apache.maven.surefire.report.ConsoleLogger;
import org.apache.maven.surefire.report.DefaultConsoleReporter;
import org.apache.maven.surefire.report.ReporterConfiguration;
@@ -83,7 +84,7 @@ public class Surefire746Test {
providerParameters.setReporterConfiguration(new ReporterConfiguration(new File(""), false));
Properties junitProps = new Properties();
- junitProps.put("parallel", "none");
+ junitProps.put( ProviderParameterNames.PARALLEL_PROP, "none");
JUnitCoreParameters jUnitCoreParameters = new JUnitCoreParameters(junitProps);
Modified: maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/AbstractDirectConfigurator.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/AbstractDirectConfigurator.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/AbstractDirectConfigurator.java (original)
+++ maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/AbstractDirectConfigurator.java Mon Dec 5 19:15:17 2011
@@ -26,6 +26,7 @@ import java.util.Iterator;
import java.util.List;
import java.util.Map;
+import org.apache.maven.surefire.booter.ProviderParameterNames;
import org.apache.maven.surefire.testset.TestSetFailedException;
import org.apache.maven.surefire.util.NestedRuntimeException;
import org.testng.TestNG;
@@ -38,10 +39,10 @@ public abstract class AbstractDirectConf
AbstractDirectConfigurator()
{
Map options = new HashMap();
- options.put( "groups", new Setter( "setGroups", String.class ) );
- options.put( "excludedgroups", new Setter( "setExcludedGroups", String.class ) );
+ options.put( ProviderParameterNames.TESTNG_GROUPS_PROP, new Setter( "setGroups", String.class ) );
+ options.put( ProviderParameterNames.TESTNG_EXCLUDEDGROUPS_PROP, new Setter( "setExcludedGroups", String.class ) );
options.put( "junit", new Setter( "setJUnit", Boolean.class ) );
- options.put( "threadcount", new Setter( "setThreadCount", int.class ) );
+ options.put( ProviderParameterNames.THREADCOUNT_PROP, new Setter( "setThreadCount", int.class ) );
options.put( "usedefaultlisteners", new Setter( "setUseDefaultListeners", boolean.class ) );
this.setters = options;
}
Modified: maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java (original)
+++ maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG4751Configurator.java Mon Dec 5 19:15:17 2011
@@ -19,6 +19,8 @@ package org.apache.maven.surefire.testng
* under the License.
*/
+import org.apache.maven.surefire.booter.ProviderParameterNames;
+
/**
* TestNG 4.7 and 5.1 configurator.
* <p/>
@@ -37,6 +39,6 @@ public class TestNG4751Configurator
{
public TestNG4751Configurator()
{
- setters.put( "parallel", new Setter( "setParallel", boolean.class ) );
+ setters.put( ProviderParameterNames.PARALLEL_PROP, new Setter( "setParallel", boolean.class ) );
}
}
\ No newline at end of file
Modified: maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java (original)
+++ maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG52Configurator.java Mon Dec 5 19:15:17 2011
@@ -19,6 +19,8 @@ package org.apache.maven.surefire.testng
* under the License.
*/
+import org.apache.maven.surefire.booter.ProviderParameterNames;
+
/**
* TestNG 5.2 configurator.
* <p/>
@@ -37,6 +39,6 @@ public class TestNG52Configurator
{
public TestNG52Configurator()
{
- setters.put( "parallel", new Setter( "setParallel", String.class ) );
+ setters.put( ProviderParameterNames.PARALLEL_PROP, new Setter( "setParallel", String.class ) );
}
}
\ No newline at end of file
Modified: maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java?rev=1210583&r1=1210582&r2=1210583&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java (original)
+++ maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNGMapConfigurator.java Mon Dec 5 19:15:17 2011
@@ -19,14 +19,15 @@ package org.apache.maven.surefire.testng
* under the License.
*/
-import org.apache.maven.surefire.testset.TestSetFailedException;
-import org.testng.TestNG;
-
import java.lang.reflect.Method;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.Iterator;
import java.util.Map;
+import org.apache.maven.surefire.booter.ProviderParameterNames;
+import org.apache.maven.surefire.testset.TestSetFailedException;
+
+import org.testng.TestNG;
/**
* TestNG configurator for 5.3+ versions. TestNG exposes
@@ -77,7 +78,7 @@ public class TestNGMapConfigurator
{
val = convert( val, Boolean.class );
}
- else if ( "threadcount".equals( key ) )
+ else if ( ProviderParameterNames.THREADCOUNT_PROP.equals( key ) )
{
val = convert( val, String.class );
}