diff options
| author | Rajith Muditha Attapattu <rajith@apache.org> | 2013-03-06 21:04:51 +0000 |
|---|---|---|
| committer | Rajith Muditha Attapattu <rajith@apache.org> | 2013-03-06 21:04:51 +0000 |
| commit | a23bc789330f50ef6222284ba8c4d51f5a1a22c5 (patch) | |
| tree | 51c53847f1733ebae00a11f090935f01c8e5e36b /java/client | |
| parent | 1335e9b25d34601e9ee6cc462c02b3bef885263e (diff) | |
| download | qpid-python-a23bc789330f50ef6222284ba8c4d51f5a1a22c5.tar.gz | |
QPID-3396 Changed the test cases to relect that username and password
is optional.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1453559 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'java/client')
2 files changed, 4 insertions, 5 deletions
diff --git a/java/client/src/main/java/org/apache/qpid/client/transport/ClientConnectionDelegate.java b/java/client/src/main/java/org/apache/qpid/client/transport/ClientConnectionDelegate.java index 2775354acf..e9b946d5b7 100644 --- a/java/client/src/main/java/org/apache/qpid/client/transport/ClientConnectionDelegate.java +++ b/java/client/src/main/java/org/apache/qpid/client/transport/ClientConnectionDelegate.java @@ -98,7 +98,8 @@ public class ClientConnectionDelegate extends ClientDelegate " Client restricted itself to : " + (restrictionList != null ? restrictionList : "no restriction")); } - if (CallbackHandlerRegistry.getInstance().isUserPassRequired(selectedMech)) + if ((_connectionURL.getUsername() == null || _connectionURL.getPassword() == null) + && CallbackHandlerRegistry.getInstance().isUserPassRequired(selectedMech)) { throw new ConnectionException("Username and Password is required for the selected mechanism : " + selectedMech + " Broker allows : " + brokerMechanisms + diff --git a/java/client/src/test/java/org/apache/qpid/test/unit/client/connectionurl/ConnectionURLTest.java b/java/client/src/test/java/org/apache/qpid/test/unit/client/connectionurl/ConnectionURLTest.java index 8c193622e3..7c461fa13f 100644 --- a/java/client/src/test/java/org/apache/qpid/test/unit/client/connectionurl/ConnectionURLTest.java +++ b/java/client/src/test/java/org/apache/qpid/test/unit/client/connectionurl/ConnectionURLTest.java @@ -384,11 +384,10 @@ public class ConnectionURLTest extends TestCase try { new AMQConnectionURL(url); - fail("URL Should not parse"); } catch (URLSyntaxException urise) { - assertTrue(urise.getMessage().startsWith("User information not found on url")); + fail("User information is optional in url"); } } @@ -400,11 +399,10 @@ public class ConnectionURLTest extends TestCase try { new AMQConnectionURL(url); - fail("URL Should not parse"); } catch (URLSyntaxException urise) { - assertTrue(urise.getMessage().startsWith("User information not found on url")); + fail("User information is optional in url"); } } |