diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEvent.java b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEvent.java index 48f419f701..2141f524c6 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEvent.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEvent.java @@ -49,7 +49,9 @@ import org.alfresco.service.namespace.QName; @BehaviourBean public class RemoveFromHoldAuditEvent extends AuditEvent implements HoldServicePolicies.OnRemoveFromHoldPolicy { - /** Node Service */ + /** + * Node Service + */ 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 @Behaviour diff --git a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/legacy/service/RecordsManagementAuditServiceImplTest.java b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/legacy/service/RecordsManagementAuditServiceImplTest.java index 3d246fea3c..6aee148602 100644 --- a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/legacy/service/RecordsManagementAuditServiceImplTest.java +++ b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/legacy/service/RecordsManagementAuditServiceImplTest.java @@ -72,6 +72,12 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase /** Test start time */ 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() */ @@ -748,13 +754,9 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase { doBehaviourDrivenTest(new BehaviourDrivenTest() { - final static String REMOVE_FROM_HOLD_AUDIT_EVENT = "Remove From Hold"; - String holdName = "Hold " + GUID.generate(); NodeRef hold; - Map auditEventProperties; - @Override public void given() { @@ -772,7 +774,7 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase @Override public void then() { - auditEventProperties = getAuditEntry(REMOVE_FROM_HOLD_AUDIT_EVENT).getBeforeProperties(); + Map auditEventProperties = getAuditEntry(REMOVE_FROM_HOLD_AUDIT_EVENT).getBeforeProperties(); // check remove from hold audit event includes the hold name assertEquals("Remove From Hold event does not include hold name.", holdName, @@ -806,14 +808,10 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase { doBehaviourDrivenTest(new BehaviourDrivenTest() { - final static String REMOVE_FROM_HOLD_AUDIT_EVENT = "Remove From Hold"; - String holdName1 = "Hold " + GUID.generate(); String holdName2 = "Hold " + GUID.generate(); NodeRef hold1, hold2; - List> auditEventPropertiesList; - @Override public void given() { @@ -823,26 +821,21 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase hold2 = utils.createHold(filePlan, holdName2, "Reason " + GUID.generate()); utils.addItemToHold(hold1, dmDocument); utils.addItemToHold(hold2, dmDocument); - } @Override public void when() { - utils.removeItemFromHold(hold1, dmDocument); - utils.removeItemFromHold(hold2, dmDocument); + utils.removeItemsFromHolds(Arrays.asList(hold1, hold2), Arrays.asList(dmDocument)); } @Override public void then() { - auditEventPropertiesList = getAuditEntries(REMOVE_FROM_HOLD_AUDIT_EVENT) - .stream() - .map(RecordsManagementAuditEntry::getBeforeProperties) - .collect(Collectors.toList()); + List auditEntries = getAuditEntries(REMOVE_FROM_HOLD_AUDIT_EVENT); - // check remove from hold audit event exists for both records - assertEquals(2, auditEventPropertiesList.size()); + // check remove from hold audit event exists for both holds + assertEquals(2, auditEntries.size()); } @Override @@ -867,13 +860,9 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase { doBehaviourDrivenTest(new BehaviourDrivenTest() { - final static String REMOVE_FROM_HOLD_AUDIT_EVENT = "Remove From Hold"; - String holdName = "Hold " + GUID.generate(); NodeRef hold; - List> auditEventPropertiesList; - @Override public void given() { @@ -893,13 +882,10 @@ public class RecordsManagementAuditServiceImplTest extends BaseRMTestCase @Override public void then() { - auditEventPropertiesList = getAuditEntries(REMOVE_FROM_HOLD_AUDIT_EVENT) - .stream() - .map(RecordsManagementAuditEntry::getBeforeProperties) - .collect(Collectors.toList()); + List auditEntries = getAuditEntries(REMOVE_FROM_HOLD_AUDIT_EVENT); - // check remove from hold audit event exists for both records - assertEquals(2, auditEventPropertiesList.size()); + // check remove from hold audit event exists for both documents + assertEquals(2, auditEntries.size()); } @Override diff --git a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/CommonRMTestUtils.java b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/CommonRMTestUtils.java index 63721187e4..692ac87edc 100644 --- a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/CommonRMTestUtils.java +++ b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/CommonRMTestUtils.java @@ -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 contentNodeRef the list of items which will be removed from the given holds diff --git a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEventUnitTest.java b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEventUnitTest.java index 8c84469428..57fcaad37f 100644 --- a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEventUnitTest.java +++ b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/audit/event/RemoveFromHoldAuditEventUnitTest.java @@ -48,7 +48,7 @@ import org.mockito.InjectMocks; import org.mockito.Mock; /** - * Unit tests for {@link CreateHoldAuditEvent}. + * Unit tests for {@link RemoveFromHoldAuditEvent}. * * @author Chris Shields * @since 3.3 @@ -86,7 +86,7 @@ public class RemoveFromHoldAuditEventUnitTest extends BaseUnitTest * Check that the remove from hold event calls an audit event. */ @Test - public void testAddToHoldCausesAuditEvent() + public void testRemoveFromHoldCausesAuditEvent() { removeFromHoldAuditEvent.onRemoveFromHold(holdNodeRef, contentNodeRef); verify(mockedRecordsManagementAuditService, times(1)).auditEvent(eq(contentNodeRef), any(String.class), any(Map.class), isNull(Map.class), eq(true));