[portal-commits] JBoss Portal SVN: r6052 - in trunk/identity: src/main/org/jboss/portal/identity and 2 other directories.

Author: bdaw Date: 2007-01-19 16:37:47 -0500 (Fri, 19 Jan 2007) New Revision: 6052 Modified: trunk/identity/build.xml trunk/identity/src/main/org/jboss/portal/identity/IdentityContextImpl.java trunk/identity/src/main/org/jboss/portal/test/identity/db/DBIdentityTestCase.java trunk/identity/src/main/org/jboss/portal/test/identity/db/DBTestCase.java trunk/identity/src/main/org/jboss/portal/test/identity/ldap/LDAPTestCase.java Log: - correct broken IdentityContextImpl because of tests were failing - small corrections in build scripts to hopefully make them working in CC Modified: trunk/identity/build.xml =================================================================== --- trunk/identity/build.xml 2007-01-19 21:37:46 UTC (rev 6051) +++ trunk/identity/build.xml 2007-01-19 21:37:47 UTC (rev 6052) @@ -303,19 +303,19 @@ <target name="help" depends="_default:help"/> - <target name="deploy-ldap" depends="init" if="HAVE_JDK_1.5"> + <target name="deploy-ldap" depends="init" if="HAVE_JDK_1.5" > <mkdir dir="${jboss.home}/server/default/conf/opends"/> - <copy todir="${jboss.home}/server/default/conf/opends" overwrite="true"> + <copy todir="${jboss.home}/server/default/conf/opends" overwrite="true" failonerror="false"> <fileset dir="../test/output/etc/opends"/> </copy> - <copy file="${build.lib}/test-identity.sar" todir="${jboss.home}/server/${portal.deploy.dir}"/> + <copy file="${build.lib}/test-identity.sar" todir="${jboss.home}/server/${portal.deploy.dir}" failonerror="false"/> <sleep seconds="25"/> </target> <!-- Undeployment of agent --> <target name="undeploy-ldap" depends="init" if="HAVE_JDK_1.5"> - <delete file="${jboss.home}/server/${portal.deploy.dir}/test-identity.sar"/> - <delete dir="${jboss.home}/server/default/conf/opends"/> + <delete file="${jboss.home}/server/${portal.deploy.dir}/test-identity.sar" failonerror="false"/> + <delete dir="${jboss.home}/server/default/conf/opends" failonerror="false"/> </target> <target name="tests" depends="init, tests15"> @@ -450,13 +450,13 @@ <x-test> <!--<test todir="${test.reports}" name="org.jboss.portal.test.identity.db_old.DBUserTestCase"/>--> - <!--<test todir="${test.reports}" name="org.jboss.portal.test.identity.db.DBIdentityTestCase"/>--> + <test todir="${test.reports}" name="org.jboss.portal.test.identity.db.DBIdentityTestCase"/> <!--<test todir="${test.reports}" name="org.jboss.portal.test.identity.UserProtoTestCase"/>--> <!--<test todir="${test.reports}" name="org.jboss.portal.test.identity.ConfigurationTestCase"/>--> - <!--<test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPSimpleUserModuleTestCase"/>--> - <!--<test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPSimpleRoleModuleTestCase"/>--> - <!--<test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPStaticGroupMembershipModuleTestCase"/>--> - <!--<test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPStaticRoleMembershipModuleTestCase"/>--> + <test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPSimpleUserModuleTestCase"/> + <test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPSimpleRoleModuleTestCase"/> + <test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPStaticGroupMembershipModuleTestCase"/> + <test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPStaticRoleMembershipModuleTestCase"/> <test todir="${test.reports}" name="org.jboss.portal.test.identity.ldap.LDAPUserProfileModuleTestCase"/> </x-test> <x-classpath> Modified: trunk/identity/src/main/org/jboss/portal/identity/IdentityContextImpl.java =================================================================== --- trunk/identity/src/main/org/jboss/portal/identity/IdentityContextImpl.java 2007-01-19 21:37:46 UTC (rev 6051) +++ trunk/identity/src/main/org/jboss/portal/identity/IdentityContextImpl.java 2007-01-19 21:37:47 UTC (rev 6052) @@ -24,6 +24,8 @@ import org.jboss.portal.jems.as.system.AbstractJBossService; import org.jboss.portal.common.util.CopyOnWriteRegistry; +import java.util.Collection; + /** * Keeps references to all identity related modules to enable them interactions * @@ -42,13 +44,19 @@ public void register(Object object, String name) throws IdentityException { - registry.register(object, name); + if (!registry.register(name, object)) + { + throw new IdentityException("Cannot register object in IdentityContext with name: " + name); + } if (log.isDebugEnabled()) log.debug("registering object: " + name + " ; " + object.getClass()); } public void unregister(String name) { - registry.unregister(name); + if (!registry.unregister(name)) + { + log.error("Cannot unregister object from IdentityContext with name: " + name); + } if (log.isDebugEnabled()) log.debug("unregistering object: " + name); } @@ -61,4 +69,14 @@ } return o; } + + public Collection getKeys() + { + return registry.getKeys(); + } + + public Collection getValues() + { + return registry.getRegistrations(); + } } Modified: trunk/identity/src/main/org/jboss/portal/test/identity/db/DBIdentityTestCase.java =================================================================== --- trunk/identity/src/main/org/jboss/portal/test/identity/db/DBIdentityTestCase.java 2007-01-19 21:37:46 UTC (rev 6051) +++ trunk/identity/src/main/org/jboss/portal/test/identity/db/DBIdentityTestCase.java 2007-01-19 21:37:47 UTC (rev 6052) @@ -21,11 +21,8 @@ */ package org.jboss.portal.test.identity.db; -import org.jboss.portal.identity.IdentityServiceControllerImpl; -import org.jboss.portal.identity.IdentityContext; import org.jboss.portal.identity.db.HibernateUserImpl; -import org.jboss.portal.identity.ProfileMap; -import org.jboss.portal.identity.User; +import org.jboss.portal.identity.*; import org.jboss.portal.identity.service.UserModuleService; import org.jboss.portal.identity.service.UserProfileModuleService; import org.jboss.portal.identity.service.MembershipModuleService; @@ -35,6 +32,8 @@ import junit.framework.TestSuite; import java.util.Date; +import java.util.Collection; +import java.util.Iterator; import java.text.SimpleDateFormat; /** @@ -43,6 +42,7 @@ */ public class DBIdentityTestCase extends DBTestCase { + private static final org.jboss.logging.Logger log = org.jboss.logging.Logger.getLogger(DBTestCase.class); public static TestSuite suite() throws Exception { @@ -75,6 +75,7 @@ controller.setRegisterMBeans(false); controller.start(); identityContext = controller.getIdentityContext(); + this.userModule = (UserModuleService)identityContext.getObject(IdentityContext.TYPE_USER_MODULE); this.roleModule = (RoleModuleService)identityContext.getObject(IdentityContext.TYPE_ROLE_MODULE); this.membershipModule = (MembershipModuleService)identityContext.getObject(IdentityContext.TYPE_MEMBERSHIP_MODULE); Modified: trunk/identity/src/main/org/jboss/portal/test/identity/db/DBTestCase.java =================================================================== --- trunk/identity/src/main/org/jboss/portal/test/identity/db/DBTestCase.java 2007-01-19 21:37:46 UTC (rev 6051) +++ trunk/identity/src/main/org/jboss/portal/test/identity/db/DBTestCase.java 2007-01-19 21:37:47 UTC (rev 6052) @@ -56,7 +56,7 @@ Logger.getRoot().addAppender(appender); Logger.getRoot().setLevel(Level.DEBUG); Logger.getLogger("org.hibernate").setLevel(Level.ERROR); - Logger.getLogger("org.jboss.portal.identity2").setLevel(Level.DEBUG); + Logger.getLogger("org.jboss.portal.identity").setLevel(Level.DEBUG); } public static TestSuite createTestSuite(Class clazz) throws Exception Modified: trunk/identity/src/main/org/jboss/portal/test/identity/ldap/LDAPTestCase.java =================================================================== --- trunk/identity/src/main/org/jboss/portal/test/identity/ldap/LDAPTestCase.java 2007-01-19 21:37:46 UTC (rev 6051) +++ trunk/identity/src/main/org/jboss/portal/test/identity/ldap/LDAPTestCase.java 2007-01-19 21:37:47 UTC (rev 6052) @@ -66,7 +66,7 @@ Appender appender = new ConsoleAppender(new SimpleLayout()); Logger.getRoot().addAppender(appender); Logger.getRoot().setLevel(Level.INFO); - Logger.getLogger("org.jboss.portal.identity2").setLevel(Level.DEBUG); + Logger.getLogger("org.jboss.portal.identity").setLevel(Level.DEBUG); } public static TestSuite createTestSuite(Class clazz) throws Exception