svn commit: r606812 - /maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

[df...@apache.org]

Author: dfabulich
Date: Tue Dec 25 13:08:29 2007
New Revision: 606812

URL: http://svn.apache.org/viewvc?rev=606812&view=rev
Log:
[SUREFIRE-417] Made a new "skipTests" parameter to replace skipExec.

Modified:
    maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

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=606812&r1=606811&r2=606812&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 Tue Dec 25 13:08:29 2007
@@ -71,20 +71,32 @@
 public class SurefirePlugin
     extends AbstractMojo
 {
+
     /**
-     * Set this to 'true' to bypass unit tests entirely. Its use is NOT RECOMMENDED, but quite convenient on occasion.
+     * Set this to 'true' to skip running tests, but still compile them. Its use is NOT RECOMMENDED, but quite
+     * convenient on occasion.
      * 
-     * @parameter expression="${maven.test.skip}"
+     * @parameter expression="${skipTests}"
      */
-    private boolean skip;
-
+    private boolean skipTests;
+    
     /**
-     * Set this to 'true' to bypass unit tests execution, but still compile them. Its use is NOT RECOMMENDED, but quite
-     * convenient on occasion.
+     * DEPRECATED This old parameter is just like skipTests, but bound to the old property maven.test.skip.exec.
+     * Use -DskipTests instead; it's shorter.
      * 
+     * @deprecated
      * @parameter expression="${maven.test.skip.exec}"
      */
     private boolean skipExec;
+    
+    /**
+     * Set this to 'true' to bypass unit tests entirely. Its use is NOT RECOMMENDED, especially if you
+     * enable it using the "maven.test.skip" property, because maven.test.skip disables both running the
+     * tests and compiling the tests.  Consider using the skipTests parameter instead.
+     * 
+     * @parameter expression="${maven.test.skip}"
+     */
+    private boolean skip;
 
     /**
      * Set this to true to ignore a failure during testing. Its use is NOT RECOMMENDED, but quite convenient on
@@ -512,7 +524,7 @@
     private boolean verifyParameters()
         throws MojoFailureException
     {
-        if ( skip || skipExec )
+        if ( skip || skipTests || skipExec )
         {
             getLog().info( "Tests are skipped." );
             return false;
@@ -1022,7 +1034,7 @@
      */
     public boolean isSkipExec()
     {
-        return this.skipExec;
+        return this.skipTests;
     }
 
     /**
@@ -1030,6 +1042,6 @@
      */
     public void setSkipExec( boolean skipExec )
     {
-        this.skipExec = skipExec;
+        this.skipTests = skipExec;
     }
 }