Author: olamy
Date: Wed Mar 28 15:13:55 2012
New Revision: 1306404

URL: http://svn.apache.org/viewvc?rev=1306404&view=rev
Log:
force add in ScmTckTestCase and add a new method in CommandParameters with 
default value for getBoolean

Modified:
    
maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/CommandParameters.java
    
maven/scm/trunk/maven-scm-test/src/main/java/org/apache/maven/scm/ScmTckTestCase.java

Modified: 
maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/CommandParameters.java
URL: 
http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/CommandParameters.java?rev=1306404&r1=1306403&r2=1306404&view=diff
==============================================================================
--- 
maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/CommandParameters.java
 (original)
+++ 
maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/CommandParameters.java
 Wed Mar 28 15:13:55 2012
@@ -34,8 +34,8 @@ public class CommandParameters
     implements Serializable
 {
     private static final long serialVersionUID = -7346070735958137283L;
-    
-    private Map<String, Object> parameters = new HashMap<String,Object>();
+
+    private Map<String, Object> parameters = new HashMap<String, Object>();
 
     // ----------------------------------------------------------------------
     // String
@@ -115,9 +115,9 @@ public class CommandParameters
     /**
      * Return the parameter value as int or the default value if it doesn't 
exist.
      *
-     * @param parameter The parameter
-     * @return The parameter value as a int
+     * @param parameter    The parameter
      * @param defaultValue The defaultValue
+     * @return The parameter value as a int
      * @throws ScmException if the value is in the wrong type
      */
     public int getInt( CommandParameter parameter, int defaultValue )
@@ -166,7 +166,7 @@ public class CommandParameters
     /**
      * Return the parameter value as String or the default value if it doesn't 
exist.
      *
-     * @param parameter The parameter
+     * @param parameter    The parameter
      * @param defaultValue The defaultValue
      * @return The parameter value as a Date
      * @throws ScmException if the value is in the wrong type
@@ -198,7 +198,7 @@ public class CommandParameters
      * Return the parameter value as boolean.
      *
      * @param parameter The parameter
-     * @return The parameter value as a String
+     * @return The parameter value as a boolean
      * @throws ScmException if the parameter doesn't exist
      */
     public boolean getBoolean( CommandParameter parameter )
@@ -207,6 +207,19 @@ public class CommandParameters
         return Boolean.valueOf( getString( parameter ) ).booleanValue();
     }
 
+    /**
+     * Return the parameter value as boolean.
+     *
+     * @param parameter    The parameter
+     * @param defaultValue default value if parameter not exists
+     * @return The parameter value as a boolean
+     */
+    public boolean getBoolean( CommandParameter parameter, boolean 
defaultValue )
+        throws ScmException
+    {
+        return Boolean.valueOf( getString( parameter, Boolean.toString( 
defaultValue ) ) ).booleanValue();
+    }
+
     // ----------------------------------------------------------------------
     // ScmVersion
     // ----------------------------------------------------------------------
@@ -267,8 +280,7 @@ public class CommandParameters
     }
 
     /**
-     *
-     * @param parameter not null
+     * @param parameter    not null
      * @param defaultValue could be null
      * @return an array of files
      * @throws ScmException if any
@@ -304,8 +316,6 @@ public class CommandParameters
         return (ScmBranchParameters) getObject( ScmBranchParameters.class, 
parameter, new ScmBranchParameters() );
     }
 
-    
-    
     // ----------------------------------------------------------------------
     //
     // ----------------------------------------------------------------------
@@ -334,8 +344,8 @@ public class CommandParameters
     /**
      * Return the value object or the default value if it doesn't exist.
      *
-     * @param clazz     The type of the parameter value
-     * @param parameter The parameter
+     * @param clazz        The type of the parameter value
+     * @param parameter    The parameter
      * @param defaultValue The defaultValue
      * @return The parameter value
      * @throws ScmException if the defaultValue is in the wrong type
@@ -352,8 +362,9 @@ public class CommandParameters
 
         if ( clazz != null && !clazz.isAssignableFrom( object.getClass() ) )
         {
-            throw new ScmException( "Wrong parameter type for '" + 
parameter.getName() + ". " + "Expected: "
-                + clazz.getName() + ", got: " + object.getClass().getName() );
+            throw new ScmException(
+                "Wrong parameter type for '" + parameter.getName() + ". " + 
"Expected: " + clazz.getName() + ", got: "
+                    + object.getClass().getName() );
         }
 
         return object;

Modified: 
maven/scm/trunk/maven-scm-test/src/main/java/org/apache/maven/scm/ScmTckTestCase.java
URL: 
http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-test/src/main/java/org/apache/maven/scm/ScmTckTestCase.java?rev=1306404&r1=1306403&r2=1306404&view=diff
==============================================================================
--- 
maven/scm/trunk/maven-scm-test/src/main/java/org/apache/maven/scm/ScmTckTestCase.java
 (original)
+++ 
maven/scm/trunk/maven-scm-test/src/main/java/org/apache/maven/scm/ScmTckTestCase.java
 Wed Mar 28 15:13:55 2012
@@ -163,7 +163,7 @@ public abstract class ScmTckTestCase
         CommandParameters commandParameters = new CommandParameters();
         commandParameters.setString( CommandParameter.FORCE_ADD, 
Boolean.TRUE.toString() );
 
-        AddScmResult result = provider.add( repository, new ScmFileSet( 
workingDirectory, file ) );
+        AddScmResult result = provider.add( repository, new ScmFileSet( 
workingDirectory, file ), commandParameters );
 
         assertTrue( "Check result was successful, output: " + 
result.getCommandOutput(), result.isSuccess() );
 


Reply via email to