activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gtu...@apache.org
Subject svn commit: r1422875 - /activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/CachedLDAPSecurityLegacyTest.java
Date Mon, 17 Dec 2012 11:48:39 GMT
Author: gtully
Date: Mon Dec 17 11:48:38 2012
New Revision: 1422875

URL: http://svn.apache.org/viewvc?rev=1422875&view=rev
Log:
https://issues.apache.org/jira/browse/AMQ-4213 - fix test regressions, add producer now correctly
fails before a send

Modified:
    activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/CachedLDAPSecurityLegacyTest.java

Modified: activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/CachedLDAPSecurityLegacyTest.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/CachedLDAPSecurityLegacyTest.java?rev=1422875&r1=1422874&r2=1422875&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/CachedLDAPSecurityLegacyTest.java
(original)
+++ activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/CachedLDAPSecurityLegacyTest.java
Mon Dec 17 11:48:38 2012
@@ -86,9 +86,10 @@ public class CachedLDAPSecurityLegacyTes
         conn.start();
         Queue queue = sess.createQueue("ADMIN.FOO");
 
-        MessageProducer producer = sess.createProducer(queue);
         try {
-            producer.send(sess.createTextMessage("test"));
+            // https://issues.apache.org/jira/browse/AMQ-4213
+            // sync send on producer so it throws
+            MessageProducer producer = sess.createProducer(queue);
             fail("expect auth exception");
         } catch (JMSException expected) {
         }
@@ -102,9 +103,10 @@ public class CachedLDAPSecurityLegacyTes
         conn.start();
         Queue queue = sess.createQueue("TEST.FOO,ADMIN.FOO");
 
-        MessageProducer producer = sess.createProducer(queue);
         try {
-            producer.send(sess.createTextMessage("test"));
+            // https://issues.apache.org/jira/browse/AMQ-4213
+            // sync send on producer so it throws
+            MessageProducer producer = sess.createProducer(queue);
             fail("expect auth exception");
         } catch (JMSException expected) {
         }



Mime
View raw message