epugh 2004/01/10 08:14:53 Modified: security/memory/src/test/org/apache/fulcrum/security/acl AccessControlListTest.java Log: remove test for unused code Revision Changes Path 1.2 +94 -110 jakarta-turbine-fulcrum/security/memory/src/test/org/apache/fulcrum/security/acl/AccessControlListTest.java Index: AccessControlListTest.java =================================================================== RCS file: /home/cvs/jakarta-turbine-fulcrum/security/memory/src/test/org/apache/fulcrum/security/acl/AccessControlListTest.java,v retrieving revision 1.1 retrieving revision 1.2 diff -u -r1.1 -r1.2 --- AccessControlListTest.java 5 Dec 2003 23:21:55 -0000 1.1 +++ AccessControlListTest.java 10 Jan 2004 16:14:53 -0000 1.2 @@ -56,7 +56,6 @@ import java.util.HashMap; import java.util.Map; -import org.apache.fulcrum.factory.FactoryService; import org.apache.fulcrum.security.GroupManager; import org.apache.fulcrum.security.PermissionManager; import org.apache.fulcrum.security.RoleManager; @@ -66,13 +65,11 @@ import org.apache.fulcrum.security.entity.Permission; import org.apache.fulcrum.security.entity.Role; import org.apache.fulcrum.security.entity.User; - import org.apache.fulcrum.security.model.dynamic.DynamicAccessControlList; import org.apache.fulcrum.security.model.dynamic.DynamicAccessControlListImpl; import org.apache.fulcrum.security.model.dynamic.DynamicModelManager; import org.apache.fulcrum.security.model.dynamic.entity.DynamicGroup; import org.apache.fulcrum.security.model.dynamic.entity.DynamicRole; - import org.apache.fulcrum.security.util.GroupSet; import org.apache.fulcrum.security.util.PermissionSet; import org.apache.fulcrum.security.util.RoleSet; @@ -86,11 +83,11 @@ */ public class AccessControlListTest extends BaseUnitTest { - private FactoryService factoryService = null; + private UserManager userManager; private GroupManager groupManager; - private RoleManager roleManager; - private DynamicModelManager modelManager; + private RoleManager roleManager; + private DynamicModelManager modelManager; private PermissionManager permissionManager; private DynamicAccessControlList acl; private static int counter = 1; @@ -111,33 +108,22 @@ public void setUp() throws Exception { super.setUp(); - - this.setRoleFileName(null); - this.setConfigurationFileName("src/test/AccessControlList.xml"); - factoryService = (FactoryService) this.lookup(FactoryService.ROLE); - SecurityService securityService = (SecurityService) lookup(SecurityService.ROLE); - userManager = securityService.getUserManager(); - groupManager = securityService.getGroupManager(); - roleManager = securityService.getRoleManager(); - permissionManager = securityService.getPermissionManager(); - modelManager = (DynamicModelManager)securityService.getModelManager(); - - } - /* - * Class to test for Object getInstance(String, Object[], String[]) - */ - public void testGetInstanceStringObjectArrayStringArray() throws Exception - { - Object params[] = new Object[1]; - String sourceValu = "testing"; - params[0] = sourceValu; - String signature[] = new String[1]; - signature[0] = "java.lang.String"; - Object object = factoryService.getInstance("java.lang.StringBuffer", params, signature); - assertTrue(object instanceof StringBuffer); - assertEquals(sourceValu, object.toString()); + + this.setRoleFileName(null); + this.setConfigurationFileName("src/test/AccessControlList.xml"); + + SecurityService securityService = + (SecurityService) lookup(SecurityService.ROLE); + userManager = securityService.getUserManager(); + groupManager = securityService.getGroupManager(); + roleManager = securityService.getRoleManager(); + permissionManager = securityService.getPermissionManager(); + modelManager = (DynamicModelManager) securityService.getModelManager(); + } - public void testCreatingDefaultAccessControlListViaFactory() throws Exception + + public void testCreatingDefaultAccessControlListViaFactory() + throws Exception { Group group = getGroup(); Role role = getRole(); @@ -145,8 +131,8 @@ user = userManager.getUserInstance("User 1"); userManager.addUser(user, "secretpassword"); modelManager.grant(user, group); - modelManager.grant(group, role); - modelManager.grant(role, permission); + modelManager.grant(group, role); + modelManager.grant(role, permission); RoleSet roleSet = new RoleSet(); PermissionSet permissionSet = new PermissionSet(); roleSet.add(role); @@ -155,15 +141,12 @@ Map permissionSets = new HashMap(); roleSets.put(group, roleSet); permissionSets.put(role, permissionSet); - Object params[] = new Object[2]; - params[0] = roleSets; - params[1] = permissionSets; - String signature[] = new String[2]; - signature[0] = "java.util.Map"; - signature[1] = "java.util.Map"; - Object object = - factoryService.getInstance(DynamicAccessControlListImpl.class.getName(), params, signature); - assertTrue(object instanceof DynamicAccessControlListImpl); + AccessControlList acl = userManager.getACL(user); + assertTrue(acl instanceof DynamicAccessControlList); + DynamicAccessControlList dacl = (DynamicAccessControlList) acl; + assertTrue(dacl.hasRole(role)); + assertTrue(dacl.hasPermission(permission)); + } public void testGetRolesGroup() throws Exception { @@ -171,8 +154,8 @@ Role role = getRole(); Role role2 = getRole(); Role role3 = getRole(); - modelManager.grant(group, role); - modelManager.grant(group, role2); + modelManager.grant(group, role); + modelManager.grant(group, role2); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -192,10 +175,10 @@ Role role = getRole(); Role role2 = getRole(); Role role3 = getRole(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role2); - modelManager.grant(group2, role3); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role2); + modelManager.grant(group2, role3); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -220,14 +203,14 @@ Permission permission = getPermission(); Permission permission2 = getPermission(); Permission permission3 = getPermission(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role3); - modelManager.grant(role, permission); - modelManager.grant(role, permission2); - modelManager.grant(role, permission3); - modelManager.grant(role2, permission2); - modelManager.grant(role2, permission3); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role3); + modelManager.grant(role, permission); + modelManager.grant(role, permission2); + modelManager.grant(role, permission3); + modelManager.grant(role2, permission2); + modelManager.grant(role2, permission3); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -254,14 +237,14 @@ Permission permission = getPermission(); Permission permission2 = getPermission(); Permission permission3 = getPermission(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role3); - modelManager.grant(role, permission); - modelManager.grant(role, permission2); - modelManager.grant(role, permission3); - modelManager.grant(role2, permission2); - modelManager.grant(role2, permission3); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role3); + modelManager.grant(role, permission); + modelManager.grant(role, permission2); + modelManager.grant(role, permission3); + modelManager.grant(role2, permission2); + modelManager.grant(role2, permission3); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -283,10 +266,10 @@ Role role = getRole(); Role role2 = getRole(); Role role3 = getRole(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role); - modelManager.grant(group2, role3); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role); + modelManager.grant(group2, role3); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -310,10 +293,10 @@ Role role = getRole(); Role role2 = getRole(); Role role3 = getRole(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role); - modelManager.grant(group2, role3); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role); + modelManager.grant(group2, role3); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -340,10 +323,10 @@ Role role = getRole(); Role role2 = getRole(); Role role3 = getRole(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role); - modelManager.grant(group2, role3); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role); + modelManager.grant(group2, role3); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -371,15 +354,15 @@ Permission permission2 = getPermission(); Permission permission3 = getPermission(); Permission permission4 = getPermission(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role3); - modelManager.grant(role, permission); - modelManager.grant(role, permission2); - modelManager.grant(role, permission3); - modelManager.grant(role2, permission2); - modelManager.grant(role2, permission3); - modelManager.grant(role3, permission4); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role3); + modelManager.grant(role, permission); + modelManager.grant(role, permission2); + modelManager.grant(role, permission3); + modelManager.grant(role2, permission2); + modelManager.grant(role2, permission3); + modelManager.grant(role3, permission4); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -412,17 +395,17 @@ Permission permission3 = getPermission(); Permission permission4 = getPermission(); Permission permission5 = getPermission(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role3); - modelManager.grant(group3, role4); - modelManager.grant(role, permission); - modelManager.grant(role, permission2); - modelManager.grant(role, permission3); - modelManager.grant(role2, permission2); - modelManager.grant(role2, permission3); - modelManager.grant(role3, permission4); - modelManager.grant(role4, permission5); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role3); + modelManager.grant(group3, role4); + modelManager.grant(role, permission); + modelManager.grant(role, permission2); + modelManager.grant(role, permission3); + modelManager.grant(role2, permission2); + modelManager.grant(role2, permission3); + modelManager.grant(role3, permission4); + modelManager.grant(role4, permission5); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -458,17 +441,17 @@ Permission permission3 = getPermission(); Permission permission4 = getPermission(); Permission permission5 = getPermission(); - modelManager.grant(group, role); - modelManager.grant(group, role2); - modelManager.grant(group2, role3); - modelManager.grant(group3, role4); - modelManager.grant(role, permission); - modelManager.grant(role, permission2); - modelManager.grant(role, permission3); - modelManager.grant(role2, permission2); - modelManager.grant(role2, permission3); - modelManager.grant(role3, permission4); - modelManager.grant(role4, permission5); + modelManager.grant(group, role); + modelManager.grant(group, role2); + modelManager.grant(group2, role3); + modelManager.grant(group3, role4); + modelManager.grant(role, permission); + modelManager.grant(role, permission2); + modelManager.grant(role, permission3); + modelManager.grant(role2, permission2); + modelManager.grant(role2, permission3); + modelManager.grant(role3, permission4); + modelManager.grant(role4, permission5); Map roleSets = new HashMap(); Map permissionSets = new HashMap(); roleSets.put(group, ((DynamicGroup) group).getRoles()); @@ -504,7 +487,8 @@ } private Permission getPermission() throws Exception { - Permission permission = permissionManager.getPermissionInstance("Permission " + getId()); + Permission permission = + permissionManager.getPermissionInstance("Permission " + getId()); permissionManager.addPermission(permission); return permission; }
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]