diff --git a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/CodeExtractortTestCase.java b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/CodeExtractortTestCase.java index 6c28b33d9018371984b5ee7a9be8edf2e698c2f3..de2ac726668222ef2df7cbe2e450731f037f7809 100644 --- a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/CodeExtractortTestCase.java +++ b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/CodeExtractortTestCase.java @@ -21,33 +21,32 @@ package ch.systemsx.cisd.etlserver; */ public abstract class CodeExtractortTestCase { - protected static final String PREFIX = IDataSetInfoExtractor.EXTRACTOR_KEY + "."; protected static final String ENTITY_SEPARATOR = - PREFIX + DefaultDataSetInfoExtractor.ENTITY_SEPARATOR_PROPERTY_NAME; - + DefaultDataSetInfoExtractor.ENTITY_SEPARATOR_PROPERTY_NAME; + protected static final String SUB_ENTITY_SEPARATOR = - PREFIX + DefaultDataSetInfoExtractor.SUB_ENTITY_SEPARATOR_PROPERTY_NAME; + DefaultDataSetInfoExtractor.SUB_ENTITY_SEPARATOR_PROPERTY_NAME; protected static final String INDEX_OF_SAMPLE_CODE = - PREFIX + DefaultDataSetInfoExtractor.INDEX_OF_SAMPLE_CODE; + DefaultDataSetInfoExtractor.INDEX_OF_SAMPLE_CODE; protected static final String INDEX_OF_EXPERIMENT_IDENTIFIER = - PREFIX + DefaultDataSetInfoExtractor.INDEX_OF_EXPERIMENT_IDENTIFIER; - + DefaultDataSetInfoExtractor.INDEX_OF_EXPERIMENT_IDENTIFIER; + protected static final String INDEX_OF_PARENT_DATA_SET_CODE = - PREFIX + DefaultDataSetInfoExtractor.INDEX_OF_PARENT_DATA_SET_CODES; + DefaultDataSetInfoExtractor.INDEX_OF_PARENT_DATA_SET_CODES; protected static final String INDEX_OF_DATA_PRODUCER_CODE = - PREFIX + DefaultDataSetInfoExtractor.INDEX_OF_DATA_PRODUCER_CODE; + DefaultDataSetInfoExtractor.INDEX_OF_DATA_PRODUCER_CODE; protected static final String INDEX_OF_DATA_PRODUCTION_DATE = - PREFIX + DefaultDataSetInfoExtractor.INDEX_OF_DATA_PRODUCTION_DATE; + DefaultDataSetInfoExtractor.INDEX_OF_DATA_PRODUCTION_DATE; protected static final String DATA_PRODUCTION_DATE_FORMAT = - PREFIX + DefaultDataSetInfoExtractor.DATA_PRODUCTION_DATE_FORMAT; + DefaultDataSetInfoExtractor.DATA_PRODUCTION_DATE_FORMAT; protected static final String DATA_SET_PROPERTIES_FILE_NAME_KEY = - PREFIX + DefaultDataSetInfoExtractor.DATA_SET_PROPERTIES_FILE_NAME_KEY; - + DefaultDataSetInfoExtractor.DATA_SET_PROPERTIES_FILE_NAME_KEY; + } diff --git a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/DefaultDataSetInfoExtractorTest.java b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/DefaultDataSetInfoExtractorTest.java index cb485f04cc46d243f7cb2de60668844bcf9f0835..976bb9b4770721d5f54cb91bba622aa1eefb77c8 100644 --- a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/DefaultDataSetInfoExtractorTest.java +++ b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/DefaultDataSetInfoExtractorTest.java @@ -58,7 +58,7 @@ public final class DefaultDataSetInfoExtractorTest extends CodeExtractortTestCas public void testExtractContainedSampleIdentifier() { Properties properties = new Properties(); - properties.setProperty(asPropertyName(GROUP_CODE), "my-group"); + properties.setProperty(GROUP_CODE, "my-group"); final IDataSetInfoExtractor extractor = new DefaultDataSetInfoExtractor(properties); final DataSetInformation dsInfo1 = @@ -77,7 +77,7 @@ public final class DefaultDataSetInfoExtractorTest extends CodeExtractortTestCas { Properties properties = new Properties(); properties.setProperty(INDEX_OF_EXPERIMENT_IDENTIFIER, "1"); - properties.setProperty(asPropertyName(GROUP_CODE), "my-group"); + properties.setProperty(GROUP_CODE, "my-group"); final IDataSetInfoExtractor extractor = new DefaultDataSetInfoExtractor(properties); final DataSetInformation dsInfo = @@ -147,29 +147,24 @@ public final class DefaultDataSetInfoExtractorTest extends CodeExtractortTestCas public void testGroupCodeGlobal() { final Properties properties = new Properties(); - properties.setProperty(asPropertyName(INDEX_OF_GROUP_CODE), ""); + properties.setProperty(INDEX_OF_GROUP_CODE, ""); String globalGroupCode = "g1"; - properties.setProperty(asPropertyName(GROUP_CODE), globalGroupCode); + properties.setProperty(GROUP_CODE, globalGroupCode); final DataSetInformation dsInfo = extractDatasetInfo(properties, "sampleCode"); assertEquals(globalGroupCode, dsInfo.getSpaceCode()); } - private static String asPropertyName(String propertyName) - { - return IDataSetInfoExtractor.EXTRACTOR_KEY + "." + propertyName; - } - @Test // set group code as the local setting for one datasets. It should override the global setting. public void testGroupCodeDatasetSpecific() { final Properties properties = new Properties(); - properties.setProperty(asPropertyName(INDEX_OF_GROUP_CODE), "-2"); + properties.setProperty(INDEX_OF_GROUP_CODE, "-2"); String globalGroupCode = "globalGroup"; - properties.setProperty(asPropertyName(GROUP_CODE), globalGroupCode); + properties.setProperty(GROUP_CODE, globalGroupCode); String localGroupCode = "localGroup"; String fileName = @@ -185,8 +180,8 @@ public final class DefaultDataSetInfoExtractorTest extends CodeExtractortTestCas public void testGroupCodeDatasetSpecificMissingFails() { final Properties properties = new Properties(); - properties.setProperty(asPropertyName(INDEX_OF_GROUP_CODE), "-2"); - properties.setProperty(asPropertyName(GROUP_CODE), "any"); + properties.setProperty(INDEX_OF_GROUP_CODE, "-2"); + properties.setProperty(GROUP_CODE, "any"); extractDatasetInfo(properties, "sampleCode"); } diff --git a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/threev/DataSetInfoExtractorForDataAcquisitionTest.java b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/threev/DataSetInfoExtractorForDataAcquisitionTest.java index 3c661f9a31ddbe5342667c3d6b3523806f78040f..4ab98db8d3bffede2c109aca0e8517d31b29ec6e 100644 --- a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/threev/DataSetInfoExtractorForDataAcquisitionTest.java +++ b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/threev/DataSetInfoExtractorForDataAcquisitionTest.java @@ -36,10 +36,10 @@ import ch.systemsx.cisd.openbis.dss.generic.shared.dto.DataSetInformation; public class DataSetInfoExtractorForDataAcquisitionTest extends CodeExtractortTestCase { private static final String INDICES_OF_DATA_SET_CODE_ENTITIES = - PREFIX + DataSetInfoExtractorForDataAcquisition.INDICES_OF_DATA_SET_CODE_ENTITIES; + DataSetInfoExtractorForDataAcquisition.INDICES_OF_DATA_SET_CODE_ENTITIES; private static final String DATA_SET_CODE_ENTITIES_GLUE = - PREFIX + AbstractDataSetInfoExtractorFor3V.DATA_SET_CODE_ENTITIES_GLUE; + AbstractDataSetInfoExtractorFor3V.DATA_SET_CODE_ENTITIES_GLUE; @Test public void testHappyCaseWithOnlyMandatoryPorperties() diff --git a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/threev/DataSetInfoExtractorForImageAnalysisTest.java b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/threev/DataSetInfoExtractorForImageAnalysisTest.java index 2680989d431bccfc59085d2564a75be35a324dfe..4b4330019a04a9c64ccbabd7103e51b57086a519 100644 --- a/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/threev/DataSetInfoExtractorForImageAnalysisTest.java +++ b/datastore_server/sourceTest/java/ch/systemsx/cisd/etlserver/threev/DataSetInfoExtractorForImageAnalysisTest.java @@ -35,7 +35,7 @@ import ch.systemsx.cisd.openbis.dss.generic.shared.dto.DataSetInformation; public class DataSetInfoExtractorForImageAnalysisTest extends CodeExtractortTestCase { private static final String INDICES_OF_PARENT_DATA_SET_CODE_ENTITIES = - PREFIX + DataSetInfoExtractorForImageAnalysis.INDICES_OF_PARENT_DATA_SET_CODE_ENTITIES; + DataSetInfoExtractorForImageAnalysis.INDICES_OF_PARENT_DATA_SET_CODE_ENTITIES; @Test public void testHappyCaseWithOnlyMandatoryPorperties()