svn commit: r383150 - /maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java

[brett]

Author: brett
Date: Sat Mar  4 08:11:25 2006
New Revision: 383150

URL: http://svn.apache.org/viewcvs?rev=383150&view=rev
Log:
fix default thread count and add validation

Modified:
    
maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java

Modified: 
maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java
URL: 
http://svn.apache.org/viewcvs/maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java?rev=383150&r1=383149&r2=383150&view=diff
==============================================================================
--- 
maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java
 (original)
+++ 
maven/plugins/branches/maven-surefire-plugin-testng/src/main/java/org/apache/maven/test/SurefirePlugin.java
 Sat Mar  4 08:11:25 2006
@@ -288,7 +288,7 @@
      * for this execution. Only makes sense to use in conjunction with 
parallel.
      *
      * @parameter expression="${threadCount}"
-     * default-value="0"
+     * default-value="5"
      */
     private int threadCount;
 
@@ -340,6 +340,14 @@
         }
         else
         {
+            if ( parallel )
+            {
+                if ( threadCount < 1 )
+                {
+                    throw new MojoFailureException( "Must have at least one 
thread in parallel mode" );
+                }
+            }
+
             SurefireBooter surefireBooter = constructSurefireBooter();
 
             getLog().info( "Surefire report directory: " + reportsDirectory );