From ea1c13f5f592eb968d00a5771c01a34922a29926 Mon Sep 17 00:00:00 2001 From: cramakri <cramakri> Date: Wed, 12 Jan 2011 11:57:01 +0000 Subject: [PATCH] LMS-1966 Renamed TransferredDataSetHandlerDataSetRegistrationAlgorithm to DataSetRegistrationHelper. SVN: 19385 --- .../DataSetRegistrationAlgorithmRunner.java | 4 ++-- ...gorithm.java => DataSetRegistrationHelper.java} | 6 +++--- .../cisd/etlserver/TransferredDataSetHandler.java | 14 +++++++------- .../cisd/etlserver/api/v1/PutDataSetExecutor.java | 4 ++-- .../etlserver/TransferredDataSetHandlerTest.java | 10 +++++----- 5 files changed, 19 insertions(+), 19 deletions(-) rename datastore_server/source/java/ch/systemsx/cisd/etlserver/{TransferredDataSetHandlerDataSetRegistrationAlgorithm.java => DataSetRegistrationHelper.java} (97%) diff --git a/datastore_server/source/java/ch/systemsx/cisd/etlserver/DataSetRegistrationAlgorithmRunner.java b/datastore_server/source/java/ch/systemsx/cisd/etlserver/DataSetRegistrationAlgorithmRunner.java index 79c175c3ed0..9775ded8e6a 100644 --- a/datastore_server/source/java/ch/systemsx/cisd/etlserver/DataSetRegistrationAlgorithmRunner.java +++ b/datastore_server/source/java/ch/systemsx/cisd/etlserver/DataSetRegistrationAlgorithmRunner.java @@ -47,13 +47,13 @@ public class DataSetRegistrationAlgorithmRunner private final IDataSetRegistrationAlgorithmRunnerDelegate delegate; public DataSetRegistrationAlgorithmRunner( - TransferredDataSetHandlerDataSetRegistrationAlgorithm registrationHelper) + DataSetRegistrationHelper registrationHelper) { this(registrationHelper, new NoOpDelegate()); } public DataSetRegistrationAlgorithmRunner( - TransferredDataSetHandlerDataSetRegistrationAlgorithm registrationHelper, + DataSetRegistrationHelper registrationHelper, IDataSetRegistrationAlgorithmRunnerDelegate delegate) { this(registrationHelper.getRegistrationAlgorithm(), delegate); diff --git a/datastore_server/source/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandlerDataSetRegistrationAlgorithm.java b/datastore_server/source/java/ch/systemsx/cisd/etlserver/DataSetRegistrationHelper.java similarity index 97% rename from datastore_server/source/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandlerDataSetRegistrationAlgorithm.java rename to datastore_server/source/java/ch/systemsx/cisd/etlserver/DataSetRegistrationHelper.java index b939c0a0105..89bff7aa93e 100644 --- a/datastore_server/source/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandlerDataSetRegistrationAlgorithm.java +++ b/datastore_server/source/java/ch/systemsx/cisd/etlserver/DataSetRegistrationHelper.java @@ -40,7 +40,7 @@ import ch.systemsx.cisd.openbis.dss.generic.shared.IEncapsulatedOpenBISService; import ch.systemsx.cisd.openbis.dss.generic.shared.dto.DataSetInformation; import ch.systemsx.cisd.openbis.generic.shared.basic.dto.DatabaseInstance; -public abstract class TransferredDataSetHandlerDataSetRegistrationAlgorithm implements +public abstract class DataSetRegistrationHelper implements DataSetRegistrationAlgorithm.IRollbackDelegate { @Private @@ -63,7 +63,7 @@ public abstract class TransferredDataSetHandlerDataSetRegistrationAlgorithm impl protected final DataSetRegistrationAlgorithm registrationAlgorithm; - public TransferredDataSetHandlerDataSetRegistrationAlgorithm(File incomingDataSetFile, + public DataSetRegistrationHelper(File incomingDataSetFile, IDelegatedActionWithResult<Boolean> cleanAftrewardsAction, IPreRegistrationAction preRegistrationAction, IPostRegistrationAction postRegistrationAction) @@ -83,7 +83,7 @@ public abstract class TransferredDataSetHandlerDataSetRegistrationAlgorithm impl this.incomingDataSetFile = algorithmState.getIncomingDataSetFile(); } - public TransferredDataSetHandlerDataSetRegistrationAlgorithm(File incomingDataSetFile, + public DataSetRegistrationHelper(File incomingDataSetFile, IDelegatedActionWithResult<Boolean> cleanAftrewardsAction, IPreRegistrationAction preRegistrationAction, IPostRegistrationAction postRegistrationAction, IDataSetInApplicationServerRegistrator appServerRegistrator) diff --git a/datastore_server/source/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandler.java b/datastore_server/source/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandler.java index 6ecf4feea80..ef7a50a0ccc 100644 --- a/datastore_server/source/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandler.java +++ b/datastore_server/source/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandler.java @@ -221,7 +221,7 @@ public final class TransferredDataSetHandler extends AbstractTopLevelDataSetRegi public List<DataSetInformation> handleDataSet(final File dataSet) { - final TransferredDataSetHandlerDataSetRegistrationAlgorithm registrationHelper = + final DataSetRegistrationHelper registrationHelper = createRegistrationHelper(dataSet); return new DataSetRegistrationAlgorithmRunner(registrationHelper).runAlgorithm(); } @@ -232,7 +232,7 @@ public final class TransferredDataSetHandler extends AbstractTopLevelDataSetRegi { dataSetInformation.setInstanceCode(getHomeDatabaseInstance().getCode()); dataSetInformation.setInstanceUUID(getHomeDatabaseInstance().getUuid()); - final TransferredDataSetHandlerDataSetRegistrationAlgorithm registrationHelper = + final DataSetRegistrationHelper registrationHelper = createRegistrationHelper(dataSet, dataSetInformation, registrator); return new DataSetRegistrationAlgorithmRunner(registrationHelper).runAlgorithm(); } @@ -289,7 +289,7 @@ public final class TransferredDataSetHandler extends AbstractTopLevelDataSetRegi // Helper class // - private TransferredDataSetHandlerDataSetRegistrationAlgorithm createRegistrationHelper(File file) + private DataSetRegistrationHelper createRegistrationHelper(File file) { if (useIsFinishedMarkerFile) { @@ -300,7 +300,7 @@ public final class TransferredDataSetHandler extends AbstractTopLevelDataSetRegi } } - private TransferredDataSetHandlerDataSetRegistrationAlgorithm createRegistrationHelper( + private DataSetRegistrationHelper createRegistrationHelper( File dataSet, DataSetInformation dataSetInformation, DataSetRegistrationAlgorithm.IDataSetInApplicationServerRegistrator registrator) { @@ -315,7 +315,7 @@ public final class TransferredDataSetHandler extends AbstractTopLevelDataSetRegi } } - private TransferredDataSetHandlerDataSetRegistrationAlgorithm createRegistrationHelperWithIsFinishedFile( + private DataSetRegistrationHelper createRegistrationHelperWithIsFinishedFile( final File isFinishedFile, final DataSetInformation dsInfo, DataSetRegistrationAlgorithm.IDataSetInApplicationServerRegistrator registrator) { @@ -353,7 +353,7 @@ public final class TransferredDataSetHandler extends AbstractTopLevelDataSetRegi } } - private TransferredDataSetHandlerDataSetRegistrationAlgorithm createRegistrationHelperWithQuietPeriodFilter( + private DataSetRegistrationHelper createRegistrationHelperWithQuietPeriodFilter( File incomingDataSetFile, final DataSetInformation dsInfo, DataSetRegistrationAlgorithm.IDataSetInApplicationServerRegistrator registrator) { @@ -435,7 +435,7 @@ public final class TransferredDataSetHandler extends AbstractTopLevelDataSetRegi return ok; } - private class RegistrationHelper extends TransferredDataSetHandlerDataSetRegistrationAlgorithm + private class RegistrationHelper extends DataSetRegistrationHelper { RegistrationHelper(TransferredDataSetHandler transferredDataSetHandler, diff --git a/datastore_server/source/java/ch/systemsx/cisd/etlserver/api/v1/PutDataSetExecutor.java b/datastore_server/source/java/ch/systemsx/cisd/etlserver/api/v1/PutDataSetExecutor.java index 541ba1dcb10..de57e604fb1 100644 --- a/datastore_server/source/java/ch/systemsx/cisd/etlserver/api/v1/PutDataSetExecutor.java +++ b/datastore_server/source/java/ch/systemsx/cisd/etlserver/api/v1/PutDataSetExecutor.java @@ -52,7 +52,7 @@ import ch.systemsx.cisd.etlserver.IPostRegistrationAction; import ch.systemsx.cisd.etlserver.IPreRegistrationAction; import ch.systemsx.cisd.etlserver.IStorageProcessor; import ch.systemsx.cisd.etlserver.ITypeExtractor; -import ch.systemsx.cisd.etlserver.TransferredDataSetHandlerDataSetRegistrationAlgorithm; +import ch.systemsx.cisd.etlserver.DataSetRegistrationHelper; import ch.systemsx.cisd.etlserver.validation.IDataSetValidator; import ch.systemsx.cisd.openbis.dss.generic.shared.IEncapsulatedOpenBISService; import ch.systemsx.cisd.openbis.dss.generic.shared.api.v1.FileInfoDssDTO; @@ -486,7 +486,7 @@ class PutDataSetExecutor implements IDataSetHandlerRpc * * @author Chandrasekhar Ramakrishnan */ - private class RegistrationHelper extends TransferredDataSetHandlerDataSetRegistrationAlgorithm + private class RegistrationHelper extends DataSetRegistrationHelper { /** * @param service The provider of global state for the data set registration algorithm diff --git a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandlerTest.java b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandlerTest.java index 6118a43a1a9..173b9c99a53 100644 --- a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandlerTest.java +++ b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/TransferredDataSetHandlerTest.java @@ -82,7 +82,7 @@ import ch.systemsx.cisd.openbis.generic.shared.dto.identifier.ExperimentIdentifi * @author Franz-Josef Elmer */ @Friend(toClasses = - { TransferredDataSetHandler.class, TransferredDataSetHandlerDataSetRegistrationAlgorithm.class, + { TransferredDataSetHandler.class, DataSetRegistrationHelper.class, IdentifiedDataStrategy.class, PluginTaskProviders.class, DssPropertyParametersUtil.class }) public final class TransferredDataSetHandlerTest extends AbstractFileSystemTestCase @@ -308,7 +308,7 @@ public final class TransferredDataSetHandlerTest extends AbstractFileSystemTestC private final String createLogMsgOfSuccess(final DataSetInformation dataSet) { - return TransferredDataSetHandlerDataSetRegistrationAlgorithm.SUCCESSFULLY_REGISTERED + "Data Set Code::" + return DataSetRegistrationHelper.SUCCESSFULLY_REGISTERED + "Data Set Code::" + DATA_SET_CODE + ";Data Set Type::" + DATA_SET_TYPE.getCode() + ";Experiment Identifier::" + dataSet.getExperimentIdentifier().toString() + ";Sample Identifier::" + dataSet.getSampleIdentifier().toString() @@ -466,7 +466,7 @@ public final class TransferredDataSetHandlerTest extends AbstractFileSystemTestC final DataSetInformation dataSet, final String dataSetCode, final String recipient) { expectations.one(mailClient).sendMessage( - String.format(TransferredDataSetHandlerDataSetRegistrationAlgorithm.EMAIL_SUBJECT_TEMPLATE, dataSet + String.format(DataSetRegistrationHelper.EMAIL_SUBJECT_TEMPLATE, dataSet .getExperimentIdentifier().getExperimentCode()), getNotificationEmailContent(dataSet, dataSetCode), null, null, recipient); } @@ -698,7 +698,7 @@ public final class TransferredDataSetHandlerTest extends AbstractFileSystemTestC createLogMsgOfSuccess(dataSetInformation)); final LogMonitoringAppender appender2 = LogMonitoringAppender.addAppender(LogCategory.NOTIFY, String.format( - TransferredDataSetHandlerDataSetRegistrationAlgorithm.DATA_SET_STORAGE_FAILURE_TEMPLATE, + DataSetRegistrationHelper.DATA_SET_STORAGE_FAILURE_TEMPLATE, dataSetInformation)); handler.handle(isFinishedFolder); @@ -756,7 +756,7 @@ public final class TransferredDataSetHandlerTest extends AbstractFileSystemTestC createLogMsgOfSuccess(dataSetInformation)); final LogMonitoringAppender appender2 = LogMonitoringAppender.addAppender(LogCategory.NOTIFY, String.format( - TransferredDataSetHandlerDataSetRegistrationAlgorithm.DATA_SET_REGISTRATION_FAILURE_TEMPLATE, + DataSetRegistrationHelper.DATA_SET_REGISTRATION_FAILURE_TEMPLATE, dataSetInformation)); handler.handle(isFinishedFolder); -- GitLab