Author: tv
Date: Thu Aug 16 20:01:13 2012
New Revision: 1374028

URL: http://svn.apache.org/viewvc?rev=1374028&view=rev
Log:
Fix test of null values

Modified:
    
turbine/fulcrum/trunk/security/api/src/test/org/apache/fulcrum/security/entity/impl/SecurityEntityImplTest.java

Modified: 
turbine/fulcrum/trunk/security/api/src/test/org/apache/fulcrum/security/entity/impl/SecurityEntityImplTest.java
URL: 
http://svn.apache.org/viewvc/turbine/fulcrum/trunk/security/api/src/test/org/apache/fulcrum/security/entity/impl/SecurityEntityImplTest.java?rev=1374028&r1=1374027&r2=1374028&view=diff
==============================================================================
--- 
turbine/fulcrum/trunk/security/api/src/test/org/apache/fulcrum/security/entity/impl/SecurityEntityImplTest.java
 (original)
+++ 
turbine/fulcrum/trunk/security/api/src/test/org/apache/fulcrum/security/entity/impl/SecurityEntityImplTest.java
 Thu Aug 16 20:01:13 2012
@@ -30,9 +30,9 @@ public class SecurityEntityImplTest exte
 {
 
     /**
-       * Constructor for SecurityEntityImplTest.
-       * @param arg0
-       */
+     * Constructor for SecurityEntityImplTest.
+     * @param arg0
+     */
     public SecurityEntityImplTest(String arg0)
     {
         super(arg0);
@@ -49,14 +49,14 @@ public class SecurityEntityImplTest exte
         assertEquals("hello",se.getName());
         se.setName("HelLo");
                assertEquals("hello",se.getName());
-               //try {
+               try
+               {
                    se.setName(null);
-                   assertNull(se.getName());
-       /*          fail("Should throw an InvalidParameterException");
+                   fail("Should throw an InvalidParameterException");
                }
-               catch(InvalidParameterException ipe){
+               catch(IllegalArgumentException ipe)
+               {
                    //good
                }
-       */
     }
 }


Reply via email to