diff options
| author | Robert Gemmell <robbie@apache.org> | 2013-05-01 12:14:40 +0000 |
|---|---|---|
| committer | Robert Gemmell <robbie@apache.org> | 2013-05-01 12:14:40 +0000 |
| commit | d5b676fcd2b16ba2d416963b39df2d22828fc7d5 (patch) | |
| tree | ba7b945356484f704c701269908045344e8a8ae1 /qpid/java/broker/src/test | |
| parent | df37800ef7d66d4f02eb99a36e272cd967f7bf07 (diff) | |
| download | qpid-python-d5b676fcd2b16ba2d416963b39df2d22828fc7d5.tar.gz | |
QPID-4795: update naming of the UserManagement MBeans to remove restriction preventing there being multiple AuthenticationProviders capable of user management
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1477965 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/java/broker/src/test')
| -rw-r--r-- | qpid/java/broker/src/test/java/org/apache/qpid/server/model/adapter/AuthenticationProviderFactoryTest.java | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/qpid/java/broker/src/test/java/org/apache/qpid/server/model/adapter/AuthenticationProviderFactoryTest.java b/qpid/java/broker/src/test/java/org/apache/qpid/server/model/adapter/AuthenticationProviderFactoryTest.java index 4a76c1837c..fbb1e4105f 100644 --- a/qpid/java/broker/src/test/java/org/apache/qpid/server/model/adapter/AuthenticationProviderFactoryTest.java +++ b/qpid/java/broker/src/test/java/org/apache/qpid/server/model/adapter/AuthenticationProviderFactoryTest.java @@ -33,7 +33,6 @@ import java.util.UUID; import junit.framework.TestCase; -import org.apache.qpid.server.configuration.IllegalConfigurationException; import org.apache.qpid.server.model.AuthenticationProvider; import org.apache.qpid.server.model.Broker; import org.apache.qpid.server.model.PasswordCredentialManagingAuthenticationProvider; @@ -114,8 +113,7 @@ public class AuthenticationProviderFactoryTest extends TestCase return provider; } - @SuppressWarnings("unchecked") - public void testCreatePasswordCredentialManagingAuthenticationProviderFailsWhenAnotherOneAlreadyExist() + public void testCreatePasswordCredentialManagingAuthenticationProviderFailsWhenAnotherOneAlready() { Broker broker = mock(Broker.class); PasswordCredentialManagingAuthenticationProvider anotherProvider = mock(PasswordCredentialManagingAuthenticationProvider.class); @@ -127,15 +125,12 @@ public class AuthenticationProviderFactoryTest extends TestCase when(loader.atLeastOneInstanceOf(AuthenticationManagerFactory.class)).thenReturn(Collections.singleton(managerFactory)); AuthenticationProviderFactory providerFactory = new AuthenticationProviderFactory(loader); - try - { - providerFactory.create(UUID.randomUUID(), broker, new HashMap<String, Object>()); - fail("Creation of anaother PasswordCredentialManagingAuthenticationProvider should fail"); - } - catch (IllegalConfigurationException e) - { - // pass - } + + UUID randomUUID = UUID.randomUUID(); + AuthenticationProvider provider = providerFactory.create(randomUUID, broker, new HashMap<String, Object>()); + + assertNotNull("Provider is not created", provider); + assertEquals("Unexpected ID", randomUUID, provider.getId()); } @SuppressWarnings("unchecked") |