Skip to content
Snippets Groups Projects
Commit b32c54b0 authored by felmer's avatar felmer
Browse files

LMS-2818 Add plugin types DATA_SOURCES and SERVICES

SVN: 24595
parent 5de057cb
No related branches found
No related tags found
No related merge requests found
...@@ -31,6 +31,7 @@ import ch.systemsx.cisd.common.logging.LogFactory; ...@@ -31,6 +31,7 @@ import ch.systemsx.cisd.common.logging.LogFactory;
import ch.systemsx.cisd.common.utilities.PropertyParametersUtil; import ch.systemsx.cisd.common.utilities.PropertyParametersUtil;
import ch.systemsx.cisd.common.utilities.PropertyParametersUtil.SectionProperties; import ch.systemsx.cisd.common.utilities.PropertyParametersUtil.SectionProperties;
import ch.systemsx.cisd.common.utilities.PropertyUtils; import ch.systemsx.cisd.common.utilities.PropertyUtils;
import ch.systemsx.cisd.openbis.dss.generic.shared.Constants;
import ch.systemsx.cisd.openbis.dss.generic.shared.dto.PluginServletConfig; import ch.systemsx.cisd.openbis.dss.generic.shared.dto.PluginServletConfig;
import ch.systemsx.cisd.openbis.dss.generic.shared.utils.DssPropertyParametersUtil; import ch.systemsx.cisd.openbis.dss.generic.shared.utils.DssPropertyParametersUtil;
...@@ -78,8 +79,6 @@ public final class ConfigParameters implements IServletPropertiesManager ...@@ -78,8 +79,6 @@ public final class ConfigParameters implements IServletPropertiesManager
public static final String KEYSTORE_KEY_PASSWORD_KEY = KEYSTORE + "key-password"; public static final String KEYSTORE_KEY_PASSWORD_KEY = KEYSTORE + "key-password";
static final String PLUGIN_SERVICES_LIST_KEY = "plugin-services";
static final String PLUGIN_SERVICE_CLASS_KEY = "class"; static final String PLUGIN_SERVICE_CLASS_KEY = "class";
static final String PLUGIN_SERVICE_PATH_KEY = "path"; static final String PLUGIN_SERVICE_PATH_KEY = "path";
...@@ -173,7 +172,7 @@ public final class ConfigParameters implements IServletPropertiesManager ...@@ -173,7 +172,7 @@ public final class ConfigParameters implements IServletPropertiesManager
pluginServlets = new LinkedHashMap<String, PluginServletConfig>(); pluginServlets = new LinkedHashMap<String, PluginServletConfig>();
SectionProperties[] pluginServicesProperties = SectionProperties[] pluginServicesProperties =
PropertyParametersUtil.extractSectionProperties(properties, PropertyParametersUtil.extractSectionProperties(properties,
PLUGIN_SERVICES_LIST_KEY, false); Constants.PLUGIN_SERVICES_LIST_KEY, false);
addServletsProperties("", pluginServicesProperties); addServletsProperties("", pluginServicesProperties);
webstartJarPath = webstartJarPath =
......
...@@ -16,7 +16,6 @@ ...@@ -16,7 +16,6 @@
package ch.systemsx.cisd.openbis.dss.generic.shared; package ch.systemsx.cisd.openbis.dss.generic.shared;
import ch.rinn.restrictions.Private;
/** /**
* Some common constants. * Some common constants.
...@@ -32,11 +31,15 @@ public class Constants ...@@ -32,11 +31,15 @@ public class Constants
public static final String INPUT_THREAD_NAMES = "inputs"; public static final String INPUT_THREAD_NAMES = "inputs";
/** property with repotring plugins names separated by delimiter */ /** property with repotring plugins names separated by delimiter */
@Private
public static final String REPORTING_PLUGIN_NAMES = "reporting-plugins"; public static final String REPORTING_PLUGIN_NAMES = "reporting-plugins";
/** property with processing plugins names separated by delimiter */ /** property with processing plugins names separated by delimiter */
@Private
public static final String PROCESSING_PLUGIN_NAMES = "processing-plugins"; public static final String PROCESSING_PLUGIN_NAMES = "processing-plugins";
/** Key of service property which is a list of data source IDs. */
public static final String DATA_SOURCES_KEY = "data-sources";
/** Key of service property which is a list of servlet services. */
public static final String PLUGIN_SERVICES_LIST_KEY = "plugin-services";
} }
...@@ -70,8 +70,6 @@ import ch.systemsx.cisd.openbis.dss.generic.shared.utils.DssPropertyParametersUt ...@@ -70,8 +70,6 @@ import ch.systemsx.cisd.openbis.dss.generic.shared.utils.DssPropertyParametersUt
*/ */
public class DataSourceProvider implements IDataSourceProvider public class DataSourceProvider implements IDataSourceProvider
{ {
static final String DATA_SOURCES_KEY = "data-sources";
private static final String DATA_SOURCE_FACTORY_CLASS_KEY = "factory-class"; private static final String DATA_SOURCE_FACTORY_CLASS_KEY = "factory-class";
private static final Logger operationLog = private static final Logger operationLog =
...@@ -87,7 +85,7 @@ public class DataSourceProvider implements IDataSourceProvider ...@@ -87,7 +85,7 @@ public class DataSourceProvider implements IDataSourceProvider
dataSources = new HashMap<String, DataSource>(); dataSources = new HashMap<String, DataSource>();
SectionProperties[] props = SectionProperties[] props =
PropertyParametersUtil PropertyParametersUtil
.extractSectionProperties(properties, DATA_SOURCES_KEY, false); .extractSectionProperties(properties, Constants.DATA_SOURCES_KEY, false);
for (SectionProperties sectionProperties : props) for (SectionProperties sectionProperties : props)
{ {
Properties dataSourceProperties = sectionProperties.getProperties(); Properties dataSourceProperties = sectionProperties.getProperties();
......
...@@ -64,6 +64,8 @@ class CorePluginsInjector ...@@ -64,6 +64,8 @@ class CorePluginsInjector
enum PluginType enum PluginType
{ {
DROP_BOXES("drop-boxes", Constants.INPUT_THREAD_NAMES), DROP_BOXES("drop-boxes", Constants.INPUT_THREAD_NAMES),
DATA_SOURCES("data-sources", Constants.DATA_SOURCES_KEY),
SERVICES("services", Constants.PLUGIN_SERVICES_LIST_KEY),
REPORTING_PLUGINS("reporting-plugins", Constants.REPORTING_PLUGIN_NAMES), REPORTING_PLUGINS("reporting-plugins", Constants.REPORTING_PLUGIN_NAMES),
PROCESSING_PLUGINS("processing-plugins", Constants.PROCESSING_PLUGIN_NAMES), PROCESSING_PLUGINS("processing-plugins", Constants.PROCESSING_PLUGIN_NAMES),
MAINTENANCE_TASKS("maintenance-tasks", MaintenanceTaskUtils.DEFAULT_MAINTENANCE_PLUGINS_PROPERTY_NAME), MAINTENANCE_TASKS("maintenance-tasks", MaintenanceTaskUtils.DEFAULT_MAINTENANCE_PLUGINS_PROPERTY_NAME),
......
...@@ -23,6 +23,7 @@ import org.testng.AssertJUnit; ...@@ -23,6 +23,7 @@ import org.testng.AssertJUnit;
import org.testng.annotations.Test; import org.testng.annotations.Test;
import ch.systemsx.cisd.common.exceptions.ConfigurationFailureException; import ch.systemsx.cisd.common.exceptions.ConfigurationFailureException;
import ch.systemsx.cisd.openbis.dss.generic.shared.Constants;
import ch.systemsx.cisd.openbis.dss.generic.shared.dto.PluginServletConfig; import ch.systemsx.cisd.openbis.dss.generic.shared.dto.PluginServletConfig;
/** /**
...@@ -59,7 +60,7 @@ public class ConfigParametersTest extends AssertJUnit ...@@ -59,7 +60,7 @@ public class ConfigParametersTest extends AssertJUnit
public void testPluginServices() public void testPluginServices()
{ {
Properties properties = createMandatoryProperties(); Properties properties = createMandatoryProperties();
properties.setProperty(ConfigParameters.PLUGIN_SERVICES_LIST_KEY, "s1, s2"); properties.setProperty(Constants.PLUGIN_SERVICES_LIST_KEY, "s1, s2");
properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class1"); properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class1");
properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_PATH_KEY, "path1"); properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_PATH_KEY, "path1");
properties.setProperty("s2." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class2"); properties.setProperty("s2." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class2");
...@@ -80,7 +81,7 @@ public class ConfigParametersTest extends AssertJUnit ...@@ -80,7 +81,7 @@ public class ConfigParametersTest extends AssertJUnit
public void testPluginServicesWithDuplicatedPath() public void testPluginServicesWithDuplicatedPath()
{ {
Properties properties = createMandatoryProperties(); Properties properties = createMandatoryProperties();
properties.setProperty(ConfigParameters.PLUGIN_SERVICES_LIST_KEY, "s1, s2"); properties.setProperty(Constants.PLUGIN_SERVICES_LIST_KEY, "s1, s2");
properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class1"); properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class1");
properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_PATH_KEY, "path1"); properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_PATH_KEY, "path1");
properties.setProperty("s2." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class2"); properties.setProperty("s2." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class2");
...@@ -117,7 +118,7 @@ public class ConfigParametersTest extends AssertJUnit ...@@ -117,7 +118,7 @@ public class ConfigParametersTest extends AssertJUnit
public void testAddServletPropertiesWithExistingPath() public void testAddServletPropertiesWithExistingPath()
{ {
Properties properties = createMandatoryProperties(); Properties properties = createMandatoryProperties();
properties.setProperty(ConfigParameters.PLUGIN_SERVICES_LIST_KEY, "s1"); properties.setProperty(Constants.PLUGIN_SERVICES_LIST_KEY, "s1");
properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class1"); properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_CLASS_KEY, "class1");
properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_PATH_KEY, "path1"); properties.setProperty("s1." + ConfigParameters.PLUGIN_SERVICE_PATH_KEY, "path1");
ConfigParameters configParameters = new ConfigParameters(properties); ConfigParameters configParameters = new ConfigParameters(properties);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment