You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by gt...@apache.org on 2012/03/02 14:53:31 UTC
svn commit: r1296204 -
/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/LDAPAuthorizationMapTest.java
Author: gtully
Date: Fri Mar 2 13:53:31 2012
New Revision: 1296204
URL: http://svn.apache.org/viewvc?rev=1296204&view=rev
Log:
https://issues.apache.org/jira/browse/AMQ-3749 - Composite destinations break simple authorisation through role aggregation. additional tests and fix - fix broken test, expecting wrong behaviour
Modified:
activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/LDAPAuthorizationMapTest.java
Modified: activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/LDAPAuthorizationMapTest.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/LDAPAuthorizationMapTest.java?rev=1296204&r1=1296203&r2=1296204&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/LDAPAuthorizationMapTest.java (original)
+++ activemq/trunk/activemq-core/src/test/java/org/apache/activemq/security/LDAPAuthorizationMapTest.java Fri Mar 2 13:53:31 2012
@@ -141,10 +141,7 @@ public class LDAPAuthorizationMapTest ex
public void testComposite() {
ActiveMQDestination q1 = new ActiveMQQueue("queue1,topic://topic1");
Set aclsq1 = authMap.getWriteACLs(q1);
- assertEquals(3, aclsq1.size());
- assertTrue(aclsq1.contains(new GroupPrincipal("role1")));
- assertTrue(aclsq1.contains(new GroupPrincipal("role2")));
- assertTrue(aclsq1.contains(new GroupPrincipal("role3")));
+ assertEquals(0, aclsq1.size());
}
@Test