Refactor API method to upload electronic record.

Throw assertion error if the upload is unsuccessful (rather than returning a boolean).
This commit is contained in:
Tom Page
2017-11-09 12:00:10 +00:00
parent e2a88600b0
commit 80ed8c4e63

View File

@@ -27,6 +27,7 @@
package org.alfresco.rest.v0; package org.alfresco.rest.v0;
import static org.apache.http.HttpStatus.SC_OK; import static org.apache.http.HttpStatus.SC_OK;
import static org.testng.AssertJUnit.assertTrue;
import java.text.MessageFormat; import java.text.MessageFormat;
import java.util.Map; import java.util.Map;
@@ -224,20 +225,20 @@ public class RecordsAPI extends BaseAPI
/** /**
* Uploads an electronic record * Uploads an electronic record
* * <p>
* eg. of usage for creating record directly in Unfiled Records : uploadElectronicRecord(getAdminName(), getAdminPassword(), recordPropertiesStringMap, UNFILED_RECORDS_BREADCRUMB, DocumentType.HTML)
* @param username the username * @param username the username
* @param password the password * @param password the password
* @param properties a map of record properties and their values * @param properties a map of record properties and their values
* @param folderName the folder inside which the record will be created, it needs to have a unique name * @param folderName the folder inside which the record will be created, it needs to have a unique name, as this method doesn't check other containers than the folder name
* @return true if the creation of the record has been successful * @throws AssertionError if the upload was unsuccessful.
* eg. of usage for creating record directly in Unfiled Records : uploadElectronicRecord(getAdminName(), getAdminPassword(), recordPropertiesStringMap, UNFILED_RECORDS_BREADCRUMB, DocumentType.HTML)
* the folder name in which the record gets created has to have a unique name, as this method doesn't check other containers than the folder name
*/ */
public boolean uploadElectronicRecord(String username, String password, Map<RMProperty, String> properties, String folderName, DocumentType documentType) public void uploadElectronicRecord(String username, String password, Map<RMProperty, String> properties, String folderName, DocumentType documentType)
{ {
String recordName = getPropertyValue(properties, RMProperty.NAME); String recordName = getPropertyValue(properties, RMProperty.NAME);
String recordContent = getPropertyValue(properties, RMProperty.CONTENT); String recordContent = getPropertyValue(properties, RMProperty.CONTENT);
return (getRecord(username, password, folderName, recordName) != null) || (contentService.createDocumentInFolder(username, password, RM_SITE_ID, folderName, documentType, recordName, recordContent) != null); boolean success = (getRecord(username, password, folderName, recordName) != null) || (contentService.createDocumentInFolder(username, password, RM_SITE_ID, folderName, documentType, recordName, recordContent) != null);
assertTrue("Failed to upload electronic record to " + folderName, success);
} }
/** /**