inteligr8/alfresco-community-repo
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-community-repo.git synced 2025-07-31 17:39:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

Code review changes

Browse Source
This commit is contained in:
Chris Shields
2019-11-19 21:11:50 +00:00
parent 4a89c38f2d
commit bc5c5bd51d
4 changed files with 21 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEvent.java
View File

@@ -49,7 +49,9 @@ import org.alfresco.service.namespace.QName;
@BehaviourBean @BehaviourBean
public class RemoveFromHoldAuditEvent extends AuditEvent implements HoldServicePolicies.OnRemoveFromHoldPolicy public class RemoveFromHoldAuditEvent extends AuditEvent implements HoldServicePolicies.OnRemoveFromHoldPolicy
{ {
/** Node Service */ /**
* Node Service
*/
private NodeService nodeService; private NodeService nodeService;
/** /**
@@ -63,7 +65,7 @@ public class RemoveFromHoldAuditEvent extends AuditEvent implements HoldServiceP
} }
/** /**
* @see @see org.alfresco.module.org_alfresco_module_rm.hold.HoldServicePolicies.OnRemoveFromHoldPolicy#onRemoveFromHold(org.alfresco.service.cmr.repository.NodeRef, org.alfresco.service.cmr.repository.NodeRef) * @see org.alfresco.module.org_alfresco_module_rm.hold.HoldServicePolicies.OnRemoveFromHoldPolicy#onRemoveFromHold(org.alfresco.service.cmr.repository.NodeRef, org.alfresco.service.cmr.repository.NodeRef)
*/ */
@Override @Override
@Behaviour @Behaviour

42
rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/legacy/service/RecordsManagementAuditServiceImplTest.java
View File

@@ -72,6 +72,12 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase
/** Test start time */ /** Test start time */
private Date testStartTime; private Date testStartTime;
/**
* Remove from hold audit event name.
*/
private static final String REMOVE_FROM_HOLD_AUDIT_EVENT = "Remove From Hold";
/** /**
* @see org.alfresco.module.org_alfresco_module_rm.test.util.BaseRMTestCase#setUp() * @see org.alfresco.module.org_alfresco_module_rm.test.util.BaseRMTestCase#setUp()
*/ */
@@ -748,13 +754,9 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase
{ {
doBehaviourDrivenTest(new BehaviourDrivenTest() doBehaviourDrivenTest(new BehaviourDrivenTest()
{ {
final static String REMOVE_FROM_HOLD_AUDIT_EVENT = "Remove From Hold";
String holdName = "Hold " + GUID.generate(); String holdName = "Hold " + GUID.generate();
NodeRef hold; NodeRef hold;
Map<QName, Serializable> auditEventProperties;
@Override @Override
public void given() public void given()
{ {
@@ -772,7 +774,7 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase
@Override @Override
public void then() public void then()
{ {
auditEventProperties = getAuditEntry(REMOVE_FROM_HOLD_AUDIT_EVENT).getBeforeProperties(); Map<QName, Serializable> auditEventProperties = getAuditEntry(REMOVE_FROM_HOLD_AUDIT_EVENT).getBeforeProperties();
// check remove from hold audit event includes the hold name // check remove from hold audit event includes the hold name
assertEquals("Remove From Hold event does not include hold name.", holdName, assertEquals("Remove From Hold event does not include hold name.", holdName,
@@ -806,14 +808,10 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase
{ {
doBehaviourDrivenTest(new BehaviourDrivenTest() doBehaviourDrivenTest(new BehaviourDrivenTest()
{ {
final static String REMOVE_FROM_HOLD_AUDIT_EVENT = "Remove From Hold";
String holdName1 = "Hold " + GUID.generate(); String holdName1 = "Hold " + GUID.generate();
String holdName2 = "Hold " + GUID.generate(); String holdName2 = "Hold " + GUID.generate();
NodeRef hold1, hold2; NodeRef hold1, hold2;
List<Map<QName, Serializable>> auditEventPropertiesList;
@Override @Override
public void given() public void given()
{ {
@@ -823,26 +821,21 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase
hold2 = utils.createHold(filePlan, holdName2, "Reason " + GUID.generate()); hold2 = utils.createHold(filePlan, holdName2, "Reason " + GUID.generate());
utils.addItemToHold(hold1, dmDocument); utils.addItemToHold(hold1, dmDocument);
utils.addItemToHold(hold2, dmDocument); utils.addItemToHold(hold2, dmDocument);
} }
@Override @Override
public void when() public void when()
{ {
utils.removeItemFromHold(hold1, dmDocument); utils.removeItemsFromHolds(Arrays.asList(hold1, hold2), Arrays.asList(dmDocument));
utils.removeItemFromHold(hold2, dmDocument);
} }
@Override @Override
public void then() public void then()
{ {
auditEventPropertiesList = getAuditEntries(REMOVE_FROM_HOLD_AUDIT_EVENT) List<RecordsManagementAuditEntry> auditEntries = getAuditEntries(REMOVE_FROM_HOLD_AUDIT_EVENT);
.stream()
.map(RecordsManagementAuditEntry::getBeforeProperties)
.collect(Collectors.toList());
// check remove from hold audit event exists for both records // check remove from hold audit event exists for both holds
assertEquals(2, auditEventPropertiesList.size()); assertEquals(2, auditEntries.size());
} }
@Override @Override
@@ -867,13 +860,9 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase
{ {
doBehaviourDrivenTest(new BehaviourDrivenTest() doBehaviourDrivenTest(new BehaviourDrivenTest()
{ {
final static String REMOVE_FROM_HOLD_AUDIT_EVENT = "Remove From Hold";
String holdName = "Hold " + GUID.generate(); String holdName = "Hold " + GUID.generate();
NodeRef hold; NodeRef hold;
List<Map<QName, Serializable>> auditEventPropertiesList;
@Override @Override
public void given() public void given()
{ {
@@ -893,13 +882,10 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase
@Override @Override
public void then() public void then()
{ {
auditEventPropertiesList = getAuditEntries(REMOVE_FROM_HOLD_AUDIT_EVENT) List<RecordsManagementAuditEntry> auditEntries = getAuditEntries(REMOVE_FROM_HOLD_AUDIT_EVENT);
.stream()
.map(RecordsManagementAuditEntry::getBeforeProperties)
.collect(Collectors.toList());
// check remove from hold audit event exists for both records // check remove from hold audit event exists for both documents
assertEquals(2, auditEventPropertiesList.size()); assertEquals(2, auditEntries.size());
} }
@Override @Override

2
rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/CommonRMTestUtils.java
View File

@@ -456,7 +456,7 @@ public class CommonRMTestUtils implements RecordsManagementModel
} }
/** /**
* Util method to remove items from a holds. * Util method to remove items from holds.
* *
* @param holdNodeRefs the list {@link NodeRef}s of the holds * @param holdNodeRefs the list {@link NodeRef}s of the holds
* @param contentNodeRef the list of items which will be removed from the given holds * @param contentNodeRef the list of items which will be removed from the given holds

4
rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEventUnitTest.java
View File

@@ -48,7 +48,7 @@ import org.mockito.InjectMocks;
import org.mockito.Mock; import org.mockito.Mock;
/** /**
* Unit tests for {@link CreateHoldAuditEvent}. * Unit tests for {@link RemoveFromHoldAuditEvent}.
* *
* @author Chris Shields * @author Chris Shields
* @since 3.3 * @since 3.3
@@ -86,7 +86,7 @@ public class RemoveFromHoldAuditEventUnitTest extends BaseUnitTest
* Check that the remove from hold event calls an audit event. * Check that the remove from hold event calls an audit event.
*/ */
@Test @Test
public void testAddToHoldCausesAuditEvent() public void testRemoveFromHoldCausesAuditEvent()
{ {
removeFromHoldAuditEvent.onRemoveFromHold(holdNodeRef, contentNodeRef); removeFromHoldAuditEvent.onRemoveFromHold(holdNodeRef, contentNodeRef);
verify(mockedRecordsManagementAuditService, times(1)).auditEvent(eq(contentNodeRef), any(String.class), any(Map.class), isNull(Map.class), eq(true)); verify(mockedRecordsManagementAuditService, times(1)).auditEvent(eq(contentNodeRef), any(String.class), any(Map.class), isNull(Map.class), eq(true));