From 9964b70b527e53498792c4e0a4e85ee8071d742c Mon Sep 17 00:00:00 2001 From: Roxana Lucanu-Ghetu Date: Tue, 14 Jun 2016 12:27:17 +0300 Subject: [PATCH 1/5] Throw exception when trying to create records from different types other than cm:content --- .../model/rma/type/RecordsManagementContainerType.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java index 659b509aa9..cfbb51f0be 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java @@ -27,6 +27,7 @@ package org.alfresco.module.org_alfresco_module_rm.model.rma.type; +import org.alfresco.error.AlfrescoRuntimeException; import org.alfresco.model.ContentModel; import org.alfresco.module.org_alfresco_module_rm.identifier.IdentifierService; import org.alfresco.module.org_alfresco_module_rm.model.BaseBehaviourBean; @@ -147,6 +148,13 @@ public class RecordsManagementContainerType extends BaseBehaviourBean } else { + // Throw exception if the type is not cm:content + if (!ContentModel.TYPE_FOLDER.equals(childType) && + !dictionaryService.isSubClass(childType, ContentModel.TYPE_CONTENT)) + { + throw new AlfrescoRuntimeException("Record can't be created from this type."); + } + // We need to automatically cast the created folder to RM type if it is a plain folder // This occurs if the RM folder has been created via IMap, WebDav, etc if (!nodeService.hasAspect(child, ASPECT_FILE_PLAN_COMPONENT)) From 6f99d9435792cac9ac04ae2c480ebe66e8b9b912 Mon Sep 17 00:00:00 2001 From: Roxana Lucanu-Ghetu Date: Fri, 17 Jun 2016 12:25:22 +0300 Subject: [PATCH 2/5] Changed check to throw exception when we try to create records from different types other than cm:folder --- .../rma/type/RecordsManagementContainerType.java | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java index cfbb51f0be..14340ae938 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java @@ -148,17 +148,15 @@ public class RecordsManagementContainerType extends BaseBehaviourBean } else { - // Throw exception if the type is not cm:content - if (!ContentModel.TYPE_FOLDER.equals(childType) && - !dictionaryService.isSubClass(childType, ContentModel.TYPE_CONTENT)) - { - throw new AlfrescoRuntimeException("Record can't be created from this type."); - } - // We need to automatically cast the created folder to RM type if it is a plain folder // This occurs if the RM folder has been created via IMap, WebDav, etc if (!nodeService.hasAspect(child, ASPECT_FILE_PLAN_COMPONENT)) { + // Throw exception if the type is not cm:folder + if (!ContentModel.TYPE_FOLDER.equals(childType)) + { + throw new AlfrescoRuntimeException("Record can't be created from this type."); + } // check the type of the parent to determine what 'kind' of artifact to create NodeRef parent = childAssocRef.getParentRef(); QName parentType = nodeService.getType(parent); From a75fe88eb9645c7b3e1a7154b0f0c52052bd8e24 Mon Sep 17 00:00:00 2001 From: Roxana Lucanu-Ghetu Date: Mon, 27 Jun 2016 10:58:01 +0300 Subject: [PATCH 3/5] Unit and integration test for RM-3450 --- .../messages/action-service.properties | 3 +- .../type/RecordsManagementContainerType.java | 6 +- .../test/integration/issue/RM3450Test.java | 63 +++++++++++++++++++ .../test/util/TestModel.java | 1 + .../test/resources/test-model.xml | 3 + .../RecordsManagementContainerTypeTest.java | 62 ++++++++++++++---- 6 files changed, 125 insertions(+), 13 deletions(-) create mode 100644 rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/RM3450Test.java diff --git a/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/messages/action-service.properties b/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/messages/action-service.properties index 5c6a7d4daf..c0d8b967b9 100644 --- a/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/messages/action-service.properties +++ b/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/messages/action-service.properties @@ -36,4 +36,5 @@ rm.action.records_only_undeclared=Only records can be completed. rm.action.event-not-undone=The event {0} can't be undone, because it's not defined on the disposition lifecycle. rm.action.node-not-record-category=The disposition schedule could not be created, because the actioned upon node ({0}) was not a record category. rm.action.parameter-not-supplied=The parameter ''{0}'' has not been supplied. -rm.action.delete-not-hold-type=The hold couldn't be deleted, because the node isn't of type {0}. (actionedUponNodeRef={1}) \ No newline at end of file +rm.action.delete-not-hold-type=The hold couldn't be deleted, because the node isn't of type {0}. (actionedUponNodeRef={1}) +rm.action.cast-to-rm-type=Cannot cast to RM type. \ No newline at end of file diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java index 14340ae938..522cd5d96e 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerType.java @@ -44,6 +44,7 @@ import org.alfresco.repo.security.authentication.AuthenticationUtil.RunAsWork; import org.alfresco.service.cmr.repository.ChildAssociationRef; import org.alfresco.service.cmr.repository.NodeRef; import org.alfresco.service.namespace.QName; +import org.springframework.extensions.surf.util.I18NUtil; /** * rma:recordsManagementContainer behaviour bean. @@ -70,6 +71,9 @@ public class RecordsManagementContainerType extends BaseBehaviourBean /** record folder service */ protected RecordFolderService recordFolderService; + /** I18N */ + private static final String MSG_CANNOT_CAST_TO_RM_TYPE = "rm.action.cast-to-rm-type"; + /** * @param identifierService identifier service */ @@ -155,7 +159,7 @@ public class RecordsManagementContainerType extends BaseBehaviourBean // Throw exception if the type is not cm:folder if (!ContentModel.TYPE_FOLDER.equals(childType)) { - throw new AlfrescoRuntimeException("Record can't be created from this type."); + throw new AlfrescoRuntimeException(I18NUtil.getMessage(MSG_CANNOT_CAST_TO_RM_TYPE)); } // check the type of the parent to determine what 'kind' of artifact to create NodeRef parent = childAssocRef.getParentRef(); diff --git a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/RM3450Test.java b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/RM3450Test.java new file mode 100644 index 0000000000..3146febbbb --- /dev/null +++ b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/RM3450Test.java @@ -0,0 +1,63 @@ +/* + * #%L + * Alfresco Records Management Module + * %% + * Copyright (C) 2005 - 2016 Alfresco Software Limited + * %% + * This file is part of the Alfresco software. + * - + * If the software was purchased under a paid Alfresco license, the terms of + * the paid license agreement will prevail. Otherwise, the software is + * provided under the following open source license terms: + * - + * 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 . + * #L% + */ +package org.alfresco.module.org_alfresco_module_rm.test.integration.issue; + +import org.alfresco.error.AlfrescoRuntimeException; +import org.alfresco.module.org_alfresco_module_rm.test.util.BaseRMTestCase; +import org.alfresco.module.org_alfresco_module_rm.test.util.TestModel; +import org.alfresco.util.GUID; +import org.springframework.extensions.surf.util.I18NUtil; + +/** + * Integration test for RM-3450 + * + * @author Roxana Lucanu + * @since 2.4.1 + */ +public class RM3450Test extends BaseRMTestCase +{ + + private static final String MSG_CANNOT_CAST_TO_RM_TYPE = "rm.action.cast-to-rm-type"; + + public void testRM3450() throws Exception + { + doTestInTransaction(new FailureTest + ( + I18NUtil.getMessage(MSG_CANNOT_CAST_TO_RM_TYPE), + AlfrescoRuntimeException.class + ) + { + @Override + public void run() throws Exception + { + //expect failure + fileFolderService.create(unfiledContainer, GUID.generate(), TestModel.NOT_RM_FOLDER_TYPE).getNodeRef(); + } + }, ADMIN_USER); + } + +} diff --git a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/TestModel.java b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/TestModel.java index 08b6854a56..15e7c45939 100644 --- a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/TestModel.java +++ b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/util/TestModel.java @@ -40,4 +40,5 @@ public interface TestModel public static final QName ASPECT_RECORD_METADATA = QName.createQName(TEST_URI, "recordMetaData"); public static final QName PROPERTY_RECORD_METADATA = QName.createQName(TEST_URI, "recordMetaDataProperty"); + public static final QName NOT_RM_FOLDER_TYPE = QName.createQName(TEST_URI, "notRmFolderType"); } diff --git a/rm-community/rm-community-repo/test/resources/test-model.xml b/rm-community/rm-community-repo/test/resources/test-model.xml index 41869338d5..9eafa717a3 100644 --- a/rm-community/rm-community-repo/test/resources/test-model.xml +++ b/rm-community/rm-community-repo/test/resources/test-model.xml @@ -28,6 +28,9 @@ cm:content + + cm:folder + diff --git a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeTest.java b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeTest.java index 6f9fd8a661..7c78c6f080 100644 --- a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeTest.java +++ b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeTest.java @@ -26,16 +26,19 @@ */ package org.alfresco.module.org_alfresco_module_rm.model.rma.type; -import static org.mockito.Mockito.never; -import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.when; - -import org.alfresco.model.ContentModel; -import org.alfresco.module.org_alfresco_module_rm.model.RecordsManagementModel; -import org.alfresco.module.org_alfresco_module_rm.test.util.BaseUnitTest; -import org.alfresco.service.cmr.repository.ChildAssociationRef; -import org.alfresco.service.cmr.repository.NodeRef; -import org.junit.Test; +import static org.junit.Assert.assertTrue; +import static org.mockito.Mockito.never; +import static org.mockito.Mockito.verify; +import static org.mockito.Mockito.when; + +import org.alfresco.error.AlfrescoRuntimeException; +import org.alfresco.model.ContentModel; +import org.alfresco.module.org_alfresco_module_rm.model.RecordsManagementModel; +import org.alfresco.module.org_alfresco_module_rm.test.util.BaseUnitTest; +import org.alfresco.module.org_alfresco_module_rm.test.util.TestModel; +import org.alfresco.service.cmr.repository.ChildAssociationRef; +import org.alfresco.service.cmr.repository.NodeRef; +import org.junit.Test; import org.mockito.InjectMocks; /** @@ -46,7 +49,7 @@ import org.mockito.InjectMocks; public class RecordsManagementContainerTypeTest extends BaseUnitTest { /** test object */ - private @InjectMocks RecordsManagementContainerType recordManagementContainerType; + private @InjectMocks RecordsManagementContainerType recordManagementContainerType; /** * Having the Unfilled Record container and a folder @@ -92,6 +95,28 @@ public class RecordsManagementContainerTypeTest extends BaseUnitTest /* Then the node type should not be changed to TYPE_RECORD_FOLDER */ verify(mockedNodeService, never()).setType(rmFolder, TYPE_RECORD_FOLDER); verify(mockedRecordFolderService, never()).setupRecordFolder(rmFolder); + } + + /** + * Trying to create a RM folder and its sub-types via SFDC, IMap, WebDav, etc + * Check that exception is thrown on creating child associations + */ + @Test + public void testRM3450() + { + NodeRef rmContainer = generateRMContainer(); + NodeRef nonRmFolder = generateNonRmFolderNode(); + + ChildAssociationRef childAssoc = new ChildAssociationRef(ContentModel.ASSOC_CONTAINS, rmContainer, TestModel.NOT_RM_FOLDER_TYPE, nonRmFolder); + try + { + recordManagementContainerType.onCreateChildAssociation(childAssoc, true); + assertTrue("Expected to throw exception on create child association.", false); + } + catch (Throwable e) + { + assertTrue(e instanceof AlfrescoRuntimeException); + } } /** @@ -121,5 +146,20 @@ public class RecordsManagementContainerTypeTest extends BaseUnitTest when(mockedNodeService.hasAspect(rmFolder, ContentModel.ASPECT_HIDDEN)).thenReturn(hasHiddenAspect); when(mockedNodeService.hasAspect(rmFolder, ASPECT_FILE_PLAN_COMPONENT)).thenReturn(false); return rmFolder; + } + + /** + * Generates a folder node + * @return reference to the created folder + */ + private NodeRef generateNonRmFolderNode() + { + NodeRef nonRmFolder = generateNodeRef(); + when(mockedDictionaryService.isSubClass(TestModel.NOT_RM_FOLDER_TYPE, ContentModel.TYPE_FOLDER)).thenReturn(true); + when(mockedNodeService.getType(nonRmFolder)).thenReturn(TestModel.NOT_RM_FOLDER_TYPE); + when(mockedNodeService.exists(nonRmFolder)).thenReturn(true); + when(mockedNodeService.hasAspect(nonRmFolder, ContentModel.ASPECT_HIDDEN)).thenReturn(false); + when(mockedNodeService.hasAspect(nonRmFolder, ASPECT_FILE_PLAN_COMPONENT)).thenReturn(false); + return nonRmFolder; } } From d2588048abb449802d676dc639efe8ef43042f52 Mon Sep 17 00:00:00 2001 From: Roxana Lucanu-Ghetu Date: Thu, 30 Jun 2016 12:00:23 +0300 Subject: [PATCH 4/5] Called the method to test inside another transaction. --- .../test/integration/issue/RM3450Test.java | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/RM3450Test.java b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/RM3450Test.java index 3146febbbb..6c325ab290 100644 --- a/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/RM3450Test.java +++ b/rm-community/rm-community-repo/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/RM3450Test.java @@ -29,6 +29,7 @@ package org.alfresco.module.org_alfresco_module_rm.test.integration.issue; import org.alfresco.error.AlfrescoRuntimeException; import org.alfresco.module.org_alfresco_module_rm.test.util.BaseRMTestCase; import org.alfresco.module.org_alfresco_module_rm.test.util.TestModel; +import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback; import org.alfresco.util.GUID; import org.springframework.extensions.surf.util.I18NUtil; @@ -54,8 +55,14 @@ public class RM3450Test extends BaseRMTestCase @Override public void run() throws Exception { - //expect failure - fileFolderService.create(unfiledContainer, GUID.generate(), TestModel.NOT_RM_FOLDER_TYPE).getNodeRef(); + transactionService.getRetryingTransactionHelper().doInTransaction(new RetryingTransactionCallback() + { + public Object execute() throws Exception + { + fileFolderService.create(unfiledContainer, GUID.generate(), TestModel.NOT_RM_FOLDER_TYPE).getNodeRef(); + return null; + } + }, false, true); } }, ADMIN_USER); } From 7de04dd516984a70c27700259968dc89eeed4368 Mon Sep 17 00:00:00 2001 From: Roxana Lucanu-Ghetu Date: Tue, 5 Jul 2016 11:25:08 +0300 Subject: [PATCH 5/5] Renamed unit test classes and changed error message. --- .../messages/action-service.properties | 2 +- ...ContainerTypeTest.java => HoldContainerTypeUnitTest.java} | 2 +- ...Test.java => RecordsManagementContainerTypeUnitTest.java} | 5 +++-- ...ainerTypeTest.java => TransferContainerTypeUnitTest.java} | 2 +- 4 files changed, 6 insertions(+), 5 deletions(-) rename rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/{HoldContainerTypeTest.java => HoldContainerTypeUnitTest.java} (95%) rename rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/{RecordsManagementContainerTypeTest.java => RecordsManagementContainerTypeUnitTest.java} (95%) rename rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/{TransferContainerTypeTest.java => TransferContainerTypeUnitTest.java} (94%) diff --git a/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/messages/action-service.properties b/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/messages/action-service.properties index c0d8b967b9..7b444b2cc9 100644 --- a/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/messages/action-service.properties +++ b/rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/messages/action-service.properties @@ -37,4 +37,4 @@ rm.action.event-not-undone=The event {0} can't be undone, because it's not defin rm.action.node-not-record-category=The disposition schedule could not be created, because the actioned upon node ({0}) was not a record category. rm.action.parameter-not-supplied=The parameter ''{0}'' has not been supplied. rm.action.delete-not-hold-type=The hold couldn't be deleted, because the node isn't of type {0}. (actionedUponNodeRef={1}) -rm.action.cast-to-rm-type=Cannot cast to RM type. \ No newline at end of file +rm.action.cast-to-rm-type=Can't upload a custom folder type to the records management file plan. \ No newline at end of file diff --git a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/HoldContainerTypeTest.java b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/HoldContainerTypeUnitTest.java similarity index 95% rename from rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/HoldContainerTypeTest.java rename to rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/HoldContainerTypeUnitTest.java index 60ac81f490..4048585913 100644 --- a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/HoldContainerTypeTest.java +++ b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/HoldContainerTypeUnitTest.java @@ -46,7 +46,7 @@ import org.mockito.InjectMocks; * @author Mihai Cozma * @since 2.4 */ -public class HoldContainerTypeTest extends BaseUnitTest +public class HoldContainerTypeUnitTest extends BaseUnitTest { /** test object */ private @InjectMocks HoldContainerType holdContainerType; diff --git a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeTest.java b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeUnitTest.java similarity index 95% rename from rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeTest.java rename to rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeUnitTest.java index 7c78c6f080..5589f7a410 100644 --- a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeTest.java +++ b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/RecordsManagementContainerTypeUnitTest.java @@ -27,6 +27,7 @@ package org.alfresco.module.org_alfresco_module_rm.model.rma.type; import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; import static org.mockito.Mockito.never; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -46,7 +47,7 @@ import org.mockito.InjectMocks; * @author Ana Bozianu * @since 2.4 */ -public class RecordsManagementContainerTypeTest extends BaseUnitTest +public class RecordsManagementContainerTypeUnitTest extends BaseUnitTest { /** test object */ private @InjectMocks RecordsManagementContainerType recordManagementContainerType; @@ -111,7 +112,7 @@ public class RecordsManagementContainerTypeTest extends BaseUnitTest try { recordManagementContainerType.onCreateChildAssociation(childAssoc, true); - assertTrue("Expected to throw exception on create child association.", false); + fail("Expected to throw exception on create child association."); } catch (Throwable e) { diff --git a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/TransferContainerTypeTest.java b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/TransferContainerTypeUnitTest.java similarity index 94% rename from rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/TransferContainerTypeTest.java rename to rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/TransferContainerTypeUnitTest.java index 95fe49f052..69a7e20b2f 100644 --- a/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/TransferContainerTypeTest.java +++ b/rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/model/rma/type/TransferContainerTypeUnitTest.java @@ -40,7 +40,7 @@ import org.mockito.InjectMocks; * @author Mihai Cozma * @since 2.4 */ -public class TransferContainerTypeTest extends BaseUnitTest +public class TransferContainerTypeUnitTest extends BaseUnitTest { /** test object */ private @InjectMocks TransferContainerType transferContainerType;