--- a/common/src/main/java/com/redhat/thermostat/common/config/StartupConfiguration.java	Thu Mar 29 17:15:22 2012 +0200
+++ b/common/src/main/java/com/redhat/thermostat/common/config/StartupConfiguration.java	Thu Mar 29 17:19:01 2012 +0200
@@ -36,246 +36,7 @@
 
 package com.redhat.thermostat.common.config;
 
-import java.io.FileReader;
-import java.io.IOException;
-import java.io.Reader;
-import java.net.InetAddress;
-import java.net.UnknownHostException;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.Map.Entry;
-import java.util.Properties;
-import java.util.logging.Level;
-import java.util.logging.Logger;
-
-import com.redhat.thermostat.common.config.Defaults;
-import com.redhat.thermostat.common.Constants;
-import com.redhat.thermostat.common.LaunchException;
-import com.redhat.thermostat.common.utils.LoggingUtils;
-
-public class StartupConfiguration {
-
-    private static Logger logger = LoggingUtils.getLogger(StartupConfiguration.class);
-
-    private final long startTimestamp;
-
-    private Properties props = new Properties();
-    private Level logLevel;
-    private boolean localMode;
-    private int mongodPort;
-    private String mongoScript;
-    private int mongosPort;
-    private String databaseURI;
-    private String completeDatabaseURI;
-    private Arguments arguments;
-
-    private String hostname;
-
-    public StartupConfiguration(long startTime, String[] args) throws LaunchException {
-
-        initFromDefaults();
-        initFromArguments(args);
-        initFromProperties();
-
-        try {
-            InetAddress addr = InetAddress.getLocalHost();
-            hostname = addr.getCanonicalHostName();
-        } catch (UnknownHostException e) {
-            logger.log(Level.FINE, "Error determining local hostname.");
-        }
-
-        startTimestamp = startTime;
-    }
-
-    private void initFromDefaults() {
-        logLevel = Defaults.LOGGING_LEVEL;
-        localMode = Defaults.LOCAL_MODE;
-        mongodPort = Defaults.MONGOD_PORT;
-        mongosPort = Defaults.MONGOS_PORT;
-        databaseURI = Defaults.DATABASE_URI;
-    }
-
-    private void initFromProperties() throws LaunchException {
-        if (props.getProperty(Constants.AGENT_PROPERTY_MONGOD_PORT) != null) {
-            mongodPort = Integer.valueOf(props.getProperty(Constants.AGENT_PROPERTY_MONGOD_PORT));
-        }
-        if (props.getProperty(Constants.AGENT_PROPERTY_MONGO_LAUNCH_SCRIPT) != null) {
-            mongoScript = props.getProperty(Constants.AGENT_PROPERTY_MONGO_LAUNCH_SCRIPT);
-            logger.finest("Mongo launch script at: " + mongoScript);
-        } else {
-            throw new LaunchException("No mongo launch script in properties.");
-        }
-        if (props.getProperty(Constants.AGENT_PROPERTY_MONGOS_PORT) != null) {
-            mongosPort = Integer.valueOf(props.getProperty(Constants.AGENT_PROPERTY_MONGOS_PORT));
-        }
-    }
-
-    private void initFromArguments(String[] args) throws LaunchException {
-        arguments = new Arguments(args);
-        if (arguments.isModeSpecified()) {
-            localMode = arguments.getLocalMode();
-        }
-        if (arguments.isLogLevelSpecified()) {
-            logLevel = arguments.getLogLevel();
-        }
-    }
-    
-    public Level getLogLevel() {
-        return logLevel;
-    }
-
-    public String getMongoLaunchScript() {
-        return mongoScript;
-    }
-
-    public synchronized String getDatabaseURIAsString() {
-        if (completeDatabaseURI == null) {
-            if (localMode) {
-                completeDatabaseURI = getDBUriString();
-            } else {
-                completeDatabaseURI = getClusterDBUriString();
-            }
-        }
-        return completeDatabaseURI;
-    }
-    
-    public String getDBUriString() {
-        return databaseURI + ":" + mongodPort;
-    }
-    
-    // FIXME: this needs to go away, it's an implementation detail
-    public String getClusterDBUriString() {
-        return databaseURI + ":" + mongosPort;
-    }    
-    
-
-    public String getHostname() {
-        return hostname;
-    }
+public interface StartupConfiguration {
 
-    public boolean getLocalMode() {
-        boolean local = false;
-        try {
-            local = arguments.getLocalMode();
-        } catch (IllegalStateException ise) {
-            local = false;
-        }
-        return local;
-    }
-
-    public List<String> getStartupBackendClassNames() {
-        String fullPropertyString = props.getProperty(Constants.AGENT_PROPERTY_BACKENDS);
-        if ((fullPropertyString == null) // Avoid NPE
-                || (fullPropertyString.length() == 0)) { /* If we do the split() on this empty string,
-                                                          * it will produce an array of size 1 containing
-                                                          * the empty string, which we do not want.
-                                                          */
-            return new ArrayList<String>();
-        } else {
-            return Arrays.asList(fullPropertyString.trim().split(","));
-        }
-    }
-
-    public Map<String, String> getStartupBackendConfigMap(String backendName) {
-        String prefix = backendName + ".";
-        Map<String, String> configMap = new HashMap<String, String>();
-        for (Entry<Object, Object> e : props.entrySet()) {
-            String key = (String) e.getKey();
-            if (key.startsWith(prefix)) {
-                String value = (String) e.getValue();
-                String mapKey = key.substring(prefix.length());
-                configMap.put(mapKey, value);
-            }
-        }
-        return configMap;
-    }
-
-    public long getStartTime() {
-        return startTimestamp;
-    }
-
-    /**
-     * Exposes the command line arguments in a more object-oriented style.
-     * <p>
-     * Please check that an option was specified (using the various is*Specified() methods) before using its value.
-     */
-    private class Arguments {
-        private final boolean localMode;
-        private final boolean modeSpecified;
-        private final Level logLevel;
-        private final boolean logLevelSpecified;
-
-        public Arguments(String[] args) throws LaunchException {
-            boolean local = false;
-            boolean explicitMode = false;
-            Level level = null;
-            boolean explicitLogLevel = false;
-            boolean noProps = true;
-            for (int index = 0; index < args.length; index++) {
-                if (args[index].equals(Constants.AGENT_ARGUMENT_LOGLEVEL)) {
-                    index++;
-                    if (index < args.length) {
-                        try {
-                            level = Level.parse(args[index].toUpperCase());
-                            explicitLogLevel = true;
-                        } catch (IllegalArgumentException iae) {
-                            throw new LaunchException("Invalid argument for " + Constants.AGENT_ARGUMENT_LOGLEVEL, iae);
-                        }
-                    } else {
-                        throw new LaunchException("Missing argument for " + Constants.AGENT_ARGUMENT_LOGLEVEL);
-                    }
-                } else if (args[index].equals(Constants.AGENT_ARGUMENT_LOCAL)) {
-                    explicitMode = true;
-                    local = true;
-                } else if (args[index].equals(Constants.AGENT_ARGUMENT_PROPERTIES)) {
-                    logger.finest("Properties argument specified.");
-                    index++;
-                    if (index < args.length) {
-                        String propFile = args[index];
-                        logger.finest("Properties file: " + propFile);
-                        try (Reader reader = new FileReader(propFile)) {
-                            props.load(reader);
-                        } catch (IOException ioe) {
-                            throw new LaunchException("Unable to read properties file at " + propFile);
-                        }
-                        noProps = false;
-                    } else {
-                        throw new LaunchException("Missing argument for " + Constants.AGENT_ARGUMENT_PROPERTIES);
-                    }
-                }
-            }
-            if (noProps) {
-                throw new LaunchException("Required properties file not specified.");
-            }
-            logLevel = level;
-            logLevelSpecified = explicitLogLevel;
-            localMode = local;
-            modeSpecified = explicitMode;
-        }
-
-        public boolean isModeSpecified() {
-            return modeSpecified;
-        }
-
-        public boolean getLocalMode() {
-            if (!isModeSpecified()) {
-                throw new IllegalStateException("local mode is not valid");
-            }
-            return localMode;
-        }
-
-        public boolean isLogLevelSpecified() {
-            return logLevelSpecified;
-        }
-
-        public Level getLogLevel() {
-            if (!isLogLevelSpecified()) {
-                throw new IllegalStateException("log level not explicitly specified");
-            }
-            return logLevel;
-        }
-    }
+    String getDBConnectionString();
 }