svn commit: r1133543 - in /archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security: DefaultUserRepositoriesTest.java RoleManagerTest.java

[ol...@apache.org]

Author: olamy
Date: Wed Jun  8 20:12:23 2011
New Revision: 1133543

URL: http://svn.apache.org/viewvc?rev=1133543&view=rev
Log:
ensure cleanup datas for other tests

Modified:
    archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/DefaultUserRepositoriesTest.java
    archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/RoleManagerTest.java

Modified: archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/DefaultUserRepositoriesTest.java
URL: http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/DefaultUserRepositoriesTest.java?rev=1133543&r1=1133542&r2=1133543&view=diff
==============================================================================
--- archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/DefaultUserRepositoriesTest.java (original)
+++ archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/DefaultUserRepositoriesTest.java Wed Jun  8 20:12:23 2011
@@ -19,14 +19,15 @@ package org.apache.maven.archiva.securit
  * under the License.
  */
 
-import java.util.List;
-
 import org.apache.commons.lang.StringUtils;
+import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 
+import java.util.List;
+
 /**
- * DefaultUserRepositoriesTest 
+ * DefaultUserRepositoriesTest
  *
  * @version $Id$
  */
@@ -36,7 +37,8 @@ public class DefaultUserRepositoriesTest
 
 
     @Before
-    public void setUp() throws Exception
+    public void setUp()
+        throws Exception
     {
         super.setUp();
         restoreGuestInitialValues( USER_ALPACA );
@@ -67,10 +69,21 @@ public class DefaultUserRepositoriesTest
         // the global repo observer role.
         assignGlobalRepositoryObserverRole( USER_ADMIN );
 
-        assertRepoIds( new String[] { "central", "corporate" }, userRepos.getObservableRepositoryIds( USER_ALPACA ) );
-        assertRepoIds( new String[] { "coporate" }, userRepos.getObservableRepositoryIds( USER_GUEST ) );
-        assertRepoIds( new String[] { "central", "internal", "corporate", "snapshots", "secret" }, userRepos
-            .getObservableRepositoryIds( USER_ADMIN ) );
+        assertRepoIds( new String[]{ "central", "corporate" }, userRepos.getObservableRepositoryIds( USER_ALPACA ) );
+        assertRepoIds( new String[]{ "coporate" }, userRepos.getObservableRepositoryIds( USER_GUEST ) );
+        assertRepoIds( new String[]{ "central", "internal", "corporate", "snapshots", "secret" },
+                       userRepos.getObservableRepositoryIds( USER_ADMIN ) );
+
+    }
+
+    @After
+    public void tearDown()
+        throws Exception
+    {
+        super.tearDown();
+        restoreGuestInitialValues( USER_ALPACA );
+        restoreGuestInitialValues( USER_GUEST );
+        restoreGuestInitialValues( USER_ADMIN );
     }
 
     private void assertRepoIds( String[] expectedRepoIds, List<String> observableRepositoryIds )
@@ -80,8 +93,9 @@ public class DefaultUserRepositoriesTest
         if ( expectedRepoIds.length != observableRepositoryIds.size() )
         {
             fail( "Size of Observable Repository Ids wrong, expected <" + expectedRepoIds.length + "> but got <"
-                + observableRepositoryIds.size() + "> instead. \nExpected: [" + StringUtils.join( expectedRepoIds, "," )
-                + "]\nActual: [" + StringUtils.join( observableRepositoryIds.iterator(), "," ) + "]" );
+                      + observableRepositoryIds.size() + "> instead. \nExpected: ["
+                      + StringUtils.join( expectedRepoIds, "," ) + "]\nActual: ["
+                      + StringUtils.join( observableRepositoryIds.iterator(), "," ) + "]" );
         }
     }
 

Modified: archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/RoleManagerTest.java
URL: http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/RoleManagerTest.java?rev=1133543&r1=1133542&r2=1133543&view=diff
==============================================================================
--- archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/RoleManagerTest.java (original)
+++ archiva/trunk/archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/maven/archiva/security/RoleManagerTest.java Wed Jun  8 20:12:23 2011
@@ -38,9 +38,7 @@ import javax.inject.Inject;
 public class RoleManagerTest
     extends TestCase
 {
-    /**
-     * @plexus.requirement role-hint="default"
-     */
+
     @Inject
     RoleManager roleManager;