Fix build failure in usermanager.implementation

[controller.git] / opendaylight / usermanager / implementation / src / test / java / org / opendaylight / controller / usermanager / internal / UserManagerImplTest.java

diff --git a/opendaylight/usermanager/implementation/src/test/java/org/opendaylight/controller/usermanager/internal/UserManagerImplTest.java b/opendaylight/usermanager/implementation/src/test/java/org/opendaylight/controller/usermanager/internal/UserManagerImplTest.java
index 67c273b73f98ebdb9a856062a0676d77393a1604..f2cbe784fce5377347515a3a3131a5d2ec58d3ac 100644 (file)
--- a/opendaylight/usermanager/implementation/src/test/java/org/opendaylight/controller/usermanager/internal/UserManagerImplTest.java
+++ b/opendaylight/usermanager/implementation/src/test/java/org/opendaylight/controller/usermanager/internal/UserManagerImplTest.java
@@ -104,15 +104,9 @@ public class UserManagerImplTest {
         }
     }
 
-    /**
-     * Test method for
-     * {@link org.opendaylight.controller.usermanager.internal.UserManager#addAAAProvider(org.opendaylight.controller.usermanager.IAAAProvider)}
-     * .
-     */
-    @Test
-    public void testAddAAAProvider() {
+    private IAAAProvider getAnonymousAAAProvider(final String providerName) {
         // instantiate an anonymous AAAProvider
-        IAAAProvider a3p = new IAAAProvider() {
+        return new IAAAProvider() {
 
             @Override
             public AuthResponse authService(String userName, String password,
@@ -122,25 +116,26 @@ public class UserManagerImplTest {
 
             @Override
             public String getName() {
-                return "dummyAAAProvider";
+                return providerName;
             }
         };
-
-        um.addAAAProvider(a3p);
-        assertEquals(a3p, um.getAAAProvider("dummyAAAProvider"));
-
     }
 
     /**
      * Test method for
+     * {@link org.opendaylight.controller.usermanager.internal.UserManager#addAAAProvider(org.opendaylight.controller.usermanager.IAAAProvider)}
+     * and for
      * {@link org.opendaylight.controller.usermanager.internal.UserManager#removeAAAProvider(org.opendaylight.controller.usermanager.IAAAProvider)}
-     * and for for
+     * and
      * {@link org.opendaylight.controller.usermanager.internal.UserManager#getAAAProvider(java.lang.String)}
-     * .
      */
     @Test
-    public void testRemoveAAAProvider() {
-        um.removeAAAProvider(um.getAAAProvider("dummyAAAProvider"));
+    public void testAddGetRemoveAAAProvider() {
+        final String providerName = "dummyAAAProvider";
+        IAAAProvider a3p = getAnonymousAAAProvider(providerName);
+        um.addAAAProvider(a3p);
+        assertEquals(a3p, um.getAAAProvider(providerName));
+        um.removeAAAProvider(um.getAAAProvider(providerName));
         assertTrue(um.getAAAProviderNames().isEmpty());
     }