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

[br...@apache.org]

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 );