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

Revert "RM-4619 - added the fileplan component check back but in different place"

Browse Source 
This reverts commit e1b0e994a1.
This commit is contained in:
Ana Bozianu
2017-01-30 16:53:58 +02:00
parent 40b61c5c62
commit 257880cb62
6 changed files with 25 additions and 48 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
rm-community/rm-community-repo/unit-test/java/org/alfresco/rm/rest/api/impl/RMNodesImplRelativePathUnitTest.java
View File

@@ -85,7 +85,6 @@ public class RMNodesImplRelativePathUnitTest extends BaseUnitTest
* Given any parent node
*/
NodeRef parentNode = AlfMock.generateNodeRef(mockedNodeService);
when(mockedFilePlanService.isFilePlanComponent(parentNode)).thenReturn(true);
/*
* When trying to create a node in the parent node with no relative path
@@ -113,7 +112,6 @@ public class RMNodesImplRelativePathUnitTest extends BaseUnitTest
* Given a parent node and an existing path c1/f1 under it
*/
NodeRef parentNode = AlfMock.generateNodeRef(mockedNodeService);
when(mockedFilePlanService.isFilePlanComponent(parentNode)).thenReturn(true);
String category = "c1";
NodeRef categoryNode = AlfMock.generateNodeRef(mockedNodeService);
@@ -156,7 +154,6 @@ public class RMNodesImplRelativePathUnitTest extends BaseUnitTest
String category1 = "c1";
NodeRef categoryNode1 = AlfMock.generateNodeRef(mockedNodeService);
when(mockedNodeService.getChildByName(fileplanNodeRef, ContentModel.ASSOC_CONTAINS, category1)).thenReturn(categoryNode1);
when(mockedFilePlanService.isFilePlanComponent(fileplanNodeRef)).thenReturn(true);
// create c2
String category2 = "c2";
@@ -203,7 +200,6 @@ public class RMNodesImplRelativePathUnitTest extends BaseUnitTest
*/
NodeRef unfiledRecordContainer = AlfMock.generateNodeRef(mockedNodeService);
when(mockedNodeService.getType(unfiledRecordContainer)).thenReturn(TYPE_UNFILED_RECORD_CONTAINER);
when(mockedFilePlanService.isFilePlanComponent(unfiledRecordContainer)).thenReturn(true);
/*
* When trying to create a content node in the relative path f1/f2/f3
@@ -259,7 +255,6 @@ public class RMNodesImplRelativePathUnitTest extends BaseUnitTest
*/
NodeRef fileplanNodeRef = AlfMock.generateNodeRef(mockedNodeService);
when(mockedNodeService.getType(fileplanNodeRef)).thenReturn(TYPE_FILE_PLAN);
when(mockedFilePlanService.isFilePlanComponent(fileplanNodeRef)).thenReturn(true);
/*
* When trying to create a folder node in the relative path c1/c2/c3

4
rm-community/rm-community-repo/unit-test/java/org/alfresco/rm/rest/api/impl/RMNodesImplUnitTest.java
View File

@@ -318,9 +318,10 @@ public class RMNodesImplUnitTest extends BaseUnitTest
setupCompanyHomeAndPrimaryParent(nodeRef);
when(mockedFilePlanService.isFilePlanComponent(nodeRef)).thenReturn(false);
List<String> includeParamList = new ArrayList<String>();
try
{
rmNodesImpl.validateNode(nodeRef.getId());
rmNodesImpl.getFolderOrDocument(nodeRef, null, null, includeParamList, null);
fail("Expected exception since the requested node is not a fileplan component.");
}
catch(InvalidParameterException ex)
@@ -627,7 +628,6 @@ public class RMNodesImplUnitTest extends BaseUnitTest
public void testValidateNode() throws Exception
{
NodeRef nodeRef = AlfMock.generateNodeRef(mockedNodeService);
when(mockedFilePlanService.isFilePlanComponent(nodeRef)).thenReturn(true);
NodeRef validateOrLookupNode = rmNodesImpl.validateNode(nodeRef.getId());
assertEquals(nodeRef, validateOrLookupNode);
}

33
rm-community/rm-community-repo/unit-test/java/org/alfresco/rm/rest/api/impl/RecordsImplUnitTest.java
View File

@@ -40,7 +40,6 @@ import org.alfresco.module.org_alfresco_module_rm.record.RecordService;
import org.alfresco.module.org_alfresco_module_rm.test.util.AlfMock;
import org.alfresco.module.org_alfresco_module_rm.test.util.MockAuthenticationUtilHelper;
import org.alfresco.module.org_alfresco_module_rm.util.AuthenticationUtil;
import org.alfresco.rest.api.Nodes;
import org.alfresco.rest.framework.resource.parameters.Parameters;
import org.alfresco.rm.rest.api.RMNodes;
import org.alfresco.rm.rest.api.Records;
@@ -78,9 +77,7 @@ public class RecordsImplUnitTest
@Mock
protected AuthenticationUtil mockedAuthenticationUtil;
@Mock
protected RMNodes mockedRMNodes;
@Mock
protected Nodes mockedNodes;
protected RMNodes mockedNodes;
@InjectMocks
private RecordsImpl recordsImpl;
@@ -122,7 +119,7 @@ public class RecordsImplUnitTest
* Then a record is created under the existing fileplan from the provided file
*/
verify(mockedRecordService).createRecord(fileplan, mockedFile, false);
verify(mockedRMNodes).getFolderOrDocument(mockedFile.getId(), params);
verify(mockedNodes).getFolderOrDocument(mockedFile.getId(), params);
}
/**
@@ -137,7 +134,7 @@ public class RecordsImplUnitTest
* Given a record
*/
NodeRef mockedRecord = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
when(mockedNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
/*
* When trying to filing a record providing a blank destination path
@@ -160,7 +157,7 @@ public class RecordsImplUnitTest
*/
// mock the record to file
NodeRef mockedRecord = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
when(mockedNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
// mock the current primary parent
NodeRef mockedPrimaryParent = AlfMock.generateNodeRef(mockedNodeService);
@@ -172,7 +169,7 @@ public class RecordsImplUnitTest
// mock the target record folder to file the record into
NodeRef mockedTargetRecordFolder = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.getOrCreatePath(mockedTargetRecordFolder.getId(), null, TYPE_CONTENT)).thenReturn(mockedTargetRecordFolder);
when(mockedNodes.getOrCreatePath(mockedTargetRecordFolder.getId(), null, TYPE_CONTENT)).thenReturn(mockedTargetRecordFolder);
when(mockedNodeService.getType(mockedTargetRecordFolder)).thenReturn(TYPE_RECORD_FOLDER);
when(mockedDictionaryService.isSubClass(TYPE_RECORD_FOLDER, TYPE_RECORD_FOLDER)).thenReturn(true);
@@ -188,7 +185,7 @@ public class RecordsImplUnitTest
/*
* Then the record is moved under the destination folder
*/
verify(mockedRMNodes).getOrCreatePath(mockedTargetRecordFolder.getId(), null, TYPE_CONTENT);
verify(mockedNodes).getOrCreatePath(mockedTargetRecordFolder.getId(), null, TYPE_CONTENT);
verify(mockedFileFolderService).moveFrom(mockedRecord, mockedPrimaryParent, mockedTargetRecordFolder, null);
}
@@ -206,7 +203,7 @@ public class RecordsImplUnitTest
*/
// mock the record to file
NodeRef mockedRecord = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
when(mockedNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
// mock the current primary parent
NodeRef mockedPrimaryParent = AlfMock.generateNodeRef(mockedNodeService);
@@ -223,7 +220,7 @@ public class RecordsImplUnitTest
// mock the target record folder to file the record into
String relativePath = "category/recordFolder";
NodeRef mockedTargetRecordFolder = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.getOrCreatePath(fileplan.getId(), relativePath, TYPE_CONTENT)).thenReturn(mockedTargetRecordFolder);
when(mockedNodes.getOrCreatePath(fileplan.getId(), relativePath, TYPE_CONTENT)).thenReturn(mockedTargetRecordFolder);
when(mockedNodeService.getType(mockedTargetRecordFolder)).thenReturn(TYPE_RECORD_FOLDER);
when(mockedDictionaryService.isSubClass(TYPE_RECORD_FOLDER, TYPE_RECORD_FOLDER)).thenReturn(true);
@@ -239,7 +236,7 @@ public class RecordsImplUnitTest
/*
* Then the record is moved under the destination folder relative to the fileplan
*/
verify(mockedRMNodes).getOrCreatePath(fileplan.getId(), relativePath, TYPE_CONTENT);
verify(mockedNodes).getOrCreatePath(fileplan.getId(), relativePath, TYPE_CONTENT);
verify(mockedFileFolderService).moveFrom(mockedRecord, mockedPrimaryParent, mockedTargetRecordFolder, null);
}
@@ -257,7 +254,7 @@ public class RecordsImplUnitTest
*/
// mock the record to file
NodeRef mockedRecord = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
when(mockedNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
// mock the current primary parent
NodeRef mockedPrimaryParent = AlfMock.generateNodeRef(mockedNodeService);
@@ -273,7 +270,7 @@ public class RecordsImplUnitTest
// mock the target record folder to file the record into
String relativePath = "category/recordFolder";
NodeRef mockedTargetRecordFolder = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.getOrCreatePath(mockedTargetCategory.getId(), relativePath, TYPE_CONTENT)).thenReturn(mockedTargetRecordFolder);
when(mockedNodes.getOrCreatePath(mockedTargetCategory.getId(), relativePath, TYPE_CONTENT)).thenReturn(mockedTargetRecordFolder);
when(mockedNodeService.getType(mockedTargetRecordFolder)).thenReturn(TYPE_RECORD_FOLDER);
when(mockedDictionaryService.isSubClass(TYPE_RECORD_FOLDER, TYPE_RECORD_FOLDER)).thenReturn(true);
@@ -290,7 +287,7 @@ public class RecordsImplUnitTest
/*
* Then the record is moved under the correct destination folder
*/
verify(mockedRMNodes).getOrCreatePath(mockedTargetCategory.getId(), relativePath, TYPE_CONTENT);
verify(mockedNodes).getOrCreatePath(mockedTargetCategory.getId(), relativePath, TYPE_CONTENT);
verify(mockedFileFolderService).moveFrom(mockedRecord, mockedPrimaryParent, mockedTargetRecordFolder, null);
}
@@ -307,7 +304,7 @@ public class RecordsImplUnitTest
*/
// mock the record to link
NodeRef mockedRecord = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
when(mockedNodes.validateNode(mockedRecord.getId())).thenReturn(mockedRecord);
// mock the current primary parent
NodeRef mockedPrimaryParent = AlfMock.generateNodeRef(mockedNodeService);
@@ -319,7 +316,7 @@ public class RecordsImplUnitTest
// mock the target record folder to file the record into
NodeRef mockedTargetRecordFolder = AlfMock.generateNodeRef(mockedNodeService);
when(mockedRMNodes.getOrCreatePath(mockedTargetRecordFolder.getId(), null, TYPE_CONTENT)).thenReturn(mockedTargetRecordFolder);
when(mockedNodes.getOrCreatePath(mockedTargetRecordFolder.getId(), null, TYPE_CONTENT)).thenReturn(mockedTargetRecordFolder);
when(mockedNodeService.getType(mockedTargetRecordFolder)).thenReturn(TYPE_RECORD_FOLDER);
/*
@@ -334,7 +331,7 @@ public class RecordsImplUnitTest
/*
* Then the record is linked to the destination folder
*/
verify(mockedRMNodes).getOrCreatePath(mockedTargetRecordFolder.getId(), null, TYPE_CONTENT);
verify(mockedNodes).getOrCreatePath(mockedTargetRecordFolder.getId(), null, TYPE_CONTENT);
verify(mockedRecordService).link(mockedRecord, mockedTargetRecordFolder);
}
}