maven-surefire-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From steph...@apache.org
Subject svn commit: r945127 [2/2] - in /maven/surefire/trunk: ./ maven-failsafe-plugin/ maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/ maven-surefire-common/ maven-surefire-common/src/ maven-surefire-common/src/main/ maven-surefire-commo...
Date Mon, 17 May 2010 13:02:39 GMT
Added: 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=945127&view=auto
==============================================================================
--- maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java (added)
+++ maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java Mon May 17 13:02:39 2010
@@ -0,0 +1,1023 @@
+package org.apache.maven.plugin.surefire;
+
+import org.apache.maven.artifact.Artifact;
+import org.apache.maven.artifact.DependencyResolutionRequiredException;
+import org.apache.maven.artifact.factory.ArtifactFactory;
+import org.apache.maven.artifact.metadata.ArtifactMetadataSource;
+import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.resolver.ArtifactNotFoundException;
+import org.apache.maven.artifact.resolver.ArtifactResolutionException;
+import org.apache.maven.artifact.resolver.ArtifactResolutionResult;
+import org.apache.maven.artifact.resolver.ArtifactResolver;
+import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
+import org.apache.maven.artifact.resolver.filter.ExcludesArtifactFilter;
+import org.apache.maven.artifact.versioning.DefaultArtifactVersion;
+import org.apache.maven.artifact.versioning.InvalidVersionSpecificationException;
+import org.apache.maven.artifact.versioning.OverConstrainedVersionException;
+import org.apache.maven.artifact.versioning.VersionRange;
+import org.apache.maven.execution.MavenSession;
+import org.apache.maven.plugin.AbstractMojo;
+import org.apache.maven.plugin.MojoExecutionException;
+import org.apache.maven.plugin.MojoFailureException;
+import org.apache.maven.project.MavenProject;
+import org.apache.maven.surefire.booter.ForkConfiguration;
+import org.apache.maven.surefire.booter.SurefireBooter;
+import org.apache.maven.surefire.report.BriefConsoleReporter;
+import org.apache.maven.surefire.report.BriefFileReporter;
+import org.apache.maven.surefire.report.ConsoleReporter;
+import org.apache.maven.surefire.report.DetailedConsoleReporter;
+import org.apache.maven.surefire.report.FileReporter;
+import org.apache.maven.surefire.report.ForkingConsoleReporter;
+import org.apache.maven.surefire.report.XMLReporter;
+import org.apache.maven.toolchain.Toolchain;
+import org.apache.maven.toolchain.ToolchainManager;
+import org.codehaus.plexus.util.StringUtils;
+
+import java.io.File;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Properties;
+
+/**
+ * Abstract base class for running tests using Surefire.
+ *
+ * @author Stephen Connolly
+ * @version $Id: SurefirePlugin.java 945065 2010-05-17 10:26:22Z stephenc $
+ */
+public abstract class AbstractSurefireMojo
+    extends AbstractMojo
+{
+
+    private static final String BRIEF_REPORT_FORMAT = "brief";
+
+    private static final String PLAIN_REPORT_FORMAT = "plain";
+
+    // common field getters/setters
+
+    public abstract boolean isSkipTests();
+
+    public abstract void setSkipTests( boolean skipTests );
+
+    public abstract boolean isSkipITs();
+
+    public abstract void setSkipITs( boolean skipITs );
+
+    public abstract boolean isSkipExec();
+
+    public abstract void setSkipExec( boolean skipExec );
+
+    public abstract boolean isSkip();
+
+    public abstract void setSkip( boolean skip );
+
+    public abstract boolean isTestFailureIgnore();
+
+    public abstract void setTestFailureIgnore( boolean testFailureIgnore );
+
+    public abstract File getBasedir();
+
+    public abstract void setBasedir( File basedir );
+
+    public abstract File getTestClassesDirectory();
+
+    public abstract void setTestClassesDirectory( File testClassesDirectory );
+
+    public abstract File getClassesDirectory();
+
+    public abstract void setClassesDirectory( File classesDirectory );
+
+    public abstract MavenProject getProject();
+
+    public abstract void setProject( MavenProject project );
+
+    public abstract String getIgnoreClasspathElements();
+
+    public abstract void setIgnoreClasspathElements( String ignoreClasspathElements );
+
+    public abstract List getAdditionalClasspathElements();
+
+    public abstract void setAdditionalClasspathElements( List additionalClasspathElements );
+
+    public abstract File getReportsDirectory();
+
+    public abstract void setReportsDirectory( File reportsDirectory );
+
+    public abstract File getTestSourceDirectory();
+
+    public abstract void setTestSourceDirectory( File testSourceDirectory );
+
+    public abstract String getTest();
+
+    public abstract void setTest( String test );
+
+    public abstract List getIncludes();
+
+    public abstract void setIncludes( List includes );
+
+    public abstract List getExcludes();
+
+    public abstract void setExcludes( List excludes );
+
+    public abstract ArtifactRepository getLocalRepository();
+
+    public abstract void setLocalRepository( ArtifactRepository localRepository );
+
+    public abstract Properties getSystemProperties();
+
+    public abstract void setSystemProperties( Properties systemProperties );
+
+    public abstract Map getSystemPropertyVariables();
+
+    public abstract void setSystemPropertyVariables( Map systemPropertyVariables );
+
+    public abstract Properties getProperties();
+
+    public abstract void setProperties( Properties properties );
+
+    public abstract Map getPluginArtifactMap();
+
+    public abstract void setPluginArtifactMap( Map pluginArtifactMap );
+
+    public abstract Map getProjectArtifactMap();
+
+    public abstract void setProjectArtifactMap( Map projectArtifactMap );
+
+    public abstract File getSummaryFile();
+
+    public abstract void setSummaryFile( File summaryFile );
+
+    public abstract boolean isPrintSummary();
+
+    public abstract void setPrintSummary( boolean printSummary );
+
+    public abstract String getReportFormat();
+
+    public abstract void setReportFormat( String reportFormat );
+
+    public abstract boolean isUseFile();
+
+    public abstract void setUseFile( boolean useFile );
+
+    public abstract boolean isRedirectTestOutputToFile();
+
+    public abstract void setRedirectTestOutputToFile( boolean redirectTestOutputToFile );
+
+    public abstract Boolean getFailIfNoTests();
+
+    public abstract void setFailIfNoTests( Boolean failIfNoTests );
+
+    public abstract String getForkMode();
+
+    public abstract void setForkMode( String forkMode );
+
+    public abstract String getJvm();
+
+    public abstract void setJvm( String jvm );
+
+    public abstract String getArgLine();
+
+    public abstract void setArgLine( String argLine );
+
+    public abstract String getDebugForkedProcess();
+
+    public abstract void setDebugForkedProcess( String debugForkedProcess );
+
+    public abstract int getForkedProcessTimeoutInSeconds();
+
+    public abstract void setForkedProcessTimeoutInSeconds( int forkedProcessTimeoutInSeconds );
+
+    public abstract Map getEnvironmentVariables();
+
+    public abstract void setEnvironmentVariables( Map environmentVariables );
+
+    public abstract File getWorkingDirectory();
+
+    public abstract void setWorkingDirectory( File workingDirectory );
+
+    public abstract boolean isChildDelegation();
+
+    public abstract void setChildDelegation( boolean childDelegation );
+
+    public abstract String getGroups();
+
+    public abstract void setGroups( String groups );
+
+    public abstract String getExcludedGroups();
+
+    public abstract void setExcludedGroups( String excludedGroups );
+
+    public abstract File[] getSuiteXmlFiles();
+
+    public abstract void setSuiteXmlFiles( File[] suiteXmlFiles );
+
+    public abstract String getJunitArtifactName();
+
+    public abstract void setJunitArtifactName( String junitArtifactName );
+
+    public abstract String getTestNGArtifactName();
+
+    public abstract void setTestNGArtifactName( String testNGArtifactName );
+
+    public abstract int getThreadCount();
+
+    public abstract void setThreadCount( int threadCount );
+
+    public abstract String getPerCoreThreadCount();
+
+    public abstract void setPerCoreThreadCount( String perCoreThreadCount );
+
+    public abstract String getUseUnlimitedThreads();
+
+    public abstract void setUseUnlimitedThreads( String useUnlimitedThreads );
+
+    public abstract String getParallel();
+
+    public abstract void setParallel( String parallel );
+
+    public abstract boolean isTrimStackTrace();
+
+    public abstract void setTrimStackTrace( boolean trimStackTrace );
+
+    public abstract ArtifactResolver getArtifactResolver();
+
+    public abstract void setArtifactResolver( ArtifactResolver artifactResolver );
+
+    public abstract ArtifactFactory getArtifactFactory();
+
+    public abstract void setArtifactFactory( ArtifactFactory artifactFactory );
+
+    public abstract List getRemoteRepositories();
+
+    public abstract void setRemoteRepositories( List remoteRepositories );
+
+    public abstract ArtifactMetadataSource getMetadataSource();
+
+    public abstract void setMetadataSource( ArtifactMetadataSource metadataSource );
+
+    public abstract Properties getOriginalSystemProperties();
+
+    public abstract void setOriginalSystemProperties( Properties originalSystemProperties );
+
+    public abstract Properties getInternalSystemProperties();
+
+    public abstract void setInternalSystemProperties( Properties internalSystemProperties );
+
+    public abstract boolean isDisableXmlReport();
+
+    public abstract void setDisableXmlReport( boolean disableXmlReport );
+
+    public abstract Boolean getUseSystemClassLoader();
+
+    public abstract void setUseSystemClassLoader( Boolean useSystemClassLoader );
+
+    public abstract boolean isUseManifestOnlyJar();
+
+    public abstract void setUseManifestOnlyJar( boolean useManifestOnlyJar );
+
+    public abstract boolean isEnableAssertions();
+
+    public abstract void setEnableAssertions( boolean enableAssertions );
+
+    public abstract MavenSession getSession();
+
+    public abstract void setSession( MavenSession session );
+
+    public abstract String getObjectFactory();
+
+    public abstract void setObjectFactory( String objectFactory );
+
+    public abstract String getEncoding();
+
+    public abstract void setEncoding( String encoding );
+
+    public abstract ToolchainManager getToolchainManager();
+
+    public abstract void setToolchainManager( ToolchainManager toolchainManager );
+
+    // common code
+
+    protected abstract boolean isTestsSkipped();
+
+    protected abstract String getPluginName();
+
+    protected boolean verifyParameters()
+        throws MojoFailureException
+    {
+        if ( isTestsSkipped() )
+        {
+            getLog().info( "Tests are skipped." );
+            return false;
+        }
+
+        if ( !getTestClassesDirectory().exists() )
+        {
+            if ( getFailIfNoTests() != null && getFailIfNoTests().booleanValue() )
+            {
+                throw new MojoFailureException( "No tests to run!" );
+            }
+            getLog().info( "No tests to run." );
+            return false;
+        }
+
+        if ( !getWorkingDirectory().exists() )
+        {
+            if ( !getWorkingDirectory().mkdirs() )
+            {
+                throw new MojoFailureException( "Cannot create workingDirectory " + getWorkingDirectory() );
+            }
+        }
+
+        if ( !getWorkingDirectory().isDirectory() )
+        {
+            throw new MojoFailureException( "workingDirectory " + getWorkingDirectory() + " exists and is not a directory" );
+        }
+
+        if ( getUseSystemClassLoader() != null && ForkConfiguration.FORK_NEVER.equals( getForkMode() ) )
+        {
+            getLog().warn( "useSystemClassloader setting has no effect when not forking" );
+        }
+
+        return true;
+    }
+
+    protected Toolchain getToolchain()
+    {
+        Toolchain tc = null;
+
+        if ( getToolchainManager() != null )
+        {
+            tc = getToolchainManager().getToolchainFromBuildContext( "jdk", getSession() );
+        }
+
+        return tc;
+    }
+
+    /**
+     * Converts old TestNG configuration parameters over to new properties based configuration
+     * method. (if any are defined the old way)
+     */
+    private void convertTestNGParameters()
+    {
+        if ( getProperties() == null )
+        {
+            setProperties( new Properties() );
+        }
+
+        if ( this.getParallel() != null )
+        {
+            getProperties().setProperty( "parallel", this.getParallel() );
+        }
+        if ( this.getExcludedGroups() != null )
+        {
+            getProperties().setProperty( "excludegroups", this.getExcludedGroups() );
+        }
+        if ( this.getGroups() != null )
+        {
+            getProperties().setProperty( "groups", this.getGroups() );
+        }
+
+        if ( this.getThreadCount() > 0 )
+        {
+            getProperties().setProperty( "threadcount", new Integer( this.getThreadCount() ).toString() );
+        }
+        if ( this.getObjectFactory() != null )
+        {
+            getProperties().setProperty( "objectfactory", this.getObjectFactory() );
+        }
+    }
+
+    private boolean isAnyConcurrencySelected()
+    {
+        return this.getParallel() != null && this.getParallel().trim().length() > 0;
+    }
+
+    /**
+     * Converts old TestNG configuration parameters over to new properties based configuration
+     * method. (if any are defined the old way)
+     */
+    private void convertJunitCoreParameters()
+    {
+        if ( getProperties() == null )
+        {
+            setProperties( new Properties() );
+        }
+
+        if ( this.getParallel() != null )
+        {
+            getProperties().setProperty( "parallel", this.getParallel() );
+        }
+        if ( this.getThreadCount() > 0 )
+        {
+            getProperties().setProperty( "threadCount", new Integer( this.getThreadCount() ).toString() );
+        }
+        if ( this.getPerCoreThreadCount() != null )
+        {
+            getProperties().setProperty( "perCoreThreadCount", getPerCoreThreadCount() );
+        }
+        if ( this.getUseUnlimitedThreads() != null )
+        {
+            getProperties().setProperty( "useUnlimitedThreads", getUseUnlimitedThreads() );
+        }
+        Artifact configurableParallelComputer =
+            (Artifact) getProjectArtifactMap().get( "org.jdogma.junit:configurable-parallel-computer" );
+        getProperties().setProperty( "configurableParallelComputerPresent",
+                                Boolean.toString( configurableParallelComputer != null ) );
+
+    }
+
+    private boolean isJunit47Compatible( Artifact artifact )
+        throws MojoExecutionException
+    {
+        return isWithinVersionSpec( artifact, "[4.7,)" );
+    }
+
+    private boolean isAnyJunit4( Artifact artifact )
+        throws MojoExecutionException
+    {
+        return isWithinVersionSpec( artifact, "[4.0,)" );
+    }
+
+    private boolean isWithinVersionSpec( Artifact artifact, String versionSpec )
+        throws MojoExecutionException
+    {
+        if ( artifact == null )
+        {
+            return false;
+        }
+        try
+        {
+            VersionRange range = VersionRange.createFromVersionSpec( versionSpec );
+            try
+            {
+                return range.containsVersion( artifact.getSelectedVersion() );
+            }
+            catch ( NullPointerException e )
+            {
+                return range.containsVersion( new DefaultArtifactVersion( artifact.getBaseVersion() ) );
+            }
+        }
+        catch ( InvalidVersionSpecificationException e )
+        {
+            throw new MojoExecutionException( "Bug in junit 4.7 plugin. Please report with stacktrace" );
+        }
+        catch ( OverConstrainedVersionException e )
+        {
+            throw new MojoExecutionException( "Bug in junit 4.7 plugin. Please report with stacktrace" );
+        }
+    }
+
+    protected SurefireBooter constructSurefireBooter()
+        throws MojoExecutionException, MojoFailureException
+    {
+        SurefireBooter surefireBooter = new SurefireBooter();
+
+        Artifact surefireArtifact = (Artifact) getPluginArtifactMap().get( "org.apache.maven.surefire:surefire-booter" );
+        if ( surefireArtifact == null )
+        {
+            throw new MojoExecutionException( "Unable to locate surefire-booter in the list of plugin artifacts" );
+        }
+
+        surefireArtifact.isSnapshot(); // MNG-2961: before Maven 2.0.8, fixes getBaseVersion to be -SNAPSHOT if needed
+
+        Artifact junitArtifact;
+        Artifact testNgArtifact;
+        try
+        {
+            addArtifact( surefireBooter, surefireArtifact );
+
+            junitArtifact = (Artifact) getProjectArtifactMap().get( getJunitArtifactName() );
+            // SUREFIRE-378, junit can have an alternate artifact name
+            if ( junitArtifact == null && "junit:junit".equals( getJunitArtifactName() ) )
+            {
+                junitArtifact = (Artifact) getProjectArtifactMap().get( "junit:junit-dep" );
+            }
+
+            // TODO: this is pretty manual, but I'd rather not require the plugin > dependencies section right now
+            testNgArtifact = (Artifact) getProjectArtifactMap().get( getTestNGArtifactName() );
+
+            if ( testNgArtifact != null )
+            {
+                VersionRange range = VersionRange.createFromVersionSpec( "[4.7,)" );
+                if ( !range.containsVersion( new DefaultArtifactVersion( testNgArtifact.getVersion() ) ) )
+                {
+                    throw new MojoFailureException(
+                        "TestNG support requires version 4.7 or above. You have declared version "
+                            + testNgArtifact.getVersion() );
+                }
+
+                convertTestNGParameters();
+
+                if ( this.getTestClassesDirectory() != null )
+                {
+                    getProperties().setProperty( "testng.test.classpath", getTestClassesDirectory().getAbsolutePath() );
+                }
+
+                addArtifact( surefireBooter, testNgArtifact );
+
+                // The plugin uses a JDK based profile to select the right testng. We might be explicity using a
+                // different one since its based on the source level, not the JVM. Prune using the filter.
+                addProvider( surefireBooter, "surefire-testng", surefireArtifact.getBaseVersion(), testNgArtifact );
+            }
+            else if ( junitArtifact != null && isAnyJunit4( junitArtifact ) )
+            {
+                if ( isAnyConcurrencySelected() && isJunit47Compatible( junitArtifact ) )
+                {
+                    convertJunitCoreParameters();
+                    addProvider( surefireBooter, "surefire-junit47", surefireArtifact.getBaseVersion(), null );
+                }
+                else
+                {
+                    addProvider( surefireBooter, "surefire-junit4", surefireArtifact.getBaseVersion(), null );
+                }
+            }
+            else
+            {
+                // add the JUnit provider as default - it doesn't require JUnit to be present,
+                // since it supports POJO tests.
+                addProvider( surefireBooter, "surefire-junit", surefireArtifact.getBaseVersion(), null );
+            }
+        }
+        catch ( ArtifactNotFoundException e )
+        {
+            throw new MojoExecutionException(
+                "Unable to locate required surefire provider dependency: " + e.getMessage(), e );
+        }
+        catch ( InvalidVersionSpecificationException e )
+        {
+            throw new MojoExecutionException( "Error determining the TestNG version requested: " + e.getMessage(), e );
+        }
+        catch ( ArtifactResolutionException e )
+        {
+            throw new MojoExecutionException( "Error to resolving surefire provider dependency: " + e.getMessage(), e );
+        }
+
+        if ( getSuiteXmlFiles() != null && getSuiteXmlFiles().length > 0 && getTest() == null )
+        {
+            if ( testNgArtifact == null )
+            {
+                throw new MojoExecutionException( "suiteXmlFiles is configured, but there is no TestNG dependency" );
+            }
+
+            // TODO: properties should be passed in here too
+            surefireBooter.addTestSuite( "org.apache.maven.surefire.testng.TestNGXmlTestSuite",
+                                         new Object[]{ getSuiteXmlFiles(), getTestSourceDirectory().getAbsolutePath(),
+                                             testNgArtifact.getVersion(), testNgArtifact.getClassifier(),
+                                             getProperties(), getReportsDirectory() } );
+        }
+        else
+        {
+            List includes;
+            List excludes;
+
+            if ( getTest() != null )
+            {
+                // Check to see if we are running a single test. The raw parameter will
+                // come through if it has not been set.
+
+                // FooTest -> **/FooTest.java
+
+                includes = new ArrayList();
+
+                excludes = new ArrayList();
+
+                if ( getFailIfNoTests() == null )
+                {
+                    setFailIfNoTests( Boolean.TRUE );
+                }
+
+                String[] testRegexes = StringUtils.split( getTest(), "," );
+
+                for ( int i = 0; i < testRegexes.length; i++ )
+                {
+                    String testRegex = testRegexes[i];
+                    if ( testRegex.endsWith( ".java" ) )
+                    {
+                        testRegex = testRegex.substring( 0, testRegex.length() - 5 );
+                    }
+                    // Allow paths delimited by '.' or '/'
+                    testRegex = testRegex.replace( '.', '/' );
+                    includes.add( "**/" + testRegex + ".java" );
+                }
+            }
+            else
+            {
+                includes = this.getIncludes();
+
+                excludes = this.getExcludes();
+
+                // defaults here, qdox doesn't like the end javadoc value
+                // Have to wrap in an ArrayList as surefire expects an ArrayList instead of a List for some reason
+                if ( includes == null || includes.size() == 0 )
+                {
+                    includes = new ArrayList( Arrays.asList( getDefaultIncludes() ) );
+                }
+                if ( excludes == null || excludes.size() == 0 )
+                {
+                    excludes = new ArrayList( Arrays.asList( new String[]{"**/*$*"} ) );
+                }
+            }
+
+            if ( testNgArtifact != null )
+            {
+                surefireBooter.addTestSuite( "org.apache.maven.surefire.testng.TestNGDirectoryTestSuite",
+                                             new Object[]{ getTestClassesDirectory(), includes, excludes,
+                                                 getTestSourceDirectory().getAbsolutePath(), testNgArtifact.getVersion(),
+                                                 testNgArtifact.getClassifier(), getProperties(),
+                                                 getReportsDirectory() } );
+            }
+            else
+            {
+                String junitDirectoryTestSuite;
+                if ( isAnyConcurrencySelected() && isJunit47Compatible( junitArtifact ) )
+                {
+                    junitDirectoryTestSuite = "org.apache.maven.surefire.junitcore.JUnitCoreDirectoryTestSuite";
+                    getLog().info( "Concurrency config is " + getProperties().toString() );
+                    surefireBooter.addTestSuite( junitDirectoryTestSuite,
+                                                 new Object[]{ getTestClassesDirectory(), includes, excludes,
+                                                     getProperties() } );
+                }
+                else
+                {
+                    if ( isAnyJunit4( junitArtifact ) )
+                    {
+                        junitDirectoryTestSuite = "org.apache.maven.surefire.junit4.JUnit4DirectoryTestSuite";
+                    }
+                    else
+                    {
+                        // fall back to JUnit, which also contains POJO support. Also it can run
+                        // classes compiled against JUnit since it has a dependency on JUnit itself.
+                        junitDirectoryTestSuite = "org.apache.maven.surefire.junit.JUnitDirectoryTestSuite";
+                    }
+                    surefireBooter.addTestSuite( junitDirectoryTestSuite,
+                                                 new Object[]{ getTestClassesDirectory(), includes, excludes} );
+                }
+            }
+        }
+
+        List classpathElements = null;
+        try
+        {
+            classpathElements = generateTestClasspath();
+        }
+        catch ( DependencyResolutionRequiredException e )
+        {
+            throw new MojoExecutionException( "Unable to generate test classpath: " + e, e );
+        }
+
+        getLog().debug( "Test Classpath :" );
+
+        for ( Iterator i = classpathElements.iterator(); i.hasNext(); )
+        {
+            String classpathElement = (String) i.next();
+
+            getLog().debug( "  " + classpathElement );
+
+            surefireBooter.addClassPathUrl( classpathElement );
+        }
+
+        Toolchain tc = getToolchain();
+
+        if ( tc != null )
+        {
+            getLog().info( "Toolchain in " + getPluginName() + "-plugin: " + tc );
+            if ( ForkConfiguration.FORK_NEVER.equals( getForkMode() ) )
+            {
+                setForkMode( ForkConfiguration.FORK_ONCE );
+            }
+            if ( getJvm() != null )
+            {
+                getLog().warn( "Toolchains are ignored, 'executable' parameter is set to " + getJvm() );
+            }
+            else
+            {
+                setJvm( tc.findTool( "java" ) ); //NOI18N
+            }
+        }
+
+        // ----------------------------------------------------------------------
+        // Forking
+        // ----------------------------------------------------------------------
+
+        ForkConfiguration fork = new ForkConfiguration();
+
+        fork.setForkMode( getForkMode() );
+
+        processSystemProperties( !fork.isForking() );
+
+        if ( getLog().isDebugEnabled() )
+        {
+            showMap( getInternalSystemProperties(), "system property" );
+        }
+
+        if ( fork.isForking() )
+        {
+            setUseSystemClassLoader( getUseSystemClassLoader() == null ? Boolean.TRUE : getUseSystemClassLoader() );
+            fork.setUseSystemClassLoader( getUseSystemClassLoader().booleanValue() );
+            fork.setUseManifestOnlyJar( isUseManifestOnlyJar() );
+
+            fork.setSystemProperties( getInternalSystemProperties() );
+
+            if ( "true".equals( getDebugForkedProcess() ) )
+            {
+                setDebugForkedProcess(
+                    "-Xdebug -Xnoagent -Djava.compiler=NONE -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5005" );
+            }
+
+            fork.setDebugLine( getDebugForkedProcess() );
+
+            if ( getJvm() == null || "".equals( getJvm() ) )
+            {
+                // use the same JVM as the one used to run Maven (the "java.home" one)
+                setJvm( System.getProperty( "java.home" ) + File.separator + "bin" + File.separator + "java" );
+                getLog().debug( "Using JVM: " + getJvm() );
+            }
+
+            fork.setJvmExecutable( getJvm() );
+
+            if ( getWorkingDirectory() != null )
+            {
+                fork.setWorkingDirectory( getWorkingDirectory() );
+            }
+            else
+            {
+                fork.setWorkingDirectory( getBasedir() );
+            }
+
+            fork.setArgLine( getArgLine() );
+
+            fork.setEnvironmentVariables( getEnvironmentVariables() );
+
+            if ( getLog().isDebugEnabled() )
+            {
+                showMap( getEnvironmentVariables(), "environment variable" );
+
+                fork.setDebug( true );
+            }
+
+            if ( getArgLine() != null )
+            {
+                List args = Arrays.asList( getArgLine().split( " " ) );
+                if ( args.contains( "-da" ) || args.contains( "-disableassertions" ) )
+                {
+                    setEnableAssertions( false );
+                }
+            }
+        }
+
+        surefireBooter.setFailIfNoTests( getFailIfNoTests() == null ? false : getFailIfNoTests().booleanValue() );
+
+        surefireBooter.setForkedProcessTimeoutInSeconds( getForkedProcessTimeoutInSeconds() );
+
+        surefireBooter.setRedirectTestOutputToFile( isRedirectTestOutputToFile() );
+
+        surefireBooter.setForkConfiguration( fork );
+
+        surefireBooter.setChildDelegation( isChildDelegation() );
+
+        surefireBooter.setEnableAssertions( isEnableAssertions() );
+
+        surefireBooter.setReportsDirectory( getReportsDirectory() );
+
+        addReporters( surefireBooter, fork.isForking() );
+
+        return surefireBooter;
+    }
+
+    protected abstract String[] getDefaultIncludes();
+
+    /**
+     * Generate the test classpath.
+     *
+     * @return List containing the classpath elements
+     * @throws org.apache.maven.artifact.DependencyResolutionRequiredException
+     */
+    public List generateTestClasspath()
+        throws DependencyResolutionRequiredException, MojoExecutionException
+    {
+        List classpath = new ArrayList( 2 + getProject().getArtifacts().size() );
+
+        classpath.add( getTestClassesDirectory().getAbsolutePath() );
+
+        classpath.add( getClassesDirectory().getAbsolutePath() );
+
+        for ( Iterator iter = getProject().getArtifacts().iterator(); iter.hasNext(); )
+        {
+            Artifact artifact = (Artifact) iter.next();
+            if ( artifact.getArtifactHandler().isAddedToClasspath() )
+            {
+                File file = artifact.getFile();
+                if ( file != null )
+                {
+                    classpath.add( file.getPath() );
+                }
+            }
+        }
+
+        // Remove elements from the classpath according to configuration
+        if ( getIgnoreClasspathElements().equals( "all" ) )
+        {
+            classpath.clear();
+        }
+        else if ( getIgnoreClasspathElements().equals( "runtime" ) )
+        {
+            classpath.removeAll( getProject().getRuntimeClasspathElements() );
+        }
+        else if ( !getIgnoreClasspathElements().equals( "none" ) )
+        {
+            throw new MojoExecutionException( "Unsupported value for ignoreClasspathElements parameter: " +
+                getIgnoreClasspathElements() );
+        }
+
+        // Add additional configured elements to the classpath
+        if ( getAdditionalClasspathElements() != null )
+        {
+            for ( Iterator iter = getAdditionalClasspathElements().iterator(); iter.hasNext(); )
+            {
+                String classpathElement = (String) iter.next();
+                classpath.add( classpathElement );
+            }
+        }
+
+        return classpath;
+    }
+
+    private void showMap( Map map, String setting )
+    {
+        for ( Iterator i = map.keySet().iterator(); i.hasNext(); )
+        {
+            String key = (String) i.next();
+            String value = (String) map.get( key );
+            getLog().debug( "Setting " + setting + " [" + key + "]=[" + value + "]" );
+        }
+    }
+
+    private void addProvider( SurefireBooter surefireBooter, String provider, String version,
+                              Artifact filteredArtifact )
+        throws ArtifactNotFoundException, ArtifactResolutionException
+    {
+        Artifact providerArtifact = getArtifactFactory().createDependencyArtifact( "org.apache.maven.surefire", provider,
+                                                                              VersionRange.createFromVersion( version ),
+                                                                              "jar", null, Artifact.SCOPE_TEST );
+        ArtifactResolutionResult result = resolveArtifact( filteredArtifact, providerArtifact );
+
+        for ( Iterator i = result.getArtifacts().iterator(); i.hasNext(); )
+        {
+            Artifact artifact = (Artifact) i.next();
+
+            getLog().debug( "Adding to " + getPluginName() + " test classpath: " + artifact.getFile().getAbsolutePath() + " Scope: " + artifact.getScope() );
+
+            surefireBooter.addSurefireClassPathUrl( artifact.getFile().getAbsolutePath() );
+        }
+    }
+
+    private ArtifactResolutionResult resolveArtifact( Artifact filteredArtifact, Artifact providerArtifact )
+        throws ArtifactResolutionException, ArtifactNotFoundException
+    {
+        ArtifactFilter filter = null;
+        if ( filteredArtifact != null )
+        {
+            filter = new ExcludesArtifactFilter(
+                Collections.singletonList( filteredArtifact.getGroupId() + ":" + filteredArtifact.getArtifactId() ) );
+        }
+
+        Artifact originatingArtifact = getArtifactFactory().createBuildArtifact( "dummy", "dummy", "1.0", "jar" );
+
+        return getArtifactResolver().resolveTransitively( Collections.singleton( providerArtifact ), originatingArtifact,
+                                                          getLocalRepository(), getRemoteRepositories(),
+                                                          getMetadataSource(), filter );
+    }
+
+    private void addArtifact( SurefireBooter surefireBooter, Artifact surefireArtifact )
+        throws ArtifactNotFoundException, ArtifactResolutionException
+    {
+        ArtifactResolutionResult result = resolveArtifact( null, surefireArtifact );
+
+        for ( Iterator i = result.getArtifacts().iterator(); i.hasNext(); )
+        {
+            Artifact artifact = (Artifact) i.next();
+
+            getLog().debug( "Adding to " + getPluginName() + " booter test classpath: " + artifact.getFile().getAbsolutePath() + " Scope: " + artifact.getScope() );
+
+            surefireBooter.addSurefireBootClassPathUrl( artifact.getFile().getAbsolutePath() );
+        }
+    }
+
+    protected void processSystemProperties( boolean setInSystem )
+    {
+        if ( this.getSystemProperties() != null )
+        {
+            for ( Iterator i = getSystemProperties().keySet().iterator(); i.hasNext(); )
+            {
+                String key = (String) i.next();
+                String value = (String) getSystemProperties().get( key );
+                getInternalSystemProperties().setProperty( key, value );
+            }
+        }
+
+        if ( this.getSystemPropertyVariables() != null )
+        {
+            for ( Iterator i = getSystemPropertyVariables().keySet().iterator(); i.hasNext(); )
+            {
+                String key = (String) i.next();
+                String value = (String) getSystemPropertyVariables().get( key );
+                //java Properties does not accept null value
+                if ( value != null )
+                {
+                    getInternalSystemProperties().setProperty( key, value );
+                }
+            }
+        }
+
+        setOriginalSystemProperties( (Properties) System.getProperties().clone() );
+
+        // We used to take all of our system properties and dump them in with the
+        // user specified properties for SUREFIRE-121, causing SUREFIRE-491.
+        // Not gonna do THAT any more... but I'm leaving this code here in case
+        // we need it later when we try to fix SUREFIRE-121 again.
+
+        // Get the properties from the MavenSession instance to make embedded use work correctly
+        Properties userSpecifiedProperties = (Properties) getSession().getExecutionProperties().clone();
+        userSpecifiedProperties.putAll( getInternalSystemProperties() );
+        //systemProperties = userSpecifiedProperties;
+
+        getInternalSystemProperties().setProperty( "basedir", getBasedir().getAbsolutePath() );
+        getInternalSystemProperties().setProperty( "user.dir", getWorkingDirectory().getAbsolutePath() );
+
+        getInternalSystemProperties().setProperty( "localRepository", getLocalRepository().getBasedir() );
+
+        if ( setInSystem )
+        {
+            // Add all system properties configured by the user
+            Iterator iter = getInternalSystemProperties().keySet().iterator();
+
+            while ( iter.hasNext() )
+            {
+                String key = (String) iter.next();
+
+                String value = getInternalSystemProperties().getProperty( key );
+
+                System.setProperty( key, value );
+            }
+        }
+    }
+
+    /**
+     * <p/>
+     * Adds Reporters that will generate reports with different formatting.
+     * <p/>
+     * The Reporter that will be added will be based on the value of the parameter useFile, reportFormat, and
+     * printSummary.
+     *
+     * @param surefireBooter The surefire booter that will run tests.
+     * @param forking
+     */
+    private void addReporters( SurefireBooter surefireBooter, boolean forking )
+    {
+        Boolean trimStackTrace = Boolean.valueOf( this.isTrimStackTrace() );
+        if ( isUseFile() )
+        {
+            if ( isPrintSummary() )
+            {
+                if ( forking )
+                {
+                    surefireBooter.addReport( ForkingConsoleReporter.class.getName(), new Object[]{trimStackTrace} );
+                }
+                else
+                {
+                    surefireBooter.addReport( ConsoleReporter.class.getName(), new Object[]{trimStackTrace} );
+                }
+            }
+
+            if ( BRIEF_REPORT_FORMAT.equals( getReportFormat() ) )
+            {
+                surefireBooter.addReport( BriefFileReporter.class.getName(),
+                                          new Object[]{ getReportsDirectory(), trimStackTrace} );
+            }
+            else if ( PLAIN_REPORT_FORMAT.equals( getReportFormat() ) )
+            {
+                surefireBooter.addReport( FileReporter.class.getName(),
+                                          new Object[]{ getReportsDirectory(), trimStackTrace} );
+            }
+        }
+        else
+        {
+            if ( BRIEF_REPORT_FORMAT.equals( getReportFormat() ) )
+            {
+                surefireBooter.addReport( BriefConsoleReporter.class.getName(), new Object[]{trimStackTrace} );
+            }
+            else if ( PLAIN_REPORT_FORMAT.equals( getReportFormat() ) )
+            {
+                surefireBooter.addReport( DetailedConsoleReporter.class.getName(), new Object[]{trimStackTrace} );
+            }
+        }
+
+        if ( !isDisableXmlReport() )
+        {
+            surefireBooter.addReport( XMLReporter.class.getName(), new Object[]{ getReportsDirectory(), trimStackTrace} );
+        }
+    }
+}

Propchange: maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
------------------------------------------------------------------------------
    svn:mime-type = text/plain

Modified: maven/surefire/trunk/maven-surefire-plugin/pom.xml
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/maven-surefire-plugin/pom.xml?rev=945127&r1=945126&r2=945127&view=diff
==============================================================================
--- maven/surefire/trunk/maven-surefire-plugin/pom.xml (original)
+++ maven/surefire/trunk/maven-surefire-plugin/pom.xml Mon May 17 13:02:39 2010
@@ -114,6 +114,10 @@
       <artifactId>surefire-booter</artifactId>
     </dependency>
     <dependency>
+      <groupId>org.apache.maven.surefire</groupId>
+      <artifactId>maven-surefire-common</artifactId>
+    </dependency>
+    <dependency>
       <groupId>org.codehaus.plexus</groupId>
       <artifactId>plexus-utils</artifactId>
     </dependency>

Modified: maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java?rev=945127&r1=945126&r2=945127&view=diff
==============================================================================
--- maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java (original)
+++ maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java Mon May 17 13:02:39 2010
@@ -45,7 +45,6 @@ import org.apache.maven.artifact.version
 import org.apache.maven.artifact.versioning.OverConstrainedVersionException;
 import org.apache.maven.artifact.versioning.VersionRange;
 import org.apache.maven.execution.MavenSession;
-import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.plugin.MojoFailureException;
 import org.apache.maven.project.MavenProject;
@@ -74,7 +73,7 @@ import org.codehaus.plexus.util.StringUt
  * @phase test
  */
 public class SurefirePlugin
-    extends AbstractMojo
+    extends AbstractSurefireMojo
 {
 
     /**
@@ -142,7 +141,7 @@ public class SurefirePlugin
      * @parameter default-value="${project}"
      * @readonly
      */
-    protected MavenProject project;
+    private MavenProject project;
 
     /**
      * the classpath elements to be excluded from classpath
@@ -501,10 +500,6 @@ public class SurefirePlugin
      */
     private ArtifactMetadataSource metadataSource;
 
-    private static final String BRIEF_REPORT_FORMAT = "brief";
-
-    private static final String PLAIN_REPORT_FORMAT = "plain";
-
     private Properties originalSystemProperties;
 
     /**
@@ -583,7 +578,8 @@ public class SurefirePlugin
         {
             SurefireBooter surefireBooter = constructSurefireBooter();
 
-            getLog().info( "Surefire report directory: " + reportsDirectory );
+            getLog().info(
+                StringUtils.capitalizeFirstLetter( getPluginName() ) + " report directory: " + getReportsDirectory() );
 
             int result;
             try
@@ -599,10 +595,10 @@ public class SurefirePlugin
                 throw new MojoExecutionException( e.getMessage(), e );
             }
 
-            if ( originalSystemProperties != null && !surefireBooter.isForking() )
+            if ( getOriginalSystemProperties() != null && !surefireBooter.isForking() )
             {
                 // restore system properties, only makes sense when not forking..
-                System.setProperties( originalSystemProperties );
+                System.setProperties( getOriginalSystemProperties() );
             }
 
             if ( result == 0 )
@@ -614,7 +610,7 @@ public class SurefirePlugin
 
             if ( result == SurefireBooter.NO_TESTS_EXIT_CODE )
             {
-                if ( ( failIfNoTests == null ) || !failIfNoTests.booleanValue() )
+                if ( ( getFailIfNoTests() == null ) || !getFailIfNoTests().booleanValue() )
                 {
                     return;
                 }
@@ -625,12 +621,12 @@ public class SurefirePlugin
             else
             {
                 // TODO: i18n
-                msg = "There are test failures.\n\nPlease refer to " + reportsDirectory
+                msg = "There are test failures.\n\nPlease refer to " + getReportsDirectory()
                     + " for the individual test results.";
 
             }
 
-            if ( testFailureIgnore )
+            if ( isTestFailureIgnore() )
             {
                 getLog().error( msg );
             }
@@ -641,717 +637,623 @@ public class SurefirePlugin
         }
     }
 
-    private boolean verifyParameters()
-        throws MojoFailureException
+    protected boolean isTestsSkipped()
     {
-        if ( skip || skipTests || skipExec )
-        {
-            getLog().info( "Tests are skipped." );
-            return false;
-        }
+        return isSkip() || isSkipTests() || isSkipExec();
+    }
 
-        if ( !testClassesDirectory.exists() )
-        {
-            if ( failIfNoTests != null && failIfNoTests.booleanValue() )
-            {
-                throw new MojoFailureException( "No tests to run!" );
-            }
-            getLog().info( "No tests to run." );
-            return false;
-        }
+    protected String getPluginName()
+    {
+        return "surefire";
+    }
 
-        if ( !workingDirectory.exists() )
-        {
-            if ( !workingDirectory.mkdirs() )
-            {
-                throw new MojoFailureException( "Cannot create workingDirectory " + workingDirectory );
-            }
-        }
+    protected String[] getDefaultIncludes()
+    {
+        return new String[]{"**/Test*.java", "**/*Test.java", "**/*TestCase.java"};
+    }
 
-        if ( !workingDirectory.isDirectory() )
-        {
-            throw new MojoFailureException( "workingDirectory " + workingDirectory + " exists and is not a directory" );
-        }
+    // now for the implementation of the field accessors
 
-        if ( useSystemClassLoader != null && ForkConfiguration.FORK_NEVER.equals( forkMode ) )
-        {
-            getLog().warn( "useSystemClassloader setting has no effect when not forking" );
-        }
+    public boolean isSkipTests()
+    {
+        return skipTests;
+    }
 
-        return true;
+    public void setSkipTests( boolean skipTests )
+    {
+        this.skipTests = skipTests;
     }
 
-    /**
-     * Converts old TestNG configuration parameters over to new properties based configuration
-     * method. (if any are defined the old way)
-     */
-    private void convertTestNGParameters()
+    public boolean isSkipExec()
     {
-        if ( properties == null )
-        {
-            properties = new Properties();
-        }
+        return skipExec;
+    }
 
-        if ( this.parallel != null )
-        {
-            properties.setProperty( "parallel", this.parallel );
-        }
-        if ( this.excludedGroups != null )
-        {
-            properties.setProperty( "excludegroups", this.excludedGroups );
-        }
-        if ( this.groups != null )
-        {
-            properties.setProperty( "groups", this.groups );
-        }
+    public void setSkipExec( boolean skipExec )
+    {
+        this.skipExec = skipExec;
+    }
 
-        if ( this.threadCount > 0 )
-        {
-            properties.setProperty( "threadcount", new Integer( this.threadCount ).toString() );
-        }
-        if ( this.objectFactory != null )
-        {
-            properties.setProperty( "objectfactory", this.objectFactory );
-        }
+    public boolean isSkip()
+    {
+        return skip;
     }
 
-    private boolean isAnyConcurrencySelected()
+    public void setSkip( boolean skip )
     {
-        return this.parallel != null && this.parallel.trim().length() > 0;
+        this.skip = skip;
     }
 
-    /**
-     * Converts old TestNG configuration parameters over to new properties based configuration
-     * method. (if any are defined the old way)
-     */
-    private void convertJunitCoreParameters()
+    public boolean isTestFailureIgnore()
     {
-        if ( properties == null )
-        {
-            properties = new Properties();
-        }
+        return testFailureIgnore;
+    }
 
-        if ( this.parallel != null )
-        {
-            properties.setProperty( "parallel", this.parallel );
-        }
-        if ( this.threadCount > 0 )
-        {
-            properties.setProperty( "threadCount", new Integer( this.threadCount ).toString() );
-        }
-        if ( this.perCoreThreadCount != null )
-        {
-            properties.setProperty( "perCoreThreadCount", perCoreThreadCount );
-        }
-        if ( this.useUnlimitedThreads != null )
-        {
-            properties.setProperty( "useUnlimitedThreads", useUnlimitedThreads );
-        }
-        Artifact configurableParallelComputer =
-            (Artifact) projectArtifactMap.get( "org.jdogma.junit:configurable-parallel-computer" );
-        properties.setProperty( "configurableParallelComputerPresent",
-                                Boolean.toString( configurableParallelComputer != null ) );
+    public void setTestFailureIgnore( boolean testFailureIgnore )
+    {
+        this.testFailureIgnore = testFailureIgnore;
+    }
 
+    public File getBasedir()
+    {
+        return basedir;
     }
 
-    private boolean isJunit47Compatible( Artifact artifact )
-        throws MojoExecutionException
+    public void setBasedir( File basedir )
     {
-        return isWithinVersionSpec( artifact, "[4.7,)" );
+        this.basedir = basedir;
     }
 
-    private boolean isAnyJunit4( Artifact artifact )
-        throws MojoExecutionException
+    public File getTestClassesDirectory()
     {
-        return isWithinVersionSpec( artifact, "[4.0,)" );
+        return testClassesDirectory;
     }
 
-    private boolean isWithinVersionSpec( Artifact artifact, String versionSpec )
-        throws MojoExecutionException
+    public void setTestClassesDirectory( File testClassesDirectory )
     {
-        if ( artifact == null )
-        {
-            return false;
-        }
-        try
-        {
-            VersionRange range = VersionRange.createFromVersionSpec( versionSpec );
-            try
-            {
-                return range.containsVersion( artifact.getSelectedVersion() );
-            }
-            catch ( NullPointerException e )
-            {
-                return range.containsVersion( new DefaultArtifactVersion( artifact.getBaseVersion() ) );
-            }
-        }
-        catch ( InvalidVersionSpecificationException e )
-        {
-            throw new MojoExecutionException( "Bug in junit 4.7 plugin. Please report with stacktrace" );
-        }
-        catch ( OverConstrainedVersionException e )
-        {
-            throw new MojoExecutionException( "Bug in junit 4.7 plugin. Please report with stacktrace" );
-        }
+        this.testClassesDirectory = testClassesDirectory;
     }
 
+    public File getClassesDirectory()
+    {
+        return classesDirectory;
+    }
 
-    private SurefireBooter constructSurefireBooter()
-        throws MojoExecutionException, MojoFailureException
+    public void setClassesDirectory( File classesDirectory )
     {
-        SurefireBooter surefireBooter = new SurefireBooter();
+        this.classesDirectory = classesDirectory;
+    }
 
-        Artifact surefireArtifact = (Artifact) pluginArtifactMap.get( "org.apache.maven.surefire:surefire-booter" );
-        if ( surefireArtifact == null )
-        {
-            throw new MojoExecutionException( "Unable to locate surefire-booter in the list of plugin artifacts" );
-        }
+    public MavenProject getProject()
+    {
+        return project;
+    }
 
-        surefireArtifact.isSnapshot(); // MNG-2961: before Maven 2.0.8, fixes getBaseVersion to be -SNAPSHOT if needed
+    public void setProject( MavenProject project )
+    {
+        this.project = project;
+    }
 
-        Artifact junitArtifact;
-        Artifact testNgArtifact;
-        try
-        {
-            addArtifact( surefireBooter, surefireArtifact );
+    public String getIgnoreClasspathElements()
+    {
+        return ignoreClasspathElements;
+    }
 
-            junitArtifact = (Artifact) projectArtifactMap.get( junitArtifactName );
-            // SUREFIRE-378, junit can have an alternate artifact name
-            if ( junitArtifact == null && "junit:junit".equals( junitArtifactName ) )
-            {
-                junitArtifact = (Artifact) projectArtifactMap.get( "junit:junit-dep" );
-            }
+    public void setIgnoreClasspathElements( String ignoreClasspathElements )
+    {
+        this.ignoreClasspathElements = ignoreClasspathElements;
+    }
 
-            // TODO: this is pretty manual, but I'd rather not require the plugin > dependencies section right now
-            testNgArtifact = (Artifact) projectArtifactMap.get( testNGArtifactName );
+    public List getAdditionalClasspathElements()
+    {
+        return additionalClasspathElements;
+    }
 
-            if ( testNgArtifact != null )
-            {
-                VersionRange range = VersionRange.createFromVersionSpec( "[4.7,)" );
-                if ( !range.containsVersion( new DefaultArtifactVersion( testNgArtifact.getVersion() ) ) )
-                {
-                    throw new MojoFailureException(
-                        "TestNG support requires version 4.7 or above. You have declared version "
-                            + testNgArtifact.getVersion() );
-                }
+    public void setAdditionalClasspathElements( List additionalClasspathElements )
+    {
+        this.additionalClasspathElements = additionalClasspathElements;
+    }
 
-                convertTestNGParameters();
+    public File getReportsDirectory()
+    {
+        return reportsDirectory;
+    }
 
-                if ( this.testClassesDirectory != null )
-                {
-                    properties.setProperty( "testng.test.classpath", testClassesDirectory.getAbsolutePath() );
-                }
+    public void setReportsDirectory( File reportsDirectory )
+    {
+        this.reportsDirectory = reportsDirectory;
+    }
 
-                addArtifact( surefireBooter, testNgArtifact );
+    public File getTestSourceDirectory()
+    {
+        return testSourceDirectory;
+    }
 
-                // The plugin uses a JDK based profile to select the right testng. We might be explicity using a
-                // different one since its based on the source level, not the JVM. Prune using the filter.
-                addProvider( surefireBooter, "surefire-testng", surefireArtifact.getBaseVersion(), testNgArtifact );
-            }
-            else if ( junitArtifact != null && isAnyJunit4( junitArtifact ) )
-            {
-                if ( isAnyConcurrencySelected() && isJunit47Compatible( junitArtifact ) )
-                {
-                    convertJunitCoreParameters();
-                    addProvider( surefireBooter, "surefire-junit47", surefireArtifact.getBaseVersion(), null );
-                }
-                else
-                {
-                    addProvider( surefireBooter, "surefire-junit4", surefireArtifact.getBaseVersion(), null );
-                }
-            }
-            else
-            {
-                // add the JUnit provider as default - it doesn't require JUnit to be present,
-                // since it supports POJO tests.
-                addProvider( surefireBooter, "surefire-junit", surefireArtifact.getBaseVersion(), null );
-            }
-        }
-        catch ( ArtifactNotFoundException e )
-        {
-            throw new MojoExecutionException(
-                "Unable to locate required surefire provider dependency: " + e.getMessage(), e );
-        }
-        catch ( InvalidVersionSpecificationException e )
-        {
-            throw new MojoExecutionException( "Error determining the TestNG version requested: " + e.getMessage(), e );
-        }
-        catch ( ArtifactResolutionException e )
-        {
-            throw new MojoExecutionException( "Error to resolving surefire provider dependency: " + e.getMessage(), e );
-        }
+    public void setTestSourceDirectory( File testSourceDirectory )
+    {
+        this.testSourceDirectory = testSourceDirectory;
+    }
 
-        if ( suiteXmlFiles != null && suiteXmlFiles.length > 0 && test == null )
-        {
-            if ( testNgArtifact == null )
-            {
-                throw new MojoExecutionException( "suiteXmlFiles is configured, but there is no TestNG dependency" );
-            }
+    public String getTest()
+    {
+        return test;
+    }
 
-            // TODO: properties should be passed in here too
-            surefireBooter.addTestSuite( "org.apache.maven.surefire.testng.TestNGXmlTestSuite",
-                                         new Object[]{suiteXmlFiles, testSourceDirectory.getAbsolutePath(),
-                                             testNgArtifact.getVersion(), testNgArtifact.getClassifier(), properties,
-                                             reportsDirectory} );
-        }
-        else
-        {
-            List includes;
-            List excludes;
+    public void setTest( String test )
+    {
+        this.test = test;
+    }
 
-            if ( test != null )
-            {
-                // Check to see if we are running a single test. The raw parameter will
-                // come through if it has not been set.
+    public List getIncludes()
+    {
+        return includes;
+    }
 
-                // FooTest -> **/FooTest.java
+    public void setIncludes( List includes )
+    {
+        this.includes = includes;
+    }
 
-                includes = new ArrayList();
+    public List getExcludes()
+    {
+        return excludes;
+    }
 
-                excludes = new ArrayList();
+    public void setExcludes( List excludes )
+    {
+        this.excludes = excludes;
+    }
 
-                if ( failIfNoTests == null )
-                {
-                    failIfNoTests = Boolean.TRUE;
-                }
+    public ArtifactRepository getLocalRepository()
+    {
+        return localRepository;
+    }
 
-                String[] testRegexes = StringUtils.split( test, "," );
+    public void setLocalRepository( ArtifactRepository localRepository )
+    {
+        this.localRepository = localRepository;
+    }
 
-                for ( int i = 0; i < testRegexes.length; i++ )
-                {
-                    String testRegex = testRegexes[i];
-                    if ( testRegex.endsWith( ".java" ) )
-                    {
-                        testRegex = testRegex.substring( 0, testRegex.length() - 5 );
-                    }
-                    // Allow paths delimited by '.' or '/'
-                    testRegex = testRegex.replace( '.', '/' );
-                    includes.add( "**/" + testRegex + ".java" );
-                }
-            }
-            else
-            {
-                includes = this.includes;
+    public Properties getSystemProperties()
+    {
+        return systemProperties;
+    }
 
-                excludes = this.excludes;
+    public void setSystemProperties( Properties systemProperties )
+    {
+        this.systemProperties = systemProperties;
+    }
 
-                // defaults here, qdox doesn't like the end javadoc value
-                // Have to wrap in an ArrayList as surefire expects an ArrayList instead of a List for some reason
-                if ( includes == null || includes.size() == 0 )
-                {
-                    includes = new ArrayList(
-                        Arrays.asList( new String[]{"**/Test*.java", "**/*Test.java", "**/*TestCase.java"} ) );
-                }
-                if ( excludes == null || excludes.size() == 0 )
-                {
-                    excludes = new ArrayList( Arrays.asList( new String[]{"**/*$*"} ) );
-                }
-            }
+    public Map getSystemPropertyVariables()
+    {
+        return systemPropertyVariables;
+    }
 
-            if ( testNgArtifact != null )
-            {
-                surefireBooter.addTestSuite( "org.apache.maven.surefire.testng.TestNGDirectoryTestSuite",
-                                             new Object[]{testClassesDirectory, includes, excludes,
-                                                 testSourceDirectory.getAbsolutePath(), testNgArtifact.getVersion(),
-                                                 testNgArtifact.getClassifier(), properties, reportsDirectory} );
-            }
-            else
-            {
-                String junitDirectoryTestSuite;
-                if ( isAnyConcurrencySelected() && isJunit47Compatible( junitArtifact ) )
-                {
-                    junitDirectoryTestSuite = "org.apache.maven.surefire.junitcore.JUnitCoreDirectoryTestSuite";
-                    getLog().info( "Concurrency config is " + properties.toString() );
-                    surefireBooter.addTestSuite( junitDirectoryTestSuite,
-                                                 new Object[]{testClassesDirectory, includes, excludes, properties} );
-                }
-                else
-                {
-                    if ( isAnyJunit4( junitArtifact ) )
-                    {
-                        junitDirectoryTestSuite = "org.apache.maven.surefire.junit4.JUnit4DirectoryTestSuite";
-                    }
-                    else
-                    {
-                        // fall back to JUnit, which also contains POJO support. Also it can run
-                        // classes compiled against JUnit since it has a dependency on JUnit itself.
-                        junitDirectoryTestSuite = "org.apache.maven.surefire.junit.JUnitDirectoryTestSuite";
-                    }
-                    surefireBooter.addTestSuite( junitDirectoryTestSuite,
-                                                 new Object[]{testClassesDirectory, includes, excludes} );
-                }
-            }
-        }
+    public void setSystemPropertyVariables( Map systemPropertyVariables )
+    {
+        this.systemPropertyVariables = systemPropertyVariables;
+    }
 
-        List classpathElements = null;
-        try 
-        {
-            classpathElements = generateTestClasspath();
-        }
-        catch ( DependencyResolutionRequiredException e )
-        {
-            throw new MojoExecutionException( "Unable to generate test classpath: " + e, e );
-        }
-        
-        getLog().debug( "Test Classpath :" );
+    public Properties getProperties()
+    {
+        return properties;
+    }
 
-        for ( Iterator i = classpathElements.iterator(); i.hasNext(); )
-        {
-            String classpathElement = (String) i.next();
+    public void setProperties( Properties properties )
+    {
+        this.properties = properties;
+    }
 
-            getLog().debug( "  " + classpathElement );
+    public Map getPluginArtifactMap()
+    {
+        return pluginArtifactMap;
+    }
 
-            surefireBooter.addClassPathUrl( classpathElement );
-        }
+    public void setPluginArtifactMap( Map pluginArtifactMap )
+    {
+        this.pluginArtifactMap = pluginArtifactMap;
+    }
 
-        Toolchain tc = getToolchain();
+    public Map getProjectArtifactMap()
+    {
+        return projectArtifactMap;
+    }
 
-        if ( tc != null )
-        {
-            getLog().info( "Toolchain in surefire-plugin: " + tc );
-            if ( ForkConfiguration.FORK_NEVER.equals( forkMode ) )
-            {
-                forkMode = ForkConfiguration.FORK_ONCE;
-            }
-            if ( jvm != null )
-            {
-                getLog().warn( "Toolchains are ignored, 'executable' parameter is set to " + jvm );
-            }
-            else
-            {
-                jvm = tc.findTool( "java" ); //NOI18N
-            }
-        }
+    public void setProjectArtifactMap( Map projectArtifactMap )
+    {
+        this.projectArtifactMap = projectArtifactMap;
+    }
 
-        // ----------------------------------------------------------------------
-        // Forking
-        // ----------------------------------------------------------------------
+    public boolean isPrintSummary()
+    {
+        return printSummary;
+    }
 
-        ForkConfiguration fork = new ForkConfiguration();
+    public void setPrintSummary( boolean printSummary )
+    {
+        this.printSummary = printSummary;
+    }
 
-        fork.setForkMode( forkMode );
+    public String getReportFormat()
+    {
+        return reportFormat;
+    }
 
-        processSystemProperties( !fork.isForking() );
+    public void setReportFormat( String reportFormat )
+    {
+        this.reportFormat = reportFormat;
+    }
 
-        if ( getLog().isDebugEnabled() )
-        {
-            showMap( internalSystemProperties, "system property" );
-        }
+    public boolean isUseFile()
+    {
+        return useFile;
+    }
 
-        if ( fork.isForking() )
-        {
-            useSystemClassLoader = useSystemClassLoader == null ? Boolean.TRUE : useSystemClassLoader;
-            fork.setUseSystemClassLoader( useSystemClassLoader.booleanValue() );
-            fork.setUseManifestOnlyJar( useManifestOnlyJar );
+    public void setUseFile( boolean useFile )
+    {
+        this.useFile = useFile;
+    }
 
-            fork.setSystemProperties( internalSystemProperties );
+    public boolean isRedirectTestOutputToFile()
+    {
+        return redirectTestOutputToFile;
+    }
 
-            if ( "true".equals( debugForkedProcess ) )
-            {
-                debugForkedProcess =
-                    "-Xdebug -Xnoagent -Djava.compiler=NONE -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5005";
-            }
+    public void setRedirectTestOutputToFile( boolean redirectTestOutputToFile )
+    {
+        this.redirectTestOutputToFile = redirectTestOutputToFile;
+    }
 
-            fork.setDebugLine( debugForkedProcess );
+    public Boolean getFailIfNoTests()
+    {
+        return failIfNoTests;
+    }
 
-            if ( jvm == null || "".equals( jvm ) )
-            {
-                // use the same JVM as the one used to run Maven (the "java.home" one)
-                jvm = System.getProperty( "java.home" ) + File.separator + "bin" + File.separator + "java";
-                getLog().debug( "Using JVM: " + jvm );
-            }
+    public void setFailIfNoTests( Boolean failIfNoTests )
+    {
+        this.failIfNoTests = failIfNoTests;
+    }
 
-            fork.setJvmExecutable( jvm );
+    public String getForkMode()
+    {
+        return forkMode;
+    }
 
-            if ( workingDirectory != null )
-            {
-                fork.setWorkingDirectory( workingDirectory );
-            }
-            else
-            {
-                fork.setWorkingDirectory( basedir );
-            }
+    public void setForkMode( String forkMode )
+    {
+        this.forkMode = forkMode;
+    }
 
-            fork.setArgLine( argLine );
+    public String getJvm()
+    {
+        return jvm;
+    }
 
-            fork.setEnvironmentVariables( environmentVariables );
+    public void setJvm( String jvm )
+    {
+        this.jvm = jvm;
+    }
 
-            if ( getLog().isDebugEnabled() )
-            {
-                showMap( environmentVariables, "environment variable" );
+    public String getArgLine()
+    {
+        return argLine;
+    }
 
-                fork.setDebug( true );
-            }
+    public void setArgLine( String argLine )
+    {
+        this.argLine = argLine;
+    }
 
-            if ( argLine != null )
-            {
-                List args = Arrays.asList( argLine.split( " " ) );
-                if ( args.contains( "-da" ) || args.contains( "-disableassertions" ) )
-                {
-                    enableAssertions = false;
-                }
-            }
-        }
+    public String getDebugForkedProcess()
+    {
+        return debugForkedProcess;
+    }
 
-        surefireBooter.setFailIfNoTests( failIfNoTests == null ? false : failIfNoTests.booleanValue() );
+    public void setDebugForkedProcess( String debugForkedProcess )
+    {
+        this.debugForkedProcess = debugForkedProcess;
+    }
 
-        surefireBooter.setForkedProcessTimeoutInSeconds( forkedProcessTimeoutInSeconds );
+    public int getForkedProcessTimeoutInSeconds()
+    {
+        return forkedProcessTimeoutInSeconds;
+    }
 
-        surefireBooter.setRedirectTestOutputToFile( redirectTestOutputToFile );
+    public void setForkedProcessTimeoutInSeconds( int forkedProcessTimeoutInSeconds )
+    {
+        this.forkedProcessTimeoutInSeconds = forkedProcessTimeoutInSeconds;
+    }
 
-        surefireBooter.setForkConfiguration( fork );
+    public Map getEnvironmentVariables()
+    {
+        return environmentVariables;
+    }
 
-        surefireBooter.setChildDelegation( childDelegation );
+    public void setEnvironmentVariables( Map environmentVariables )
+    {
+        this.environmentVariables = environmentVariables;
+    }
 
-        surefireBooter.setEnableAssertions( enableAssertions );
+    public File getWorkingDirectory()
+    {
+        return workingDirectory;
+    }
 
-        surefireBooter.setReportsDirectory( reportsDirectory );
+    public void setWorkingDirectory( File workingDirectory )
+    {
+        this.workingDirectory = workingDirectory;
+    }
 
-        addReporters( surefireBooter, fork.isForking() );
+    public boolean isChildDelegation()
+    {
+        return childDelegation;
+    }
 
-        return surefireBooter;
+    public void setChildDelegation( boolean childDelegation )
+    {
+        this.childDelegation = childDelegation;
     }
 
-    /**
-     * Generate the test classpath.
-     * 
-     * @return List containing the classpath elements
-     * @throws DependencyResolutionRequiredException
-     */
-    public List generateTestClasspath()
-        throws DependencyResolutionRequiredException, MojoExecutionException
+    public String getGroups()
     {
-        List classpath = new ArrayList( 2 + project.getArtifacts().size() );
+        return groups;
+    }
 
-        classpath.add( testClassesDirectory.getAbsolutePath() );
+    public void setGroups( String groups )
+    {
+        this.groups = groups;
+    }
 
-        classpath.add( classesDirectory.getAbsolutePath() );
+    public String getExcludedGroups()
+    {
+        return excludedGroups;
+    }
 
-        for ( Iterator iter = project.getArtifacts().iterator(); iter.hasNext(); )
-        {
-            Artifact artifact = (Artifact) iter.next();
-            if ( artifact.getArtifactHandler().isAddedToClasspath() )
-            {
-                File file = artifact.getFile();
-                if ( file != null )
-                {
-                    classpath.add( file.getPath() );
-                }
-            }
-        }
+    public void setExcludedGroups( String excludedGroups )
+    {
+        this.excludedGroups = excludedGroups;
+    }
 
-        // Remove elements from the classpath according to configuration
-        if ( ignoreClasspathElements.equals( "all" ) )
-        {
-            classpath.clear();
-        }
-        else if ( ignoreClasspathElements.equals( "runtime" ) )
-        {
-            classpath.removeAll( project.getRuntimeClasspathElements() );
-        }
-        else if ( !ignoreClasspathElements.equals( "none" ) )
-        {
-            throw new MojoExecutionException( "Unsupported value for ignoreClasspathElements parameter: " +
-                ignoreClasspathElements );
-        }
+    public File[] getSuiteXmlFiles()
+    {
+        return suiteXmlFiles;
+    }
 
-        // Add additional configured elements to the classpath
-        if ( additionalClasspathElements != null )
-        {
-            for ( Iterator iter = additionalClasspathElements.iterator(); iter.hasNext(); )
-            {
-                String classpathElement = (String) iter.next();
-                classpath.add( classpathElement );
-            }
-        }
+    public void setSuiteXmlFiles( File[] suiteXmlFiles )
+    {
+        this.suiteXmlFiles = suiteXmlFiles;
+    }
 
-        return classpath;
+    public String getJunitArtifactName()
+    {
+        return junitArtifactName;
     }
 
-    private void showMap( Map map, String setting )
+    public void setJunitArtifactName( String junitArtifactName )
     {
-        for ( Iterator i = map.keySet().iterator(); i.hasNext(); )
-        {
-            String key = (String) i.next();
-            String value = (String) map.get( key );
-            getLog().debug( "Setting " + setting + " [" + key + "]=[" + value + "]" );
-        }
+        this.junitArtifactName = junitArtifactName;
     }
 
-    private void addProvider( SurefireBooter surefireBooter, String provider, String version,
-                              Artifact filteredArtifact )
-        throws ArtifactNotFoundException, ArtifactResolutionException
-    {
-        Artifact providerArtifact = artifactFactory.createDependencyArtifact( "org.apache.maven.surefire", provider,
-                                                                              VersionRange.createFromVersion( version ),
-                                                                              "jar", null, Artifact.SCOPE_TEST );
-        ArtifactResolutionResult result = resolveArtifact( filteredArtifact, providerArtifact );
+    public String getTestNGArtifactName()
+    {
+        return testNGArtifactName;
+    }
 
-        for ( Iterator i = result.getArtifacts().iterator(); i.hasNext(); )
-        {
-            Artifact artifact = (Artifact) i.next();
+    public void setTestNGArtifactName( String testNGArtifactName )
+    {
+        this.testNGArtifactName = testNGArtifactName;
+    }
 
-            getLog().debug( "Adding to surefire test classpath: " + artifact.getFile().getAbsolutePath() + " Scope: " + artifact.getScope() );
+    public int getThreadCount()
+    {
+        return threadCount;
+    }
 
-            surefireBooter.addSurefireClassPathUrl( artifact.getFile().getAbsolutePath() );
-        }
+    public void setThreadCount( int threadCount )
+    {
+        this.threadCount = threadCount;
     }
 
-    private ArtifactResolutionResult resolveArtifact( Artifact filteredArtifact, Artifact providerArtifact )
-        throws ArtifactResolutionException, ArtifactNotFoundException
+    public String getPerCoreThreadCount()
     {
-        ArtifactFilter filter = null;
-        if ( filteredArtifact != null )
-        {
-            filter = new ExcludesArtifactFilter(
-                Collections.singletonList( filteredArtifact.getGroupId() + ":" + filteredArtifact.getArtifactId() ) );
-        }
+        return perCoreThreadCount;
+    }
 
-        Artifact originatingArtifact = artifactFactory.createBuildArtifact( "dummy", "dummy", "1.0", "jar" );
+    public void setPerCoreThreadCount( String perCoreThreadCount )
+    {
+        this.perCoreThreadCount = perCoreThreadCount;
+    }
 
-        return artifactResolver.resolveTransitively( Collections.singleton( providerArtifact ), originatingArtifact,
-                                                     localRepository, remoteRepositories, metadataSource, filter );
+    public String getUseUnlimitedThreads()
+    {
+        return useUnlimitedThreads;
     }
 
-    private void addArtifact( SurefireBooter surefireBooter, Artifact surefireArtifact )
-        throws ArtifactNotFoundException, ArtifactResolutionException
+    public void setUseUnlimitedThreads( String useUnlimitedThreads )
     {
-        ArtifactResolutionResult result = resolveArtifact( null, surefireArtifact );
+        this.useUnlimitedThreads = useUnlimitedThreads;
+    }
 
-        for ( Iterator i = result.getArtifacts().iterator(); i.hasNext(); )
-        {
-            Artifact artifact = (Artifact) i.next();
+    public String getParallel()
+    {
+        return parallel;
+    }
 
-            getLog().debug( "Adding to surefire booter test classpath: " + artifact.getFile().getAbsolutePath() + " Scope: " + artifact.getScope() );
+    public void setParallel( String parallel )
+    {
+        this.parallel = parallel;
+    }
 
-            surefireBooter.addSurefireBootClassPathUrl( artifact.getFile().getAbsolutePath() );
-        }
+    public boolean isTrimStackTrace()
+    {
+        return trimStackTrace;
     }
 
-    protected void processSystemProperties( boolean setInSystem )
+    public void setTrimStackTrace( boolean trimStackTrace )
     {
-        if ( this.systemProperties != null )
-        {
-            for ( Iterator i = systemProperties.keySet().iterator(); i.hasNext(); )
-            {
-                String key = (String) i.next();
-                String value = (String) systemProperties.get( key );
-                internalSystemProperties.setProperty( key, value );
-            }
-        }
+        this.trimStackTrace = trimStackTrace;
+    }
 
-        if ( this.systemPropertyVariables != null )
-        {
-            for ( Iterator i = systemPropertyVariables.keySet().iterator(); i.hasNext(); )
-            {
-                String key = (String) i.next();
-                String value = (String) systemPropertyVariables.get( key );
-                //java Properties does not accept null value
-                if ( value != null )
-                {
-                    internalSystemProperties.setProperty( key, value );
-                }
-            }
-        }
+    public ArtifactResolver getArtifactResolver()
+    {
+        return artifactResolver;
+    }
 
-        originalSystemProperties = (Properties) System.getProperties().clone();
+    public void setArtifactResolver( ArtifactResolver artifactResolver )
+    {
+        this.artifactResolver = artifactResolver;
+    }
 
-        // We used to take all of our system properties and dump them in with the
-        // user specified properties for SUREFIRE-121, causing SUREFIRE-491.
-        // Not gonna do THAT any more... but I'm leaving this code here in case
-        // we need it later when we try to fix SUREFIRE-121 again.
+    public ArtifactFactory getArtifactFactory()
+    {
+        return artifactFactory;
+    }
 
-        // Get the properties from the MavenSession instance to make embedded use work correctly
-        Properties userSpecifiedProperties = (Properties) session.getExecutionProperties().clone();
-        userSpecifiedProperties.putAll( internalSystemProperties );
-        //systemProperties = userSpecifiedProperties;
+    public void setArtifactFactory( ArtifactFactory artifactFactory )
+    {
+        this.artifactFactory = artifactFactory;
+    }
 
-        internalSystemProperties.setProperty( "basedir", basedir.getAbsolutePath() );
-        internalSystemProperties.setProperty( "user.dir", workingDirectory.getAbsolutePath() );
+    public List getRemoteRepositories()
+    {
+        return remoteRepositories;
+    }
 
-        internalSystemProperties.setProperty( "localRepository", localRepository.getBasedir() );
+    public void setRemoteRepositories( List remoteRepositories )
+    {
+        this.remoteRepositories = remoteRepositories;
+    }
 
-        if ( setInSystem )
-        {
-            // Add all system properties configured by the user
-            Iterator iter = internalSystemProperties.keySet().iterator();
+    public ArtifactMetadataSource getMetadataSource()
+    {
+        return metadataSource;
+    }
 
-            while ( iter.hasNext() )
-            {
-                String key = (String) iter.next();
+    public void setMetadataSource( ArtifactMetadataSource metadataSource )
+    {
+        this.metadataSource = metadataSource;
+    }
 
-                String value = internalSystemProperties.getProperty( key );
+    public Properties getOriginalSystemProperties()
+    {
+        return originalSystemProperties;
+    }
 
-                System.setProperty( key, value );
-            }
-        }
+    public void setOriginalSystemProperties( Properties originalSystemProperties )
+    {
+        this.originalSystemProperties = originalSystemProperties;
     }
 
-    /**
-     * <p/>
-     * Adds Reporters that will generate reports with different formatting.
-     * <p/>
-     * The Reporter that will be added will be based on the value of the parameter useFile, reportFormat, and
-     * printSummary.
-     *
-     * @param surefireBooter The surefire booter that will run tests.
-     * @param forking
-     */
-    private void addReporters( SurefireBooter surefireBooter, boolean forking )
+    public Properties getInternalSystemProperties()
     {
-        Boolean trimStackTrace = Boolean.valueOf( this.trimStackTrace );
-        if ( useFile )
-        {
-            if ( printSummary )
-            {
-                if ( forking )
-                {
-                    surefireBooter.addReport( ForkingConsoleReporter.class.getName(), new Object[]{trimStackTrace} );
-                }
-                else
-                {
-                    surefireBooter.addReport( ConsoleReporter.class.getName(), new Object[]{trimStackTrace} );
-                }
-            }
+        return internalSystemProperties;
+    }
 
-            if ( BRIEF_REPORT_FORMAT.equals( reportFormat ) )
-            {
-                surefireBooter.addReport( BriefFileReporter.class.getName(),
-                                          new Object[]{reportsDirectory, trimStackTrace} );
-            }
-            else if ( PLAIN_REPORT_FORMAT.equals( reportFormat ) )
-            {
-                surefireBooter.addReport( FileReporter.class.getName(),
-                                          new Object[]{reportsDirectory, trimStackTrace} );
-            }
-        }
-        else
-        {
-            if ( BRIEF_REPORT_FORMAT.equals( reportFormat ) )
-            {
-                surefireBooter.addReport( BriefConsoleReporter.class.getName(), new Object[]{trimStackTrace} );
-            }
-            else if ( PLAIN_REPORT_FORMAT.equals( reportFormat ) )
-            {
-                surefireBooter.addReport( DetailedConsoleReporter.class.getName(), new Object[]{trimStackTrace} );
-            }
-        }
+    public void setInternalSystemProperties( Properties internalSystemProperties )
+    {
+        this.internalSystemProperties = internalSystemProperties;
+    }
 
-        if ( !disableXmlReport )
-        {
-            surefireBooter.addReport( XMLReporter.class.getName(), new Object[]{reportsDirectory, trimStackTrace} );
-        }
+    public boolean isDisableXmlReport()
+    {
+        return disableXmlReport;
     }
 
-    private Toolchain getToolchain()
+    public void setDisableXmlReport( boolean disableXmlReport )
     {
-        Toolchain tc = null;
+        this.disableXmlReport = disableXmlReport;
+    }
 
-        if ( toolchainManager != null )
-        {
-            tc = toolchainManager.getToolchainFromBuildContext( "jdk", session );
-        }
+    public Boolean getUseSystemClassLoader()
+    {
+        return useSystemClassLoader;
+    }
+
+    public void setUseSystemClassLoader( Boolean useSystemClassLoader )
+    {
+        this.useSystemClassLoader = useSystemClassLoader;
+    }
+
+    public boolean isUseManifestOnlyJar()
+    {
+        return useManifestOnlyJar;
+    }
+
+    public void setUseManifestOnlyJar( boolean useManifestOnlyJar )
+    {
+        this.useManifestOnlyJar = useManifestOnlyJar;
+    }
+
+    public boolean isEnableAssertions()
+    {
+        return enableAssertions;
+    }
+
+    public void setEnableAssertions( boolean enableAssertions )
+    {
+        this.enableAssertions = enableAssertions;
+    }
+
+    public MavenSession getSession()
+    {
+        return session;
+    }
 
-        return tc;
+    public void setSession( MavenSession session )
+    {
+        this.session = session;
     }
+
+    public String getObjectFactory()
+    {
+        return objectFactory;
+    }
+
+    public void setObjectFactory( String objectFactory )
+    {
+        this.objectFactory = objectFactory;
+    }
+
+    public ToolchainManager getToolchainManager()
+    {
+        return toolchainManager;
+    }
+
+    public void setToolchainManager( ToolchainManager toolchainManager )
+    {
+        this.toolchainManager = toolchainManager;
+    }
+
+    // the following will be refactored out once the common code is all in one place
+
+    public String getEncoding()
+    {
+        return null; // ignore
+    }
+
+    public void setEncoding( String encoding )
+    {
+        // ignore
+    }
+
+    public boolean isSkipITs()
+    {
+        return false; // ignore
+    }
+
+    public void setSkipITs( boolean skipITs )
+    {
+        // ignore
+    }
+
+    public File getSummaryFile()
+    {
+        return null; // ignore
+    }
+
+    public void setSummaryFile( File summaryFile )
+    {
+        // ignore
+    }
+
 }

Modified: maven/surefire/trunk/pom.xml
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/pom.xml?rev=945127&r1=945126&r2=945127&view=diff
==============================================================================
--- maven/surefire/trunk/pom.xml (original)
+++ maven/surefire/trunk/pom.xml Mon May 17 13:02:39 2010
@@ -90,6 +90,7 @@
     <module>surefire-api</module>
     <module>surefire-booter</module>
     <module>surefire-providers</module>
+    <module>maven-surefire-common</module>
     <module>maven-surefire-plugin</module>
     <module>maven-failsafe-plugin</module>
     <module>maven-surefire-report-plugin</module>
@@ -119,6 +120,11 @@
         <version>2.6-SNAPSHOT</version>
       </dependency>
       <dependency>
+        <groupId>org.apache.maven.surefire</groupId>
+        <artifactId>maven-surefire-common</artifactId>
+        <version>2.6-SNAPSHOT</version>
+      </dependency>
+      <dependency>
         <groupId>org.codehaus.plexus</groupId>
         <artifactId>plexus-utils</artifactId>
         <version>1.5.9</version>



Mime
View raw message