From ba23b978143c6dbae51d4bc245dc4a99f485a940 Mon Sep 17 00:00:00 2001 From: Jeff Burke <Jeff.Burke@nrc-cnrc.gc.ca> Date: Mon, 27 Jul 2015 11:30:27 -0700 Subject: [PATCH] ac2: move group web classes into own package, update and change user passwords in DAO. --- projects/cadcAccessControl-Server/build.xml | 15 +--- .../web/{ => groups}/ACSearchRunner.java | 34 +++++---- .../{ => groups}/AddGroupMemberAction.java | 2 +- .../web/{ => groups}/AddUserMemberAction.java | 4 +- .../web/{ => groups}/CreateGroupAction.java | 2 +- .../web/{ => groups}/DeleteGroupAction.java | 2 +- .../web/{ => groups}/GetGroupAction.java | 2 +- .../web/{ => groups}/GetGroupNamesAction.java | 2 +- .../server/web/{ => groups}/GroupLogInfo.java | 2 +- .../server/web/{ => groups}/GroupsAction.java | 2 +- .../web/{ => groups}/GroupsActionFactory.java | 2 +- .../web/{ => groups}/GroupsServlet.java | 2 +- .../web/{ => groups}/ModifyGroupAction.java | 2 +- .../{ => groups}/RemoveGroupMemberAction.java | 3 +- .../{ => groups}/RemoveUserMemberAction.java | 3 +- .../cadc/ac/server/web/users/UsersAction.java | 24 +++---- .../test/LdapConfig.test.properties | 2 +- .../cadc/ac/server/RequestValidatorTest.java | 4 +- .../cadc/ac/server/ldap/LdapUserDAOTest.java | 69 +++---------------- .../AddGroupMemberActionTest.java | 2 +- .../{ => groups}/AddUserMemberActionTest.java | 5 +- .../{ => groups}/DeleteGroupActionTest.java | 9 +-- .../{ => groups}/GetGroupNamesActionTest.java | 2 +- .../{ => groups}/GroupActionFactoryTest.java | 4 +- .../web/{ => groups}/GroupsActionTest.java | 6 +- .../RemoveGroupMemberActionTest.java | 6 +- .../RemoveUserMemberActionTest.java | 6 +- 27 files changed, 63 insertions(+), 155 deletions(-) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/ACSearchRunner.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/AddGroupMemberAction.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/AddUserMemberAction.java (98%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/CreateGroupAction.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/DeleteGroupAction.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/GetGroupAction.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/GetGroupNamesAction.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/GroupLogInfo.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/GroupsAction.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/GroupsActionFactory.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/GroupsServlet.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/ModifyGroupAction.java (99%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/RemoveGroupMemberAction.java (98%) rename projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/{ => groups}/RemoveUserMemberAction.java (98%) rename projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/{ => groups}/AddGroupMemberActionTest.java (99%) rename projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/{ => groups}/AddUserMemberActionTest.java (98%) rename projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/{ => groups}/DeleteGroupActionTest.java (95%) rename projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/{ => groups}/GetGroupNamesActionTest.java (99%) rename projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/{ => groups}/GroupActionFactoryTest.java (99%) rename projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/{ => groups}/GroupsActionTest.java (98%) rename projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/{ => groups}/RemoveGroupMemberActionTest.java (97%) rename projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/{ => groups}/RemoveUserMemberActionTest.java (98%) diff --git a/projects/cadcAccessControl-Server/build.xml b/projects/cadcAccessControl-Server/build.xml index 2619c10d..12e68846 100644 --- a/projects/cadcAccessControl-Server/build.xml +++ b/projects/cadcAccessControl-Server/build.xml @@ -137,19 +137,7 @@ <property name="lib.commons-logging" value="${ext.lib}/commons-logging.jar"/> <property name="testingJars" -<<<<<<< HEAD value="${lib.commons-logging}:${dev.junit}:${dev.jsonassert}:${dev.httpunit}:${dev.easyMock}:${dev.selenium.server}:${dev.objenesis}:${lib.js}:${lib.nekoHTML}:${lib.xerces}"/> -======= - value="${lib.commons-logging}:${dev.junit}:${dev.httpunit}:${dev.easyMock}:${dev.selenium.server}:${dev.objenesis}:${lib.js}:${lib.nekoHTML}:${lib.xerces}"/> - - <target name="setup-test"> - <echo>******************</echo> - <echo>******************</echo> - <echo>Don't forget to set the ca.nrc.cadc.util.PropertiesReader.dir system property first!</echo> - <echo>e.g. ant -Dca.nrc.cadc.util.PropertiesReader.dir=test clean build test</echo> - <echo>******************</echo> - <echo>******************</echo> - </target> <target name="test" depends="compile,compile-test"> <echo message="Running test suite..." /> @@ -161,10 +149,9 @@ </classpath> <sysproperty key="ca.nrc.cadc.util.PropertiesReader.dir" value="test"/> <test name="ca.nrc.cadc.ac.server.ldap.LdapUserDAOTest" /> - <!--<test name="ca.nrc.cadc.ac.server.ldap.LdapGroupDAOTest" />--> + <test name="ca.nrc.cadc.ac.server.ldap.LdapGroupDAOTest" /> <formatter type="plain" usefile="false" /> </junit> </target> ->>>>>>> s1734 </project> diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/ACSearchRunner.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/ACSearchRunner.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/ACSearchRunner.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/ACSearchRunner.java index df069f8b..a6409ad4 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/ACSearchRunner.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/ACSearchRunner.java @@ -66,31 +66,15 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; - -import java.io.IOException; -import java.security.AccessControlContext; -import java.security.AccessControlException; -import java.security.AccessController; -import java.security.Principal; -import java.util.Collection; -import java.util.Date; -import java.util.Iterator; -import java.util.Set; - -import javax.security.auth.Subject; -import javax.security.auth.x500.X500Principal; -import javax.servlet.http.HttpServletResponse; - -import org.apache.log4j.Logger; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; import ca.nrc.cadc.ac.GroupNotFoundException; -import ca.nrc.cadc.ac.xml.GroupsWriter; import ca.nrc.cadc.ac.UserNotFoundException; import ca.nrc.cadc.ac.server.GroupPersistence; import ca.nrc.cadc.ac.server.PluginFactory; import ca.nrc.cadc.ac.server.RequestValidator; +import ca.nrc.cadc.ac.xml.GroupsWriter; import ca.nrc.cadc.auth.AuthenticationUtil; import ca.nrc.cadc.auth.HttpPrincipal; import ca.nrc.cadc.net.TransientException; @@ -100,9 +84,23 @@ import ca.nrc.cadc.uws.server.JobRunner; import ca.nrc.cadc.uws.server.JobUpdater; import ca.nrc.cadc.uws.server.SyncOutput; import ca.nrc.cadc.uws.util.JobLogInfo; +import org.apache.log4j.Logger; + +import javax.security.auth.Subject; +import javax.security.auth.x500.X500Principal; +import javax.servlet.http.HttpServletResponse; +import java.io.IOException; import java.io.OutputStream; import java.io.OutputStreamWriter; +import java.security.AccessControlContext; +import java.security.AccessControlException; +import java.security.AccessController; +import java.security.Principal; import java.util.ArrayList; +import java.util.Collection; +import java.util.Date; +import java.util.Iterator; +import java.util.Set; public class ACSearchRunner implements JobRunner { diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/AddGroupMemberAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/AddGroupMemberAction.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/AddGroupMemberAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/AddGroupMemberAction.java index c842e970..61669fa2 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/AddGroupMemberAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/AddGroupMemberAction.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; import ca.nrc.cadc.ac.GroupAlreadyExistsException; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/AddUserMemberAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/AddUserMemberAction.java similarity index 98% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/AddUserMemberAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/AddUserMemberAction.java index 7c687340..cdf6093a 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/AddUserMemberAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/AddUserMemberAction.java @@ -66,18 +66,16 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; import ca.nrc.cadc.ac.MemberAlreadyExistsException; import ca.nrc.cadc.ac.User; import ca.nrc.cadc.ac.server.GroupPersistence; -import ca.nrc.cadc.ac.server.UserPersistence; import ca.nrc.cadc.auth.AuthenticationUtil; import java.security.Principal; import java.util.ArrayList; import java.util.List; -import java.util.Set; public class AddUserMemberAction extends GroupsAction { diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/CreateGroupAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/CreateGroupAction.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/CreateGroupAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/CreateGroupAction.java index 47cdc9bc..f282d722 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/CreateGroupAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/CreateGroupAction.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import java.io.InputStream; import java.util.ArrayList; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/DeleteGroupAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/DeleteGroupAction.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/DeleteGroupAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/DeleteGroupAction.java index 8f619fa3..2c96f279 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/DeleteGroupAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/DeleteGroupAction.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import java.util.ArrayList; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GetGroupAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GetGroupAction.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GetGroupAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GetGroupAction.java index 97a2b52e..3d7d9646 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GetGroupAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GetGroupAction.java @@ -65,7 +65,7 @@ * $Revision: 4 $ * ************************************************************************ - */package ca.nrc.cadc.ac.server.web; + */package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; import ca.nrc.cadc.ac.server.GroupPersistence; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GetGroupNamesAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GetGroupNamesAction.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GetGroupNamesAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GetGroupNamesAction.java index a29e5fcf..a62ef952 100644 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GetGroupNamesAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GetGroupNamesAction.java @@ -67,7 +67,7 @@ ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import java.io.Writer; import java.util.Collection; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupLogInfo.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupLogInfo.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupLogInfo.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupLogInfo.java index 51261011..38739a08 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupLogInfo.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupLogInfo.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.log.ServletLogInfo; import java.util.List; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsAction.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsAction.java index 42c0ac3c..4ef18115 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsAction.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import java.io.IOException; import java.security.AccessControlException; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsActionFactory.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsActionFactory.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsActionFactory.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsActionFactory.java index 9263dde6..c2aaf942 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsActionFactory.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsActionFactory.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import java.io.IOException; import java.net.URL; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsServlet.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsServlet.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsServlet.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsServlet.java index dd62ed5c..cd468232 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/GroupsServlet.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/GroupsServlet.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import java.io.IOException; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/ModifyGroupAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/ModifyGroupAction.java similarity index 99% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/ModifyGroupAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/ModifyGroupAction.java index 425951d7..444e335d 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/ModifyGroupAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/ModifyGroupAction.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import java.io.InputStream; import java.util.ArrayList; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/RemoveGroupMemberAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/RemoveGroupMemberAction.java similarity index 98% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/RemoveGroupMemberAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/RemoveGroupMemberAction.java index 269984b1..a05d17f9 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/RemoveGroupMemberAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/RemoveGroupMemberAction.java @@ -66,14 +66,13 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; import ca.nrc.cadc.ac.GroupNotFoundException; import ca.nrc.cadc.ac.server.GroupPersistence; import java.util.ArrayList; import java.util.List; -import java.util.Set; public class RemoveGroupMemberAction extends GroupsAction { diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/RemoveUserMemberAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/RemoveUserMemberAction.java similarity index 98% rename from projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/RemoveUserMemberAction.java rename to projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/RemoveUserMemberAction.java index 9b4d34f7..20dcf8b4 100755 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/RemoveUserMemberAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/groups/RemoveUserMemberAction.java @@ -66,13 +66,12 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; import ca.nrc.cadc.ac.MemberNotFoundException; import ca.nrc.cadc.ac.User; import ca.nrc.cadc.ac.server.GroupPersistence; -import ca.nrc.cadc.ac.server.UserPersistence; import ca.nrc.cadc.auth.AuthenticationUtil; import java.security.Principal; import java.util.ArrayList; diff --git a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/users/UsersAction.java b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/users/UsersAction.java index ce700e16..df6880b9 100644 --- a/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/users/UsersAction.java +++ b/projects/cadcAccessControl-Server/src/ca/nrc/cadc/ac/server/web/users/UsersAction.java @@ -68,6 +68,17 @@ */ package ca.nrc.cadc.ac.server.web.users; +import ca.nrc.cadc.ac.PersonalDetails; +import ca.nrc.cadc.ac.User; +import ca.nrc.cadc.ac.UserNotFoundException; +import ca.nrc.cadc.ac.UserRequest; +import ca.nrc.cadc.ac.server.PluginFactory; +import ca.nrc.cadc.ac.server.UserPersistence; +import ca.nrc.cadc.net.TransientException; +import org.apache.log4j.Logger; + +import javax.security.auth.Subject; +import javax.servlet.http.HttpServletResponse; import java.io.IOException; import java.io.InputStream; import java.io.Writer; @@ -77,19 +88,6 @@ import java.security.PrivilegedActionException; import java.security.PrivilegedExceptionAction; import java.util.Map; -import javax.security.auth.Subject; -import javax.servlet.http.HttpServletResponse; - -import ca.nrc.cadc.ac.PersonalDetails; -import ca.nrc.cadc.ac.User; -import ca.nrc.cadc.ac.UserRequest; -import org.apache.log4j.Logger; - -import ca.nrc.cadc.ac.UserNotFoundException; -import ca.nrc.cadc.ac.server.PluginFactory; -import ca.nrc.cadc.ac.server.UserPersistence; -import ca.nrc.cadc.net.TransientException; - public abstract class UsersAction implements PrivilegedExceptionAction<Object> { diff --git a/projects/cadcAccessControl-Server/test/LdapConfig.test.properties b/projects/cadcAccessControl-Server/test/LdapConfig.test.properties index 0612a33b..99ee0cc8 100644 --- a/projects/cadcAccessControl-Server/test/LdapConfig.test.properties +++ b/projects/cadcAccessControl-Server/test/LdapConfig.test.properties @@ -5,7 +5,7 @@ # scp gimli2.cadc.dao.nrc.ca:~miscsw/ca.crt /tmp/ca.crt # ${JAVA_HOME}/bin/keytool -importcert -keystore ${JAVA_HOME}/jre/lib/security/cacerts -file /tmp/ca.crt server = proc5-03.cadc.dao.nrc.ca -port = 636 +port = 389 proxyUser = testproxy usersDn = ou=Users,ou=ds,dc=testcanfar userRequestsDN = ou=UserRequests,ou=ds,dc=testcanfar diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/RequestValidatorTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/RequestValidatorTest.java index 94760917..e54ea3da 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/RequestValidatorTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/RequestValidatorTest.java @@ -69,11 +69,11 @@ package ca.nrc.cadc.ac.server; import ca.nrc.cadc.ac.Role; -import ca.nrc.cadc.ac.server.web.AddUserMemberActionTest; +import ca.nrc.cadc.ac.server.web.groups.AddUserMemberActionTest; import ca.nrc.cadc.auth.AuthenticationUtil; import ca.nrc.cadc.util.Log4jInit; import ca.nrc.cadc.uws.Parameter; -import java.security.Principal; + import java.util.ArrayList; import java.util.List; import org.apache.log4j.Level; diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/ldap/LdapUserDAOTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/ldap/LdapUserDAOTest.java index fb0f5332..0e592c80 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/ldap/LdapUserDAOTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/ldap/LdapUserDAOTest.java @@ -69,7 +69,6 @@ package ca.nrc.cadc.ac.server.ldap; import ca.nrc.cadc.ac.PersonalDetails; -import ca.nrc.cadc.ac.PosixDetails; import ca.nrc.cadc.ac.User; import ca.nrc.cadc.ac.UserDetails; import ca.nrc.cadc.ac.UserRequest; @@ -132,7 +131,7 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest /** * Test of addUser method, of class LdapUserDAO. */ -// @Test + @Test public void testAddUser() throws Exception { final User<HttpPrincipal> expected = new User<HttpPrincipal>(new HttpPrincipal(getUserID())); @@ -152,7 +151,7 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest /** * Test of getUser method, of class LdapUserDAO. */ -// @Test + @Test public void testGetUser() throws Exception { Subject subject = new Subject(); @@ -167,8 +166,6 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest try { User<X500Principal> actual = getUserDAO().getUser(testUser.getUserID()); - User<X500Principal> actual = getUserDAO() - .getUser(testUser.getUserID()); check(testUser, actual); return null; @@ -185,7 +182,7 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest /** * Test of getUserGroups method, of class LdapUserDAO. */ -// @Test + @Test public void testGetUserGroups() throws Exception { Subject subject = new Subject(); @@ -198,11 +195,7 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest { try { - Collection<DN> groups = getUserDAO().getUserGroups(testUser.getUserID(), false); - assertNotNull(groups); - assertTrue(!groups.isEmpty()); - Collection<DN> groups = - getUserDAO().getUserGroups(testUser.getUserID(), + Collection<DN> groups = getUserDAO().getUserGroups(testUser.getUserID(), false); assertNotNull("Groups should not be null.", groups); @@ -211,9 +204,6 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest log.debug(groupDN); } - groups = getUserDAO().getUserGroups(testUser.getUserID(), true); - assertNotNull(groups); - assertTrue(!groups.isEmpty()); groups = getUserDAO().getUserGroups(testUser.getUserID(), true); assertNotNull("Groups should not be null.", groups); @@ -235,7 +225,7 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest /** * Test of getUserGroups method, of class LdapUserDAO. */ -// @Test + @Test public void testIsMember() throws Exception { Subject subject = new Subject(); @@ -249,16 +239,9 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest try { boolean isMember = getUserDAO().isMember(testUser.getUserID(), "foo"); - assertFalse(isMember); - boolean isMember = - getUserDAO().isMember(testUser.getUserID(), "foo"); assertFalse("Membership should not exist.", isMember); - String groupDN = "cn=cadcdaotestgroup1," + config.getGroupsDN(); - isMember = getUserDAO().isMember(testUser.getUserID(), groupDN); - assertTrue(isMember); - String groupDN = "cn=cadcdaotestgroup1," - + config.getGroupsDN(); + String groupDN = "cn=cadcdaotestgroup1," + config.getGroupsDN(); isMember = getUserDAO().isMember(testUser.getUserID(), groupDN); assertTrue("Membership should exist.", isMember); @@ -276,7 +259,7 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest /** * Test of getMember. */ -// @Test + @Test public void testGetMember() throws Exception { Subject subject = new Subject(); @@ -290,8 +273,6 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest try { User<X500Principal> actual = getUserDAO().getMember(new DN(testUserDN)); - User<X500Principal> actual = getUserDAO() - .getMember(new DN(testUserDN)); check(testUser, actual); return null; } @@ -314,12 +295,7 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest { try { -<<<<<<< HEAD User<X500Principal> actual = getUserDAO().getMember(new DN(testUserDN)); -======= - User<X500Principal> actual = getUserDAO() - .getMember(new DN(testUserDN)); ->>>>>>> 57bf534cb0ace85be32da449ca35a73e96a379b2 check(testUser, actual); return null; } @@ -334,14 +310,12 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest /** * Test of testGetCadcUserIDs. */ -// @Test + @Test public void testGetCadcUserIDs() throws Exception { Subject subject = new Subject(); - // anonymous access - int users1 = (Integer)Subject.doAs(subject, new PrivilegedExceptionAction<Object>() int users1 = (Integer) Subject .doAs(subject, new PrivilegedExceptionAction<Object>() { @@ -363,7 +337,6 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest // authenticated access subject.getPrincipals().add(testUser.getUserID()); - int users2 = (Integer)Subject.doAs(subject, new PrivilegedExceptionAction<Object>() int users2 = (Integer) Subject .doAs(subject, new PrivilegedExceptionAction<Object>() { @@ -383,11 +356,10 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest } }); assertEquals("User listing should be independent of the access type", -<<<<<<< HEAD users1, users2); } - @Test +// @Test public void testSetPassword() throws Exception { // LDAPConnection connection = @@ -507,7 +479,7 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest } -// @Test + @Test public void testUpdateUser() throws Exception { // Create a test user @@ -596,9 +568,6 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest }); assertNotNull(updatedUser); check(testUser2, updatedUser); -======= - users1, users2); ->>>>>>> 57bf534cb0ace85be32da449ca35a73e96a379b2 } private static void check(final User<? extends Principal> user1, final User<? extends Principal> user2) @@ -630,24 +599,6 @@ public class LdapUserDAOTest<T extends Principal> extends AbstractLdapDAOTest assertTrue(found); } } - if (d1 instanceof PosixDetails) - { - PosixDetails pd1 = (PosixDetails) d1; - boolean found = false; - for(UserDetails d2 : user2.details) - { - if(d2 instanceof PosixDetails) - { - PosixDetails pd2 = (PosixDetails) d2; - assertEquals(pd1, pd2); - assertEquals(pd1.getUid(), pd2.getUid()); - assertEquals(pd1.getGid(), pd2.getGid()); - assertEquals(pd1.getHomeDirectory(), pd2.getHomeDirectory()); - found = true; - } - assertTrue(found); - } - } } } diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/AddGroupMemberActionTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/AddGroupMemberActionTest.java similarity index 99% rename from projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/AddGroupMemberActionTest.java rename to projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/AddGroupMemberActionTest.java index eb092c7f..8550e9ba 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/AddGroupMemberActionTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/AddGroupMemberActionTest.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; import ca.nrc.cadc.ac.GroupAlreadyExistsException; diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/AddUserMemberActionTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/AddUserMemberActionTest.java similarity index 98% rename from projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/AddUserMemberActionTest.java rename to projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/AddUserMemberActionTest.java index 6488fd76..1cdb6f4f 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/AddUserMemberActionTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/AddUserMemberActionTest.java @@ -66,10 +66,9 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; -import ca.nrc.cadc.ac.GroupAlreadyExistsException; import ca.nrc.cadc.ac.MemberAlreadyExistsException; import ca.nrc.cadc.ac.User; import ca.nrc.cadc.ac.server.GroupPersistence; @@ -77,7 +76,7 @@ import ca.nrc.cadc.ac.server.UserPersistence; import ca.nrc.cadc.auth.AuthenticationUtil; import ca.nrc.cadc.util.Log4jInit; import java.security.Principal; -import javax.security.auth.x500.X500Principal; + import org.apache.log4j.Level; import org.apache.log4j.Logger; import org.easymock.EasyMock; diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/DeleteGroupActionTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/DeleteGroupActionTest.java similarity index 95% rename from projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/DeleteGroupActionTest.java rename to projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/DeleteGroupActionTest.java index 4562671a..09946102 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/DeleteGroupActionTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/DeleteGroupActionTest.java @@ -66,22 +66,15 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; -import ca.nrc.cadc.ac.MemberAlreadyExistsException; -import ca.nrc.cadc.ac.User; import ca.nrc.cadc.ac.server.GroupPersistence; -import ca.nrc.cadc.ac.server.UserPersistence; -import ca.nrc.cadc.auth.AuthenticationUtil; import ca.nrc.cadc.util.Log4jInit; import java.security.Principal; import org.apache.log4j.Level; import org.apache.log4j.Logger; import org.easymock.EasyMock; -import org.junit.After; -import org.junit.AfterClass; -import org.junit.Before; import org.junit.BeforeClass; import org.junit.Test; import static org.junit.Assert.*; diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GetGroupNamesActionTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GetGroupNamesActionTest.java similarity index 99% rename from projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GetGroupNamesActionTest.java rename to projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GetGroupNamesActionTest.java index 0cb59383..1f2d4d67 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GetGroupNamesActionTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GetGroupNamesActionTest.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.server.GroupPersistence; import ca.nrc.cadc.util.Log4jInit; diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GroupActionFactoryTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GroupActionFactoryTest.java similarity index 99% rename from projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GroupActionFactoryTest.java rename to projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GroupActionFactoryTest.java index f62d030c..3a9b19cc 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GroupActionFactoryTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GroupActionFactoryTest.java @@ -65,7 +65,7 @@ ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import javax.servlet.http.HttpServletRequest; @@ -77,8 +77,6 @@ import org.junit.Test; import ca.nrc.cadc.util.Log4jInit; -import java.net.URL; - public class GroupActionFactoryTest { private final static Logger log = Logger.getLogger(GroupActionFactoryTest.class); diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GroupsActionTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GroupsActionTest.java similarity index 98% rename from projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GroupsActionTest.java rename to projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GroupsActionTest.java index 08b28e97..def88525 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/GroupsActionTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/GroupsActionTest.java @@ -66,7 +66,7 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.GroupAlreadyExistsException; import ca.nrc.cadc.ac.GroupNotFoundException; @@ -76,18 +76,14 @@ import ca.nrc.cadc.ac.UserNotFoundException; import ca.nrc.cadc.net.TransientException; import ca.nrc.cadc.util.Log4jInit; -import java.io.ByteArrayOutputStream; -import java.io.OutputStream; import java.io.PrintWriter; import java.io.StringWriter; import java.security.AccessControlException; -import javax.servlet.ServletOutputStream; import javax.servlet.http.HttpServletResponse; import org.apache.log4j.Level; import org.apache.log4j.Logger; import org.easymock.EasyMock; import org.junit.BeforeClass; -import org.junit.Ignore; import org.junit.Test; import static org.junit.Assert.*; diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/RemoveGroupMemberActionTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/RemoveGroupMemberActionTest.java similarity index 97% rename from projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/RemoveGroupMemberActionTest.java rename to projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/RemoveGroupMemberActionTest.java index 7ccc5abb..749d0076 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/RemoveGroupMemberActionTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/RemoveGroupMemberActionTest.java @@ -66,10 +66,9 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; -import ca.nrc.cadc.ac.GroupAlreadyExistsException; import ca.nrc.cadc.ac.GroupNotFoundException; import ca.nrc.cadc.ac.server.GroupPersistence; import ca.nrc.cadc.util.Log4jInit; @@ -77,9 +76,6 @@ import java.security.Principal; import org.apache.log4j.Level; import org.apache.log4j.Logger; import org.easymock.EasyMock; -import org.junit.After; -import org.junit.AfterClass; -import org.junit.Before; import org.junit.BeforeClass; import org.junit.Test; import static org.junit.Assert.*; diff --git a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/RemoveUserMemberActionTest.java b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/RemoveUserMemberActionTest.java similarity index 98% rename from projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/RemoveUserMemberActionTest.java rename to projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/RemoveUserMemberActionTest.java index fea4de56..05609091 100644 --- a/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/RemoveUserMemberActionTest.java +++ b/projects/cadcAccessControl-Server/test/src/ca/nrc/cadc/ac/server/web/groups/RemoveUserMemberActionTest.java @@ -66,10 +66,9 @@ * ************************************************************************ */ -package ca.nrc.cadc.ac.server.web; +package ca.nrc.cadc.ac.server.web.groups; import ca.nrc.cadc.ac.Group; -import ca.nrc.cadc.ac.MemberAlreadyExistsException; import ca.nrc.cadc.ac.MemberNotFoundException; import ca.nrc.cadc.ac.User; import ca.nrc.cadc.ac.server.GroupPersistence; @@ -80,9 +79,6 @@ import java.security.Principal; import org.apache.log4j.Level; import org.apache.log4j.Logger; import org.easymock.EasyMock; -import org.junit.After; -import org.junit.AfterClass; -import org.junit.Before; import org.junit.BeforeClass; import org.junit.Test; import static org.junit.Assert.*; -- GitLab