[jboss-cvs] JBossAS SVN: r68963 - 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-14 22:44:38 -0500 (Mon, 14 Jan 2008)
New Revision: 68963

Modified:
   projects/security/security-jboss-sx/trunk/identity/src/main/org/jboss/security/identity/plugins/SimpleRoleGroup.java
   projects/security/security-jboss-sx/trunk/identity/src/tests/org/jboss/test/identity/impl/RoleUnitTestCase.java
Log:
SECURITY-109: addRole method

Modified: projects/security/security-jboss-sx/trunk/identity/src/main/org/jboss/security/identity/plugins/SimpleRoleGroup.java
===================================================================
--- projects/security/security-jboss-sx/trunk/identity/src/main/org/jboss/security/identity/plugins/SimpleRoleGroup.java	2008-01-15 03:44:09 UTC (rev 68962)
+++ projects/security/security-jboss-sx/trunk/identity/src/main/org/jboss/security/identity/plugins/SimpleRoleGroup.java	2008-01-15 03:44:38 UTC (rev 68963)
@@ -84,6 +84,17 @@
       return RoleType.group;
    }
 
+   /**
+    * @see RoleGroup#addRole(Role)
+    */
+   public void addRole(Role role)
+   {
+      this.roles.add(role);  
+   }
+
+   /**
+    * @see RoleGroup#getRoles()
+    */
    public List<Role> getRoles()
    {
       return roles;

Modified: projects/security/security-jboss-sx/trunk/identity/src/tests/org/jboss/test/identity/impl/RoleUnitTestCase.java
===================================================================
--- projects/security/security-jboss-sx/trunk/identity/src/tests/org/jboss/test/identity/impl/RoleUnitTestCase.java	2008-01-15 03:44:09 UTC (rev 68962)
+++ projects/security/security-jboss-sx/trunk/identity/src/tests/org/jboss/test/identity/impl/RoleUnitTestCase.java	2008-01-15 03:44:38 UTC (rev 68963)
@@ -27,6 +27,7 @@
 import org.jboss.security.identity.RoleFactory;
 import org.jboss.security.identity.plugins.SimpleRole;
 import org.jboss.security.identity.plugins.SimpleRoleGroup;
+ 
 
 //$Id$
 
@@ -74,4 +75,11 @@
       
       assertTrue(firstRoleGroup.containsAtleastOneRole(secondRoleGroup));
    }
+   
+   public void testSimpleRoleGroup()
+   {
+      SimpleRoleGroup srg = new SimpleRoleGroup("Roles");
+      srg.addRole(new SimpleRole("aRole"));
+      assertTrue(srg.containsAll(new SimpleRole("aRole"))); 
+   }
 }