You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ca...@apache.org on 2006/08/31 02:46:44 UTC
svn commit: r438731 -
/maven/shared/trunk/maven-user/maven-user-model/src/test/java/org/apache/maven/user/model/impl/DefaultUserManagerTest.java
Author: carlos
Date: Wed Aug 30 17:46:44 2006
New Revision: 438731
URL: http://svn.apache.org/viewvc?rev=438731&view=rev
Log:
uncomment failing tests
Modified:
maven/shared/trunk/maven-user/maven-user-model/src/test/java/org/apache/maven/user/model/impl/DefaultUserManagerTest.java
Modified: maven/shared/trunk/maven-user/maven-user-model/src/test/java/org/apache/maven/user/model/impl/DefaultUserManagerTest.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-user/maven-user-model/src/test/java/org/apache/maven/user/model/impl/DefaultUserManagerTest.java?rev=438731&r1=438730&r2=438731&view=diff
==============================================================================
--- maven/shared/trunk/maven-user/maven-user-model/src/test/java/org/apache/maven/user/model/impl/DefaultUserManagerTest.java (original)
+++ maven/shared/trunk/maven-user/maven-user-model/src/test/java/org/apache/maven/user/model/impl/DefaultUserManagerTest.java Wed Aug 30 17:46:44 2006
@@ -150,33 +150,33 @@
assertEquals( smcqueen, actual );
}
-// public void testUpdateUser() throws Exception
-// {
-// assertNotNull( getUserManager() );
-//
-// assertEquals( "New UserManager should contain no users.", 0, getUserManager().getUsers().size() ); //$NON-NLS-1$
-//
-// User jgarner = new User();
-// jgarner.setUsername( "jgarner" ); //$NON-NLS-1$
-// jgarner.setFullName( "James Garner" ); //$NON-NLS-1$
-// jgarner.setPassword( "the scrounger" ); //$NON-NLS-1$
-// getUserManager().addUser( jgarner );
-//
-// User fetched = getUserManager().getUser( "jgarner" ); //$NON-NLS-1$
-// assertNotNull( "User should not be null.", fetched ); //$NON-NLS-1$
-// assertEquals( "James Garner", fetched.getFullName() ); //$NON-NLS-1$
-//
-// // Change the full name, and update the user.
-// fetched.setFullName( "Flight Lt. Hendley" ); //$NON-NLS-1$
-// getUserManager().updateUser( fetched );
-//
-// // Should not change number of users being tracked.
-// assertEquals( 1, getUserManager().getUsers().size() );
-//
-// // Fetch the user and test for updated Full Name.
-// User actual = getUserManager().getUser( "jgarner" ); //$NON-NLS-1$
-// assertEquals( "Flight Lt. Hendley", actual.getFullName() ); //$NON-NLS-1$
-// }
+ public void testUpdateUser() throws Exception
+ {
+ assertNotNull( getUserManager() );
+
+ assertEquals( "New UserManager should contain no users.", 0, getUserManager().getUsers().size() ); //$NON-NLS-1$
+
+ User jgarner = new User();
+ jgarner.setUsername( "jgarner" ); //$NON-NLS-1$
+ jgarner.setFullName( "James Garner" ); //$NON-NLS-1$
+ jgarner.setPassword( "the scrounger" ); //$NON-NLS-1$
+ getUserManager().addUser( jgarner );
+
+ User fetched = getUserManager().getUser( "jgarner" ); //$NON-NLS-1$
+ assertNotNull( "User should not be null.", fetched ); //$NON-NLS-1$
+ assertEquals( "James Garner", fetched.getFullName() ); //$NON-NLS-1$
+
+ // Change the full name, and update the user.
+ fetched.setFullName( "Flight Lt. Hendley" ); //$NON-NLS-1$
+ getUserManager().updateUser( fetched );
+
+ // Should not change number of users being tracked.
+ assertEquals( 1, getUserManager().getUsers().size() );
+
+ // Fetch the user and test for updated Full Name.
+ User actual = getUserManager().getUser( "jgarner" ); //$NON-NLS-1$
+ assertEquals( "Flight Lt. Hendley", actual.getFullName() ); //$NON-NLS-1$
+ }
public void testRemoveUser() throws Exception
{
@@ -472,33 +472,33 @@
assertEquals( 2, actual.getPermissions().size() );
}
-// public void testPolicyLoginFailureLock() throws Exception
-// {
-// assertNotNull( getUserManager() );
-//
-// assertEquals( "New UserManager should contain no users.", 0, getUserManager().getUsers().size() ); //$NON-NLS-1$
-// assertEquals( "New UserManager should contain no groups.", 0, getUserManager().getUserGroups().size() ); //$NON-NLS-1$
-// assertNotNull( "New UserManager should have a Security Policy", getUserManager().getSecurityPolicy() ); //$NON-NLS-1$
-//
-// User rattenborough = new User();
-// rattenborough.setUsername( "rattenborough" ); //$NON-NLS-1$
-// rattenborough.setFullName( "Richard Attenborough" ); //$NON-NLS-1$
-// rattenborough.setPassword( "the big x" ); //$NON-NLS-1$
-//
-// getUserManager().addUser( rattenborough );
-//
-// assertEquals( 1, getUserManager().getUsers().size() );
-//
-// // Setup the policy.
-// ( (DefaultUserSecurityPolicy) getUserManager().getSecurityPolicy() ).setAllowedLoginAttempts( 3 );
-//
-// assertFalse( getUserManager().login( "rattenborough", "the big lebowski" ) );
-// assertFalse( getUserManager().getUser( "rattenborough" ).isLocked() );
-//
-// assertFalse( getUserManager().login( "rattenborough", "the big cheese" ) );
-// assertFalse( getUserManager().getUser( "rattenborough" ).isLocked() );
-//
-// assertFalse( getUserManager().login( "rattenborough", "big x" ) );
-// assertTrue( getUserManager().getUser( "rattenborough" ).isLocked() );
-// }
+ public void testPolicyLoginFailureLock() throws Exception
+ {
+ assertNotNull( getUserManager() );
+
+ assertEquals( "New UserManager should contain no users.", 0, getUserManager().getUsers().size() ); //$NON-NLS-1$
+ assertEquals( "New UserManager should contain no groups.", 0, getUserManager().getUserGroups().size() ); //$NON-NLS-1$
+ assertNotNull( "New UserManager should have a Security Policy", getUserManager().getSecurityPolicy() ); //$NON-NLS-1$
+
+ User rattenborough = new User();
+ rattenborough.setUsername( "rattenborough" ); //$NON-NLS-1$
+ rattenborough.setFullName( "Richard Attenborough" ); //$NON-NLS-1$
+ rattenborough.setPassword( "the big x" ); //$NON-NLS-1$
+
+ getUserManager().addUser( rattenborough );
+
+ assertEquals( 1, getUserManager().getUsers().size() );
+
+ // Setup the policy.
+ ( (DefaultUserSecurityPolicy) getUserManager().getSecurityPolicy() ).setAllowedLoginAttempts( 3 );
+
+ assertFalse( getUserManager().login( "rattenborough", "the big lebowski" ) );
+ assertFalse( getUserManager().getUser( "rattenborough" ).isLocked() );
+
+ assertFalse( getUserManager().login( "rattenborough", "the big cheese" ) );
+ assertFalse( getUserManager().getUser( "rattenborough" ).isLocked() );
+
+ assertFalse( getUserManager().login( "rattenborough", "big x" ) );
+ assertTrue( getUserManager().getUser( "rattenborough" ).isLocked() );
+ }
}