mirror of
https://github.com/Alfresco/alfresco-community-repo.git
synced 2025-07-07 18:25:23 +00:00
19472: ALF-725: Revert to using jTDS JDBC driver for SQL Server in 3.2 SP1, since the Microsoft driver doesn't work with the v3.2.r iBATIS stuff - All example/installer alfresco-global.properties updated - Wiki updated http://wiki.alfresco.com/wiki/Database_Configuration#MS-SQL_Databases - Logged doc bug ALF-2144 and release note bug ALF-2145 19501:Merged DEV/BELARUS/V3.2-2010_02_24 to V3.2 (with corrections) 19243: ALF-757: Cannot start up on JBoss 5.1 due to audit configuration error - Removed getPath() method because it is incompatible with JBoss and other app servers where resources can't be resolved to a file - Now use Spring ResourceLoader instead of creating FileInputStream - getLastModified() still returned where the resource resolves to a file; otherwise the server startup time 19503: (RECORD ONLY) ALF-2100: Merged HEAD to V3.2 19155: ALF-1995: Removed remaining direct dependencies on portlet API from Alfresco Explorer classes - Moved into AlfrescoFacesPortlet - portlet.jar was removed from alfresco.war for Liferay compatibility 19506: Merged PATCHES/V3.1.2 to V3.2 19218: (RECORD ONLY) Created hotfix branch off TAGS/ENTERPRISE/V3.1.2 19229: (RECORD ONLY) Merged V3.1 to V3.1.2 18577: Fix for ETHREEOH-4117, based on CHK-11154 19341: Merged DEV/BELARUS/V3.1-2010_02_05 to PATCHES/V3.1.2 (with corrections) 19156: ALF-1906: splitPersonCleanUpBootstrapBean is not able to remove duplicated users Also - improved detection of 'split' persons - added unit tests for person splitting and deleting - fixed duplicate person caching and sorting problems - prevented onUpdateProperties from firing needlessly in PersonServiceImpl and AuthorityDAOImpl when persons and authorities are created initially 19342: (RECORD ONLY) Incremented version number 19508: Merged PATCHES/V3.2.0 to V3.2 18762: (RECORD ONLY) Created hotfix branch off V3.2.0-ENTERPRISE-FINAL 18789: (RECORD ONLY) Merged BRANCHES/V3.2:r17905,18254,18319 to PATCHES/V3.2.0 r17905 | markr | 2010-01-06 16:55:12 +0000 (Wed, 06 Jan 2010) | 3 lines ETHREEOH-3809 - WCM - First test server deploy fails. added yet another transaction to read the previous snapshot transaction. added a new system test based upon the WCM services. The beginnings of testing against layered authored sandboxes. r18254 | janv | 2010-01-22 18:15:43 +0000 (Fri, 22 Jan 2010) | 1 line WCM/AVM - ETHREEOH-2057 (Submitting WCM Content through WF JSF Error - due to AVM Sync issue) r18319 | royw | 2010-01-27 12:18:27 +0000 (Wed, 27 Jan 2010) | 4 lines Merged BRANCHES/DEV/BELARUS/V3.2-2010_01_11 to V3.2 18273: ETHREEOH-3834: WCM: An extral .xml.html file is created when editing newly created content 18822: (RECORD ONLY) Merged DEV_TEMPORARY to PATCHES/V3.2.0 18478: SAP XForms errors - ACT 15969 18699: ETHREEOH-4171: HTTP 500 when filling in a WCM webform - ACT 15969 18842: (RECORD ONLY) Merged V3.2 to PATCHES/V3.2.0 18701: Merged DEV_TEMPORARY to V3.2 18693 : ETHREEOH-4182: ASR deployer fails to set the contentUrl of documents on the target system - Merged in fix related to closing output streams. - Increased coverage of unit test. 18854: (RECORD ONLY) Merged V3.2 to V3.2.0 18019: ETHREEOH-3770: LDAP sync now supports attribute range retrieval to get around limits imposed by Active Directory on multi-valued attributes - Meant that groups with more than 1000 members were getting truncated in Active Directory - Now switched on in ldap-ad and off in ldap subsystem - Also switched off result set paging in ldap subsystem by default for wider compatibility with non-AD systems 18272: Merged DEV/BELARUS/V3.2-2010_01_11 to V3.2 18257: ETHREEOH-4002: User/Group sync does not handle LDAP communication failures - Merged with corrections 18276: ETHREEOH-4002: Correction to previous checkin - modification dates are only persisted after successful processing of users and groups, so need to delete them on comms failure 18340: ETHREEOH-4069: LDAP sync cannot resolve DNs containing a slash character - Due to JNDI interpreting the slash character as a separator 18403: ETHREEOH-4008: LDAP sync should preserve case of group members - Was incorrectly extracting attributes from lower-cased DN 18846: ETHREEOH-4233: LDAP sync now synchronizes group display names - New ldap.synchronization.groupDisplayNameAttributeName property provides name of LDAP attribute 18877: (RECORD ONLY) Merged /alfresco/BRANCHES/V3.2:r18616 r18616 | markr | 2010-02-12 14:08:52 +0000 (Fri, 12 Feb 2010) | 1 line ETHREEOH-4181 - Access denied exception when deploying via avm deployment receiver 19319: ALF-2043: User ID case sensitivity issues with Sharepoint Connector and External Authentication Subsystem - DefaultRemoteUserMapper and AlfrescoUserGroupServiceHandler should use personService.getUserIdentifier() to 'normalize' a username according to case sensitivity settings - NtlmAuthenticationHandler should also leave the normalization to personService 19320: (RECORD ONLY) Incremented version label 19380: ALF-2043: Revisit user ID case sensitivity in DefaultRemoteUserMapper - Has to use public PersonService in case it is accessed outside of a transaction - Fixed regular expression matching - Added unit tests to try out all the remote user mapper options 19509: Merged PATCHES/V3.2.r to V3.2 18803: (RECORD ONLY) Created hotfix branch off V3.2.r-ENTERPRISE-FINAL 18833: (RECORD ONLY) Turn on Repo Doclib by default 19054: (RECORD ONLY) Merging V3.2 to PATCHES/V3.2.r 18787: MT: fix ETHREEOH-4125 - authority migration / batch processor (when upgrading groups from 3.1 to 3.2) 19358: (RECORD ONLY) Merged DEV/BELARUS/V3.2-2010_01_11 to PATCHES/V3.2.r 18699: ETHREEOH-4171: HTTP 500 when filling in a WCM webform 19447: (RECORD ONLY) Incremented version label 19518: ALF-757: Corrected audit config resource URL so that it resolves inside Tomcat as well as JUnit! 19525: ALF-708: Use BatchProcessor to process duplicate persons in small batches in SplitPersonCleanupBootstrapBean - Even tested in a unit test! git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@19536 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
824 lines
34 KiB
Java
824 lines
34 KiB
Java
/*
|
|
* Copyright (C) 2005-2010 Alfresco Software Limited.
|
|
*
|
|
* This file is part of Alfresco
|
|
*
|
|
* Alfresco is free software: you can redistribute it and/or modify
|
|
* it under the terms of the GNU Lesser General Public License as published by
|
|
* the Free Software Foundation, either version 3 of the License, or
|
|
* (at your option) any later version.
|
|
*
|
|
* Alfresco is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU Lesser General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Lesser General Public License
|
|
* along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
|
|
*/
|
|
package org.alfresco.repo.security.person;
|
|
|
|
import java.io.Serializable;
|
|
import java.util.HashMap;
|
|
import java.util.HashSet;
|
|
import java.util.List;
|
|
import java.util.Map;
|
|
import java.util.Set;
|
|
import java.util.concurrent.ConcurrentHashMap;
|
|
import java.util.concurrent.CountDownLatch;
|
|
import java.util.concurrent.TimeUnit;
|
|
|
|
import junit.framework.Assert;
|
|
|
|
import org.alfresco.model.ContentModel;
|
|
import org.alfresco.repo.transaction.RetryingTransactionHelper;
|
|
import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback;
|
|
import org.alfresco.service.cmr.repository.ChildAssociationRef;
|
|
import org.alfresco.service.cmr.repository.NodeRef;
|
|
import org.alfresco.service.cmr.repository.NodeService;
|
|
import org.alfresco.service.cmr.repository.StoreRef;
|
|
import org.alfresco.service.cmr.repository.datatype.DefaultTypeConverter;
|
|
import org.alfresco.service.cmr.security.AuthorityService;
|
|
import org.alfresco.service.cmr.security.NoSuchPersonException;
|
|
import org.alfresco.service.cmr.security.PermissionService;
|
|
import org.alfresco.service.cmr.security.PersonService;
|
|
import org.alfresco.service.namespace.QName;
|
|
import org.alfresco.service.transaction.TransactionService;
|
|
import org.alfresco.util.BaseSpringTest;
|
|
import org.alfresco.util.EqualsHelper;
|
|
import org.alfresco.util.GUID;
|
|
import org.alfresco.util.PropertyMap;
|
|
|
|
public class PersonTest extends BaseSpringTest
|
|
{
|
|
private TransactionService transactionService;
|
|
|
|
private PersonService personService;
|
|
|
|
private NodeService nodeService;
|
|
|
|
private NodeRef rootNodeRef;
|
|
|
|
private PermissionService permissionService;
|
|
|
|
private AuthorityService authorityService;
|
|
|
|
public PersonTest()
|
|
{
|
|
super();
|
|
// TODO Auto-generated constructor stub
|
|
}
|
|
|
|
protected void onSetUpInTransaction() throws Exception
|
|
{
|
|
transactionService = (TransactionService) applicationContext.getBean("transactionService");
|
|
personService = (PersonService) applicationContext.getBean("personService");
|
|
nodeService = (NodeService) applicationContext.getBean("nodeService");
|
|
permissionService = (PermissionService) applicationContext.getBean("permissionService");
|
|
authorityService = (AuthorityService) applicationContext.getBean("authorityService");
|
|
|
|
StoreRef storeRef = nodeService.createStore(StoreRef.PROTOCOL_WORKSPACE, "Test_" + System.currentTimeMillis());
|
|
rootNodeRef = nodeService.getRootNode(storeRef);
|
|
|
|
for (NodeRef nodeRef : personService.getAllPeople())
|
|
{
|
|
String uid = DefaultTypeConverter.INSTANCE.convert(String.class, nodeService.getProperty(nodeRef, ContentModel.PROP_USERNAME));
|
|
if (!uid.equals("admin"))
|
|
{
|
|
nodeService.deleteNode(nodeRef);
|
|
}
|
|
}
|
|
|
|
personService.setCreateMissingPeople(true);
|
|
}
|
|
|
|
protected void onTearDownInTransaction() throws Exception
|
|
{
|
|
super.onTearDownInTransaction();
|
|
}
|
|
|
|
public void xtestLazyHomeFolderCreation()
|
|
{
|
|
String firstName = "" + System.currentTimeMillis();
|
|
String lastName = String.format("%05d", -1);
|
|
final String username = GUID.generate();
|
|
String emailAddress = String.format("%s.%s@xyz.com", firstName, lastName);
|
|
PropertyMap properties = new PropertyMap(7);
|
|
properties.put(ContentModel.PROP_USERNAME, username);
|
|
properties.put(ContentModel.PROP_FIRSTNAME, firstName);
|
|
properties.put(ContentModel.PROP_LASTNAME, lastName);
|
|
properties.put(ContentModel.PROP_EMAIL, emailAddress);
|
|
final NodeRef madePerson = personService.createPerson(properties);
|
|
|
|
NodeRef homeFolder = DefaultTypeConverter.INSTANCE.convert(NodeRef.class, nodeService.getProperty(madePerson, ContentModel.PROP_HOMEFOLDER));
|
|
if (homeFolder != null)
|
|
{
|
|
throw new IllegalStateException("Home folder created eagerly");
|
|
}
|
|
|
|
setComplete();
|
|
endTransaction();
|
|
startNewTransaction();
|
|
|
|
RetryingTransactionHelper helper = transactionService.getRetryingTransactionHelper();
|
|
helper.doInTransaction(new RetryingTransactionCallback<Void>()
|
|
{
|
|
public Void execute() throws Throwable
|
|
{
|
|
assertTrue(personService.personExists(username));
|
|
NodeRef person = personService.getPerson(username);
|
|
assertEquals(madePerson, person);
|
|
NodeRef homeFolder = DefaultTypeConverter.INSTANCE.convert(NodeRef.class, nodeService.getProperty(madePerson, ContentModel.PROP_HOMEFOLDER));
|
|
if (homeFolder == null)
|
|
{
|
|
throw new IllegalStateException("Home folder not created lazily");
|
|
}
|
|
return null;
|
|
}
|
|
}, true, false);
|
|
|
|
homeFolder = DefaultTypeConverter.INSTANCE.convert(NodeRef.class, nodeService.getProperty(madePerson, ContentModel.PROP_HOMEFOLDER));
|
|
if (homeFolder == null)
|
|
{
|
|
throw new IllegalStateException("Home folder not created lazily");
|
|
}
|
|
}
|
|
|
|
public void testZones()
|
|
{
|
|
assertNull(authorityService.getAuthorityZones("derek"));
|
|
assertNull(authorityService.getAuthorityZones("null"));
|
|
|
|
personService.createPerson(createDefaultProperties("derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
Set<String> zones = authorityService.getAuthorityZones("derek");
|
|
assertEquals(2, zones.size());
|
|
authorityService.removeAuthorityFromZones("derek", zones);
|
|
assertEquals(0, authorityService.getAuthorityZones("derek").size());
|
|
authorityService.addAuthorityToZones("derek", zones);
|
|
assertEquals(2, authorityService.getAuthorityZones("derek").size());
|
|
|
|
HashSet<String> newZones = null;
|
|
personService.createPerson(createDefaultProperties("null", "null", "null", "null", "null", rootNodeRef), newZones);
|
|
assertEquals(0, authorityService.getAuthorityZones("null").size());
|
|
|
|
newZones = new HashSet<String>();
|
|
personService.createPerson(createDefaultProperties("empty", "empty", "empty", "empty", "empty", rootNodeRef), newZones);
|
|
assertEquals(0, authorityService.getAuthorityZones("empty").size());
|
|
|
|
newZones.add("One");
|
|
personService.createPerson(createDefaultProperties("1", "1", "1", "1", "1", rootNodeRef), newZones);
|
|
assertEquals(1, authorityService.getAuthorityZones("1").size());
|
|
|
|
newZones.add("Two");
|
|
personService.createPerson(createDefaultProperties("2", "2", "2", "2", "2", rootNodeRef), newZones);
|
|
assertEquals(2, authorityService.getAuthorityZones("2").size());
|
|
|
|
newZones.add("Three");
|
|
personService.createPerson(createDefaultProperties("3", "3", "3", "3", "3", rootNodeRef), newZones);
|
|
assertEquals(3, authorityService.getAuthorityZones("3").size());
|
|
|
|
HashSet<String> toRemove = null;
|
|
authorityService.removeAuthorityFromZones("3", toRemove);
|
|
assertEquals(3, authorityService.getAuthorityZones("3").size());
|
|
|
|
toRemove = new HashSet<String>();
|
|
authorityService.removeAuthorityFromZones("3", toRemove);
|
|
assertEquals(3, authorityService.getAuthorityZones("3").size());
|
|
|
|
toRemove.add("Three");
|
|
authorityService.removeAuthorityFromZones("3", toRemove);
|
|
assertEquals(2, authorityService.getAuthorityZones("3").size());
|
|
|
|
toRemove.add("Two");
|
|
authorityService.removeAuthorityFromZones("3", toRemove);
|
|
assertEquals(1, authorityService.getAuthorityZones("3").size());
|
|
|
|
toRemove.add("One");
|
|
authorityService.removeAuthorityFromZones("3", toRemove);
|
|
assertEquals(0, authorityService.getAuthorityZones("3").size());
|
|
|
|
authorityService.addAuthorityToZones("3", newZones);
|
|
assertEquals(3, authorityService.getAuthorityZones("3").size());
|
|
assertEquals(3, authorityService.getAllAuthoritiesInZone("One", null).size());
|
|
assertEquals(2, authorityService.getAllAuthoritiesInZone("Two", null).size());
|
|
assertEquals(1, authorityService.getAllAuthoritiesInZone("Three", null).size());
|
|
|
|
}
|
|
|
|
public void xtestPerformance()
|
|
{
|
|
personService.setCreateMissingPeople(false);
|
|
|
|
personService.createPerson(createDefaultProperties("derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
|
|
long create = 0;
|
|
|
|
long start;
|
|
long end;
|
|
|
|
for (int i = 0; i < 10000; i++)
|
|
{
|
|
String id = "TestUser-" + i;
|
|
start = System.nanoTime();
|
|
personService.createPerson(createDefaultProperties(id, id, id, id, id, rootNodeRef));
|
|
end = System.nanoTime();
|
|
create += (end - start);
|
|
|
|
if ((i > 0) && (i % 100 == 0))
|
|
{
|
|
System.out.println("Count = " + i);
|
|
System.out.println("Average create : " + (create / i / 1000000.0f));
|
|
start = System.nanoTime();
|
|
personService.personExists(id);
|
|
end = System.nanoTime();
|
|
System.out.println("Exists : " + ((end - start) / 1000000.0f));
|
|
|
|
start = System.nanoTime();
|
|
int size = personService.getAllPeople().size();
|
|
end = System.nanoTime();
|
|
System.out.println("Size (" + size + ") : " + ((end - start) / 1000000.0f));
|
|
}
|
|
}
|
|
}
|
|
|
|
public void testDeletePerson()
|
|
{
|
|
personService.getPerson("andy");
|
|
NodeRef n1 = nodeService.createNode(rootNodeRef, ContentModel.ASSOC_CHILDREN, QName.createQName("{namespace}one"), ContentModel.TYPE_FOLDER).getChildRef();
|
|
NodeRef n2 = nodeService.createNode(rootNodeRef, ContentModel.ASSOC_CHILDREN, QName.createQName("{namespace}two"), ContentModel.TYPE_FOLDER).getChildRef();
|
|
permissionService.setPermission(n1, "andy", PermissionService.READ, true);
|
|
permissionService.setPermission(n2, "andy", PermissionService.ALL_PERMISSIONS, true);
|
|
setComplete();
|
|
endTransaction();
|
|
startNewTransaction();
|
|
nodeService.deleteNode(n1);
|
|
setComplete();
|
|
endTransaction();
|
|
startNewTransaction();
|
|
personService.deletePerson("andy");
|
|
setComplete();
|
|
endTransaction();
|
|
startNewTransaction();
|
|
}
|
|
|
|
public void testCreateAndThenDelete()
|
|
{
|
|
personService.setCreateMissingPeople(false);
|
|
assertFalse(personService.createMissingPeople());
|
|
|
|
personService.setCreateMissingPeople(true);
|
|
assertTrue(personService.createMissingPeople());
|
|
|
|
personService.setCreateMissingPeople(false);
|
|
try
|
|
{
|
|
personService.getPerson("andy");
|
|
fail("Getting Andy should fail");
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
personService.createPerson(createDefaultProperties("andy", "Andy", "Hind", "andy@hind", "alfresco", rootNodeRef));
|
|
personService.getPerson("andy");
|
|
personService.deletePerson("andy");
|
|
try
|
|
{
|
|
personService.getPerson("andy");
|
|
fail("Getting Andy should fail");
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
}
|
|
|
|
public void testCreateMissingPeople1()
|
|
{
|
|
personService.setCreateMissingPeople(false);
|
|
assertFalse(personService.createMissingPeople());
|
|
|
|
personService.setCreateMissingPeople(true);
|
|
assertTrue(personService.createMissingPeople());
|
|
|
|
personService.setCreateMissingPeople(false);
|
|
try
|
|
{
|
|
personService.getPerson("andy");
|
|
fail("Getting Andy should fail");
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
}
|
|
|
|
public void testCreateMissingPeople2()
|
|
{
|
|
System.out.print(personService.getAllPeople());
|
|
|
|
personService.setCreateMissingPeople(false);
|
|
assertFalse(personService.createMissingPeople());
|
|
|
|
personService.setCreateMissingPeople(true);
|
|
assertTrue(personService.createMissingPeople());
|
|
|
|
NodeRef nodeRef = personService.getPerson("andy");
|
|
assertNotNull(nodeRef);
|
|
testProperties(nodeRef, "andy", "andy", "", "", "");
|
|
|
|
nodeRef = personService.getPerson("andy");
|
|
testProperties(nodeRef, "andy", "andy", "", "", "");
|
|
|
|
nodeRef = personService.getPerson("Andy");
|
|
testProperties(nodeRef, "andy", "andy", "", "", "");
|
|
|
|
assertEquals(nodeRef, personService.getPerson("Andy"));
|
|
nodeRef = personService.getPerson("Andy");
|
|
assertNotNull(nodeRef);
|
|
if (personService.getUserIdentifier("Andy").equals("Andy"))
|
|
{
|
|
testProperties(nodeRef, "Andy", "Andy", "", "", "");
|
|
}
|
|
else
|
|
{
|
|
testProperties(nodeRef, "andy", "andy", "", "", "");
|
|
}
|
|
|
|
personService.setCreateMissingPeople(false);
|
|
try
|
|
{
|
|
personService.setPersonProperties("derek", createDefaultProperties("derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
fail("Getting Derek should fail");
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
}
|
|
|
|
public void testCreateMissingPeople()
|
|
{
|
|
personService.setCreateMissingPeople(false);
|
|
assertFalse(personService.createMissingPeople());
|
|
|
|
personService.setCreateMissingPeople(true);
|
|
assertTrue(personService.createMissingPeople());
|
|
|
|
NodeRef nodeRef = personService.getPerson("andy");
|
|
assertNotNull(nodeRef);
|
|
testProperties(nodeRef, "andy", "andy", "", "", "");
|
|
|
|
personService.setCreateMissingPeople(true);
|
|
personService.setPersonProperties("derek", createDefaultProperties("derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
testProperties(personService.getPerson("derek"), "derek", "Derek", "Hulley", "dh@dh", "alfresco");
|
|
|
|
testProperties(personService.getPerson("andy"), "andy", "andy", "", "", "");
|
|
|
|
assertEquals(3, personService.getAllPeople().size());
|
|
assertTrue(personService.getAllPeople().contains(personService.getPerson("andy")));
|
|
assertTrue(personService.getAllPeople().contains(personService.getPerson("derek")));
|
|
|
|
}
|
|
|
|
public void testMutableProperties()
|
|
{
|
|
assertEquals(5, personService.getMutableProperties().size());
|
|
assertTrue(personService.getMutableProperties().contains(ContentModel.PROP_HOMEFOLDER));
|
|
assertTrue(personService.getMutableProperties().contains(ContentModel.PROP_FIRSTNAME));
|
|
assertTrue(personService.getMutableProperties().contains(ContentModel.PROP_LASTNAME));
|
|
assertTrue(personService.getMutableProperties().contains(ContentModel.PROP_EMAIL));
|
|
assertTrue(personService.getMutableProperties().contains(ContentModel.PROP_ORGID));
|
|
|
|
}
|
|
|
|
public void testPersonCRUD1()
|
|
{
|
|
personService.setCreateMissingPeople(false);
|
|
try
|
|
{
|
|
personService.getPerson("derek");
|
|
fail("Getting Derek should fail");
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
}
|
|
|
|
public void testPersonCRUD2()
|
|
{
|
|
personService.setCreateMissingPeople(false);
|
|
personService.createPerson(createDefaultProperties("derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
testProperties(personService.getPerson("derek"), "derek", "Derek", "Hulley", "dh@dh", "alfresco");
|
|
|
|
personService.setPersonProperties("derek", createDefaultProperties("derek", "Derek_", "Hulley_", "dh@dh_", "alfresco_", rootNodeRef));
|
|
|
|
testProperties(personService.getPerson("derek"), "derek", "Derek_", "Hulley_", "dh@dh_", "alfresco_");
|
|
|
|
personService.setPersonProperties("derek", createDefaultProperties("derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
|
|
testProperties(personService.getPerson("derek"), "derek", "Derek", "Hulley", "dh@dh", "alfresco");
|
|
|
|
assertEquals(2, personService.getAllPeople().size());
|
|
assertTrue(personService.getAllPeople().contains(personService.getPerson("derek")));
|
|
assertEquals(1, personService.getPeopleFilteredByProperty(ContentModel.PROP_USERNAME, "derek").size());
|
|
assertEquals(1, personService.getPeopleFilteredByProperty(ContentModel.PROP_EMAIL, "dh@dh").size());
|
|
assertEquals(1, personService.getPeopleFilteredByProperty(ContentModel.PROP_ORGID, "alfresco").size());
|
|
assertEquals(0, personService.getPeopleFilteredByProperty(ContentModel.PROP_USERNAME, "glen").size());
|
|
assertEquals(0, personService.getPeopleFilteredByProperty(ContentModel.PROP_EMAIL, "gj@email.com").size());
|
|
assertEquals(0, personService.getPeopleFilteredByProperty(ContentModel.PROP_ORGID, "microsoft").size());
|
|
|
|
personService.deletePerson("derek");
|
|
assertEquals(1, personService.getAllPeople().size());
|
|
try
|
|
{
|
|
personService.getPerson("derek");
|
|
fail("Getting Derek should fail");
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
}
|
|
|
|
public void testPersonCRUD()
|
|
{
|
|
personService.setCreateMissingPeople(false);
|
|
personService.createPerson(createDefaultProperties("Derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
testProperties(personService.getPerson("Derek"), "Derek", "Derek", "Hulley", "dh@dh", "alfresco");
|
|
|
|
personService.setPersonProperties("Derek", createDefaultProperties("derek", "Derek_", "Hulley_", "dh@dh_", "alfresco_", rootNodeRef));
|
|
|
|
testProperties(personService.getPerson("Derek"), "Derek", "Derek_", "Hulley_", "dh@dh_", "alfresco_");
|
|
|
|
personService.setPersonProperties("Derek", createDefaultProperties("derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
|
|
testProperties(personService.getPerson("Derek"), "Derek", "Derek", "Hulley", "dh@dh", "alfresco");
|
|
|
|
assertEquals(2, personService.getAllPeople().size());
|
|
assertTrue(personService.getAllPeople().contains(personService.getPerson("Derek")));
|
|
assertEquals(1, personService.getPeopleFilteredByProperty(ContentModel.PROP_USERNAME, "Derek").size());
|
|
assertEquals(1, personService.getPeopleFilteredByProperty(ContentModel.PROP_EMAIL, "dh@dh").size());
|
|
assertEquals(1, personService.getPeopleFilteredByProperty(ContentModel.PROP_ORGID, "alfresco").size());
|
|
assertEquals(0, personService.getPeopleFilteredByProperty(ContentModel.PROP_USERNAME, "Glen").size());
|
|
assertEquals(0, personService.getPeopleFilteredByProperty(ContentModel.PROP_EMAIL, "gj@email.com").size());
|
|
assertEquals(0, personService.getPeopleFilteredByProperty(ContentModel.PROP_ORGID, "microsoft").size());
|
|
assertEquals(personService.personExists("derek"), EqualsHelper.nullSafeEquals(personService.getUserIdentifier("derek"), "Derek"));
|
|
assertEquals(personService.personExists("dEREK"), EqualsHelper.nullSafeEquals(personService.getUserIdentifier("dEREK"), "Derek"));
|
|
assertEquals(personService.personExists("DEREK"), EqualsHelper.nullSafeEquals(personService.getUserIdentifier("DEREK"), "Derek"));
|
|
|
|
personService.deletePerson("Derek");
|
|
assertEquals(1, personService.getAllPeople().size());
|
|
|
|
}
|
|
|
|
private void testProperties(NodeRef nodeRef, String userName, String firstName, String lastName, String email, String orgId)
|
|
{
|
|
assertEquals(userName, DefaultTypeConverter.INSTANCE.convert(String.class, nodeService.getProperty(nodeRef, ContentModel.PROP_USERNAME)));
|
|
assertNotNull(nodeService.getProperty(nodeRef, ContentModel.PROP_HOMEFOLDER));
|
|
assertEquals(firstName, DefaultTypeConverter.INSTANCE.convert(String.class, nodeService.getProperty(nodeRef, ContentModel.PROP_FIRSTNAME)));
|
|
assertEquals(lastName, DefaultTypeConverter.INSTANCE.convert(String.class, nodeService.getProperty(nodeRef, ContentModel.PROP_LASTNAME)));
|
|
assertEquals(email, DefaultTypeConverter.INSTANCE.convert(String.class, nodeService.getProperty(nodeRef, ContentModel.PROP_EMAIL)));
|
|
assertEquals(orgId, DefaultTypeConverter.INSTANCE.convert(String.class, nodeService.getProperty(nodeRef, ContentModel.PROP_ORGID)));
|
|
}
|
|
|
|
private Map<QName, Serializable> createDefaultProperties(String userName, String firstName, String lastName, String email, String orgId, NodeRef home)
|
|
{
|
|
HashMap<QName, Serializable> properties = new HashMap<QName, Serializable>();
|
|
properties.put(ContentModel.PROP_USERNAME, userName);
|
|
properties.put(ContentModel.PROP_HOMEFOLDER, home);
|
|
properties.put(ContentModel.PROP_FIRSTNAME, firstName);
|
|
properties.put(ContentModel.PROP_LASTNAME, lastName);
|
|
properties.put(ContentModel.PROP_EMAIL, email);
|
|
properties.put(ContentModel.PROP_ORGID, orgId);
|
|
return properties;
|
|
}
|
|
|
|
public void testCaseSensitive()
|
|
{
|
|
|
|
personService.createPerson(createDefaultProperties("Derek", "Derek", "Hulley", "dh@dh", "alfresco", rootNodeRef));
|
|
|
|
try
|
|
{
|
|
NodeRef nodeRef = personService.getPerson("derek");
|
|
if (personService.getUserIdentifier("derek").equals("Derek"))
|
|
{
|
|
assertNotNull(nodeRef);
|
|
}
|
|
else
|
|
{
|
|
assertNotNull(null);
|
|
}
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
try
|
|
{
|
|
NodeRef nodeRef = personService.getPerson("deRek");
|
|
if (personService.getUserIdentifier("deRek").equals("Derek"))
|
|
{
|
|
assertNotNull(nodeRef);
|
|
}
|
|
else
|
|
{
|
|
assertNotNull(null);
|
|
}
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
try
|
|
{
|
|
|
|
NodeRef nodeRef = personService.getPerson("DEREK");
|
|
if (personService.getUserIdentifier("DEREK").equals("Derek"))
|
|
{
|
|
assertNotNull(nodeRef);
|
|
}
|
|
else
|
|
{
|
|
assertNotNull(null);
|
|
}
|
|
}
|
|
catch (PersonException pe)
|
|
{
|
|
|
|
}
|
|
personService.getPerson("Derek");
|
|
}
|
|
|
|
public void testReadOnlyTransactionHandling() throws Exception
|
|
{
|
|
// Kill the annoying Spring-managed txn
|
|
super.setComplete();
|
|
super.endTransaction();
|
|
|
|
boolean createMissingPeople = personService.createMissingPeople();
|
|
assertTrue("Default should be to create missing people", createMissingPeople);
|
|
|
|
final String username = "Derek";
|
|
// Make sure that the person is missing
|
|
RetryingTransactionCallback<Object> deletePersonWork = new RetryingTransactionCallback<Object>()
|
|
{
|
|
public Object execute() throws Throwable
|
|
{
|
|
personService.deletePerson(username);
|
|
return null;
|
|
}
|
|
};
|
|
transactionService.getRetryingTransactionHelper().doInTransaction(deletePersonWork, false, true);
|
|
// Make a read-only transaction and check that we get NoSuchPersonException
|
|
RetryingTransactionCallback<NodeRef> getMissingPersonWork = new RetryingTransactionCallback<NodeRef>()
|
|
{
|
|
public NodeRef execute() throws Throwable
|
|
{
|
|
return personService.getPerson(username);
|
|
}
|
|
};
|
|
try
|
|
{
|
|
transactionService.getRetryingTransactionHelper().doInTransaction(getMissingPersonWork, true, true);
|
|
fail("Expected auto-creation of person to fail gracefully");
|
|
}
|
|
catch (NoSuchPersonException e)
|
|
{
|
|
// Expected
|
|
}
|
|
// It should work in a write transaction, though
|
|
transactionService.getRetryingTransactionHelper().doInTransaction(getMissingPersonWork, false, true);
|
|
|
|
transactionService.getRetryingTransactionHelper().doInTransaction(deletePersonWork, false, true);
|
|
}
|
|
|
|
/**
|
|
* Disabled due to time constraints. This <i>does</i> highlight a problem, but one that won't manifest
|
|
* itself critically in the product.
|
|
*/
|
|
public void xtestSplitPersonCleanupManyTimes() throws Throwable
|
|
{
|
|
for (int i = 0; i < 100; i++) // Bump this number up to 1000 for 'real' testing
|
|
{
|
|
try
|
|
{
|
|
forceSplitPersonCleanup();
|
|
}
|
|
catch (Throwable e)
|
|
{
|
|
throw new RuntimeException("Failed on iteration " + i + " of forcing split person.", e);
|
|
}
|
|
}
|
|
}
|
|
|
|
private void forceSplitPersonCleanup() throws Exception
|
|
{
|
|
// Kill the annoying Spring-managed txn
|
|
super.setComplete();
|
|
super.endTransaction();
|
|
|
|
boolean createMissingPeople = personService.createMissingPeople();
|
|
assertTrue("Default should be to create missing people", createMissingPeople);
|
|
|
|
PersonServiceImpl personServiceImpl = (PersonServiceImpl) personService;
|
|
personServiceImpl.setDuplicateMode("LEAVE");
|
|
|
|
// The user to duplicate
|
|
final String duplicateUsername = GUID.generate();
|
|
// Make sure that the person is missing
|
|
RetryingTransactionCallback<Object> deletePersonWork = new RetryingTransactionCallback<Object>()
|
|
{
|
|
public Object execute() throws Throwable
|
|
{
|
|
personService.deletePerson(duplicateUsername);
|
|
return null;
|
|
}
|
|
};
|
|
transactionService.getRetryingTransactionHelper().doInTransaction(deletePersonWork, false, true);
|
|
// Fire off 10 threads to create the same person
|
|
int threadCount = 10;
|
|
final CountDownLatch startLatch = new CountDownLatch(threadCount);
|
|
final CountDownLatch endLatch = new CountDownLatch(threadCount);
|
|
final Map<String, NodeRef> cleanableNodeRefs = new ConcurrentHashMap<String, NodeRef>(17);
|
|
Runnable createPersonRunnable = new Runnable()
|
|
{
|
|
public void run()
|
|
{
|
|
final RetryingTransactionCallback<NodeRef> createPersonWork = new RetryingTransactionCallback<NodeRef>()
|
|
{
|
|
public NodeRef execute() throws Throwable
|
|
{
|
|
// Wait for the trigger to start
|
|
try
|
|
{
|
|
startLatch.await();
|
|
}
|
|
catch (InterruptedException e)
|
|
{
|
|
}
|
|
|
|
// Trigger
|
|
NodeRef personNodeRef = personService.getPerson(duplicateUsername);
|
|
return personNodeRef;
|
|
}
|
|
};
|
|
startLatch.countDown();
|
|
try
|
|
{
|
|
NodeRef nodeRef = transactionService.getRetryingTransactionHelper().doInTransaction(createPersonWork, false, true);
|
|
// Store the noderef for later checking
|
|
String threadName = Thread.currentThread().getName();
|
|
cleanableNodeRefs.put(threadName, nodeRef);
|
|
}
|
|
catch (Throwable e)
|
|
{
|
|
// Errrm
|
|
e.printStackTrace();
|
|
}
|
|
endLatch.countDown();
|
|
}
|
|
};
|
|
// Fire the threads
|
|
for (int i = 0; i < threadCount; i++)
|
|
{
|
|
Thread thread = new Thread(createPersonRunnable);
|
|
thread.setName(getName() + "-" + i);
|
|
thread.setDaemon(true);
|
|
thread.start();
|
|
}
|
|
// Wait for the threads to have finished
|
|
try
|
|
{
|
|
endLatch.await(60, TimeUnit.SECONDS);
|
|
}
|
|
catch (InterruptedException e)
|
|
{
|
|
}
|
|
|
|
// Now, get the user with full split person handling
|
|
personServiceImpl.setDuplicateMode("DELETE");
|
|
|
|
RetryingTransactionCallback<NodeRef> getPersonWork = new RetryingTransactionCallback<NodeRef>()
|
|
{
|
|
public NodeRef execute() throws Throwable
|
|
{
|
|
return personService.getPerson(duplicateUsername);
|
|
}
|
|
};
|
|
final NodeRef remainingNodeRef = transactionService.getRetryingTransactionHelper().doInTransaction(getPersonWork, false, true);
|
|
transactionService.getRetryingTransactionHelper().doInTransaction(new RetryingTransactionCallback<Object>(){
|
|
|
|
public Object execute() throws Throwable
|
|
{
|
|
// Should all be cleaned up now, but no way to check
|
|
for (NodeRef nodeRef : cleanableNodeRefs.values())
|
|
{
|
|
if (nodeRef.equals(remainingNodeRef))
|
|
{
|
|
// This one should still be around
|
|
continue;
|
|
}
|
|
if (nodeService.exists(nodeRef))
|
|
{
|
|
fail("Expected unused person noderef to have been cleaned up: " + nodeRef);
|
|
}
|
|
}
|
|
return null;
|
|
}
|
|
}, true, true);
|
|
}
|
|
|
|
public void testSplitDuplicates()
|
|
{
|
|
testProcessDuplicates(true);
|
|
|
|
// Test out the SplitPersonCleanupBootstrapBean for removal of the duplicates
|
|
SplitPersonCleanupBootstrapBean splitPersonBean = new SplitPersonCleanupBootstrapBean();
|
|
splitPersonBean.setNodeService(nodeService);
|
|
splitPersonBean.setPersonService(personService);
|
|
splitPersonBean.setTransactionService(transactionService);
|
|
Assert.assertEquals(9, splitPersonBean.removePeopleWithGUIDBasedIds());
|
|
|
|
}
|
|
|
|
public void testDeleteDuplicates()
|
|
{
|
|
testProcessDuplicates(false);
|
|
}
|
|
|
|
private void testProcessDuplicates(final boolean split)
|
|
{
|
|
// Kill the annoying Spring-managed txn
|
|
super.setComplete();
|
|
super.endTransaction();
|
|
|
|
// Set the duplicate processing mode
|
|
((PersonServiceImpl) personService).setDuplicateMode(split ? "SPLIT" : "DELETE");
|
|
|
|
final String duplicateUserName = GUID.generate();
|
|
final NodeRef[] duplicates = transactionService.getRetryingTransactionHelper().doInTransaction(
|
|
new RetryingTransactionCallback<NodeRef[]>()
|
|
{
|
|
|
|
public NodeRef[] execute() throws Throwable
|
|
{
|
|
NodeRef[] duplicates = new NodeRef[10];
|
|
|
|
// Generate a first person node
|
|
Map<QName, Serializable> properties = createDefaultProperties(duplicateUserName, "firstName", "lastName", "email@orgId", "orgId", null);
|
|
duplicates[0] = personService.createPerson(properties);
|
|
ChildAssociationRef container = nodeService.getPrimaryParent(duplicates[0]);
|
|
List<ChildAssociationRef> parents = nodeService.getParentAssocs(duplicates[0]);
|
|
|
|
// Generate some duplicates
|
|
for (int i = 1; i < duplicates.length; i++)
|
|
{
|
|
// Create the node with the same parent assocs
|
|
duplicates[i] = nodeService.createNode(container.getParentRef(), container.getTypeQName(),
|
|
container.getQName(), ContentModel.TYPE_PERSON, properties).getChildRef();
|
|
for (ChildAssociationRef parent : parents)
|
|
{
|
|
if (!parent.isPrimary())
|
|
{
|
|
nodeService.addChild(parent.getParentRef(), duplicates[i], parent.getTypeQName(),
|
|
parent.getQName());
|
|
}
|
|
}
|
|
}
|
|
// With the default settings, the last created node should be the one that wins
|
|
assertEquals(duplicates[duplicates.length - 1], personService.getPerson(duplicateUserName));
|
|
return duplicates;
|
|
}
|
|
}, false, true);
|
|
|
|
// Check the duplicates were processed appropriately in the previous transaction
|
|
transactionService.getRetryingTransactionHelper().doInTransaction(new RetryingTransactionCallback<Void>()
|
|
{
|
|
public Void execute() throws Throwable
|
|
{
|
|
for (int i = 0; i < duplicates.length - 1; i++)
|
|
{
|
|
if (split)
|
|
{
|
|
String newUserName = (String) nodeService
|
|
.getProperty(duplicates[i], ContentModel.PROP_USERNAME);
|
|
assertNotSame(duplicateUserName, newUserName);
|
|
}
|
|
else
|
|
{
|
|
assertFalse(nodeService.exists(duplicates[i]));
|
|
}
|
|
}
|
|
|
|
// Get rid of the non-split person
|
|
assertTrue(personService.personExists(duplicateUserName));
|
|
personService.deletePerson(duplicateUserName);
|
|
return null;
|
|
}
|
|
}, false, true);
|
|
}
|
|
}
|