[jboss-cvs] JBossAS SVN: r69225 - in projects/security/security-jboss-sx/trunk/identity/src: tests/org/jboss/test/identity/impl and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: anil.saldhana at jboss.com
Date: 2008-01-22 18:16:55 -0500 (Tue, 22 Jan 2008)
New Revision: 69225

Modified:
   projects/security/security-jboss-sx/trunk/identity/src/main/org/jboss/security/identity/plugins/SimpleRole.java
   projects/security/security-jboss-sx/trunk/identity/src/tests/org/jboss/test/identity/impl/RoleGroupUnitTestCase.java
Log:
fix the anybody role issue

Modified: projects/security/security-jboss-sx/trunk/identity/src/main/org/jboss/security/identity/plugins/SimpleRole.java
===================================================================
--- projects/security/security-jboss-sx/trunk/identity/src/main/org/jboss/security/identity/plugins/SimpleRole.java	2008-01-22 23:12:20 UTC (rev 69224)
+++ projects/security/security-jboss-sx/trunk/identity/src/main/org/jboss/security/identity/plugins/SimpleRole.java	2008-01-22 23:16:55 UTC (rev 69225)
@@ -64,7 +64,7 @@
    {
       if(anotherRole.getType() == RoleType.simple)
       {
-         if(ANYBODY.equals(roleName) || ANYBODY.equals(anotherRole.getRoleName()))
+         if(ANYBODY.equals(roleName))
             return true; 
          return roleName.equals(anotherRole.getRoleName()); 
       }

Modified: projects/security/security-jboss-sx/trunk/identity/src/tests/org/jboss/test/identity/impl/RoleGroupUnitTestCase.java
===================================================================
--- projects/security/security-jboss-sx/trunk/identity/src/tests/org/jboss/test/identity/impl/RoleGroupUnitTestCase.java	2008-01-22 23:12:20 UTC (rev 69224)
+++ projects/security/security-jboss-sx/trunk/identity/src/tests/org/jboss/test/identity/impl/RoleGroupUnitTestCase.java	2008-01-22 23:16:55 UTC (rev 69225)
@@ -44,10 +44,7 @@
       
       SimpleRoleGroup srg = new SimpleRoleGroup("Roles");
       srg.addRole(SimpleRole.ANYBODY_ROLE);
-      assertTrue(srg.containsRole(aRole));
-      
-      //All simple roles contain ANYBODY_ROLE
-      assertTrue(aRole.containsAll(SimpleRole.ANYBODY_ROLE));
+      assertTrue(srg.containsRole(aRole)); 
    }