diff --git a/dsf-gdb/org.eclipse.cdt.dsf.gdb.multicorevisualizer.ui/src/org/eclipse/cdt/dsf/gdb/multicorevisualizer/internal/utils/MementoUtils.java b/dsf-gdb/org.eclipse.cdt.dsf.gdb.multicorevisualizer.ui/src/org/eclipse/cdt/dsf/gdb/multicorevisualizer/internal/utils/MementoUtils.java index 806665a6519..c30b054e14f 100644 --- a/dsf-gdb/org.eclipse.cdt.dsf.gdb.multicorevisualizer.ui/src/org/eclipse/cdt/dsf/gdb/multicorevisualizer/internal/utils/MementoUtils.java +++ b/dsf-gdb/org.eclipse.cdt.dsf.gdb.multicorevisualizer.ui/src/org/eclipse/cdt/dsf/gdb/multicorevisualizer/internal/utils/MementoUtils.java @@ -37,20 +37,19 @@ import org.xml.sax.helpers.DefaultHandler; /** encodes and decodes memento to and from different data types; list, map, String*/ public class MementoUtils { - protected static final String ROOT_ELEMENT_TAGNAME = "root_element"; //$NON-NLS-1$ protected static final String ELEMENT_TAGNAME = "elem"; //$NON-NLS-1$ - protected static final String ATTRIBUTE_NAME = "value"; //$NON-NLS-1$ - + protected static final String ATTRIBUTE_KEY = "key"; //$NON-NLS-1$ + protected static final String ATTRIBUTE_VALUE = "value"; //$NON-NLS-1$ + /** Returns a XML memento, that encodes a single String parameter */ public static String encodeStringIntoMemento(String str) { - List list = new ArrayList(); + List list = new ArrayList<>(); list.add(str); return encodeListIntoMemento(list); } - - + /** Returns a single String parameter, decoded from a XML memento */ public static String decodeStringFromMemento(String memento) { return decodeListFromMemento(memento).get(0); @@ -71,7 +70,9 @@ public class MementoUtils { // create one XML element per map entry for (String key : keyPairValues.keySet()) { Element elem = doc.createElement(ELEMENT_TAGNAME); - elem.setAttribute(key, keyPairValues.get(key)); + // store key and value as values of 2 attributes + elem.setAttribute(ATTRIBUTE_KEY, key); + elem.setAttribute(ATTRIBUTE_VALUE, keyPairValues.get(key)); rootElement.appendChild(elem); } @@ -96,7 +97,7 @@ public class MementoUtils { /** Returns a Map of Strings, decoded from a XML memento */ public static Map decodeMapFromMemento(String memento) { - Map keyPairValues = new HashMap(); + Map keyPairValues = new HashMap<>(); Element root = null; DocumentBuilder parser; @@ -110,20 +111,26 @@ public class MementoUtils { if (node.getNodeType() == Node.ELEMENT_NODE) { Element elem = (Element) node; NamedNodeMap nodeMap = elem.getAttributes(); + String key = null; + String value = null; for(int idx = 0; idx < nodeMap.getLength(); idx++) { Node attrNode = nodeMap.item(idx); if (attrNode.getNodeType() == Node.ATTRIBUTE_NODE) { Attr attr = (Attr) attrNode; - String key = attr.getName(); - String value = attr.getValue(); - if (key != null && value != null) { - keyPairValues.put(key, value); + if (attr.getName().equals(ATTRIBUTE_KEY)) { + key = attr.getValue(); } - else { - throw new Exception(); + else if (attr.getName().equals(ATTRIBUTE_VALUE)) { + value = attr.getValue(); } } } + if (key != null && value != null) { + keyPairValues.put(key, value); + } + else { + throw new Exception(); + } } } } catch (Exception e) { @@ -148,7 +155,7 @@ public class MementoUtils { // create one XML element per list entry to save for (String lbl : labels) { Element elem = doc.createElement(ELEMENT_TAGNAME); - elem.setAttribute(ATTRIBUTE_NAME, lbl); + elem.setAttribute(ATTRIBUTE_VALUE, lbl); rootElement.appendChild(elem); } @@ -172,7 +179,7 @@ public class MementoUtils { /** Returns a List of Strings, decoded from a XML memento */ public static List decodeListFromMemento(String memento) { - List list = new ArrayList(); + List list = new ArrayList<>(); Element root = null; DocumentBuilder parser; @@ -185,7 +192,7 @@ public class MementoUtils { Node node = nodeList.item(i); if (node.getNodeType() == Node.ELEMENT_NODE) { Element elem = (Element) node; - String value = elem.getAttribute(ATTRIBUTE_NAME); + String value = elem.getAttribute(ATTRIBUTE_VALUE); if (value != null) { list.add(value); } diff --git a/dsf-gdb/org.eclipse.cdt.dsf.gdb.multicorevisualizer.ui/src/org/eclipse/cdt/dsf/gdb/multicorevisualizer/internal/utils/PersistentSettingsManager.java b/dsf-gdb/org.eclipse.cdt.dsf.gdb.multicorevisualizer.ui/src/org/eclipse/cdt/dsf/gdb/multicorevisualizer/internal/utils/PersistentSettingsManager.java index f7a9364ed7f..168f884e1d8 100644 --- a/dsf-gdb/org.eclipse.cdt.dsf.gdb.multicorevisualizer.ui/src/org/eclipse/cdt/dsf/gdb/multicorevisualizer/internal/utils/PersistentSettingsManager.java +++ b/dsf-gdb/org.eclipse.cdt.dsf.gdb.multicorevisualizer.ui/src/org/eclipse/cdt/dsf/gdb/multicorevisualizer/internal/utils/PersistentSettingsManager.java @@ -7,30 +7,100 @@ * * Contributors: * Marc Dumais (Ericsson) - initial API and implementation (bug 460837) + * Marc Dumais (Ericsson) - Bug 462353 *******************************************************************************/ package org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.utils; + import java.security.InvalidParameterException; +import java.util.ArrayList; +import java.util.HashMap; +import java.util.List; +import java.util.Map; import org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.ui.MulticoreVisualizerUIPlugin; import org.eclipse.core.runtime.preferences.IEclipsePreferences; import org.osgi.service.prefs.BackingStoreException; -/** This class manages one or more PersistentSetting objects, using a commmon - * name-space and optionally an instance id so that multiple instances can - * each have their own version of the parameter persisted */ + +/** + * This class manages one or more PersistentParameter, PersistentListParameter, + * PersistentMapParameter objects, using a common name-space and optionally an + * instance id so that multiple instances can each have their own version of + * the parameter persisted + */ public class PersistentSettingsManager { - // TODO: add a way to notify clients that the value of a global (shared) parameter - // has been updated, and that they should re-read it. - /** Class for a specific persistent parameter */ - public class PersistentParameter { - private String m_storeKey; + /** Base class for a persistent parameter */ + private abstract class AbstractPersistentParameter { + protected final Class myClazz; + protected final boolean m_perInstance; + protected final String m_storeKey; + + /** Constructor */ + public AbstractPersistentParameter(Class clazz, boolean perInstance, String storeKey) { + myClazz = clazz; + m_perInstance = perInstance; + m_storeKey = storeKey; + } + + // accessors + + /** Returns whether this parameter is persisted independently for each client instance */ + public boolean isPerInstance() { + return m_perInstance; + } + + /** Returns the class of the parameter */ + public Class getClazz() { + return myClazz; + } + + protected String getStoreKey() { + return m_storeKey; + } + + // misc + + @SuppressWarnings("unchecked") + /** Converts a value from a String to its expected generic type. This is a base + * implementation that converts some base types - Use/Override as needed for more complex + * types, such as List or Map of these types */ + protected T convertToT(String val) { + // TODO: Add other types? Float, etc + if (String.class.isAssignableFrom(getClazz())) { + return (T) val; + } + else if (Integer.class.isAssignableFrom(getClazz())) { + return (T) Integer.valueOf(val); + } + else if (Boolean.class.isAssignableFrom(getClazz())) { + return (T) Boolean.valueOf(val); + } + return null; + } + + /** Returns whether the wanted Class type is supported, to use as a persistent parameter */ + protected boolean isTypeSupported(Class clazz) { + // TODO: Add other types? Float, etc + if (String.class.isAssignableFrom(clazz) || + Integer.class.isAssignableFrom(clazz) || + Boolean.class.isAssignableFrom(clazz)) + { + return true; + } + return false; + } + + // TODO: add a way to notify clients that the value of a global (shared) parameter + // has been updated, and that they should re-read it. + } + + /** Class for a persistent parameter */ + public class PersistentParameter extends AbstractPersistentParameter { private T m_value; private T m_defaultValue; - private Class myClazz; - private boolean m_perInstance; /** * Constructor @@ -40,9 +110,7 @@ public class PersistentSettingsManager { * @param storeKey : The key used to store the parameter in the store */ public PersistentParameter(Class clazz, boolean perInstance, String storeKey) { - myClazz = clazz; - m_perInstance = perInstance; - m_storeKey = storeKey; + super(clazz, perInstance, storeKey); } /** Sets the default value to use if no persistent @@ -51,36 +119,46 @@ public class PersistentSettingsManager { m_defaultValue = defaultValue; } - /** Sets the persistent value to set */ + /** Sets the value to persist */ public void set(T value) { m_value = value; // save value in preference store persistParameter(value); } - /** Gets the persistent value, if found, else the default value */ + /** Returns the persistent value, if found, else the default value */ public T value() { if (m_value == null) { // attempt to get the value from the preference store m_value = restoreParameter(); } // parameter has one value for any/all instances - else if(!m_perInstance) { + else if(!isPerInstance()) { // do not rely on cached value, since another instance might have // changed it - reread from data store m_value = restoreParameter(); } - return (m_value!=null)? m_value : m_defaultValue; + return (m_value == null)? m_defaultValue : m_value; + } + + /** + * Gets the persistent value, optionally forcing re-reading stored value + * @param forceRefresh whether to force to re-read memento in case value changed + */ + public T value(boolean forceRefresh) { + if (forceRefresh) { + m_value = null; + } + return value(); } /** Attempts to find the parameter in the preference store. Returns null if not found */ private T restoreParameter() { IEclipsePreferences store = MulticoreVisualizerUIPlugin.getEclipsePreferenceStore(); - String memento = store.get(m_storeKey, null); + String memento = store.get(getStoreKey(), null); if (memento == null) return null; String val = MementoUtils.decodeStringFromMemento(memento); - T convertedVal = convertToT(val); return convertedVal; } @@ -93,7 +171,87 @@ public class PersistentSettingsManager { // save memento in store if (memento != null) { IEclipsePreferences store = MulticoreVisualizerUIPlugin.getEclipsePreferenceStore(); - store.put(m_storeKey, memento); + store.put(getStoreKey(), memento); + try { + store.flush(); + } catch (BackingStoreException e) { + e.printStackTrace(); + } + } + } + } + + /** Class for a persistent {@literal List} parameter */ + public class PersistentListParameter extends AbstractPersistentParameter { + private List m_value; + private List m_defaultValue; + + public PersistentListParameter(Class clazz, boolean perInstance, String storeKey) { + super(clazz, perInstance, storeKey); + } + + /** Sets the default value to use if no persistent + * value is found for this parameter */ + public void setDefault(List defaultValues) { + m_defaultValue = defaultValues; + } + + /** Sets the value to persist */ + public void set(List values) { + m_value = values; + // save value in preference store + persistParameter(values); + } + + /** Returns the persistent value, if found, else the default value */ + public List value() { + if (m_value == null) { + // attempt to get the value from the preference store + m_value = restoreParameter(); + } + // parameter has one value for any/all instances + else if(!isPerInstance()) { + // do not rely on cached value, since another instance might have + // changed it - reread from data store + m_value = restoreParameter(); + } + return (m_value == null)? m_defaultValue : m_value ; + } + + /** + * Gets the persistent value, optionally forcing re-reading stored value + * @param forceRefresh whether to force to re-read memento in case value changed + */ + public List value(boolean forceRefresh) { + if (forceRefresh) { + m_value = null; + } + return value(); + } + + /** Attempts to find the parameter in the preference store. Returns null if not found */ + private List restoreParameter() { + IEclipsePreferences store = MulticoreVisualizerUIPlugin.getEclipsePreferenceStore(); + String memento = store.get(getStoreKey(), null); + if (memento == null) return null; + + List vals = MementoUtils.decodeListFromMemento(memento); + // convert from List to List + List convertedVal = convertToT(vals); + return convertedVal; + } + + /** Saves parameter's value in preference store */ + private void persistParameter(List values) { + // Convert List to List + List strList = convertTListToStringList(values); + // create memento from List + String memento = MementoUtils.encodeListIntoMemento(strList); + + // save memento in store + if (memento != null) { + IEclipsePreferences store = MulticoreVisualizerUIPlugin.getEclipsePreferenceStore(); + store.put(getStoreKey(), memento); try { store.flush(); } catch (BackingStoreException e) { @@ -102,21 +260,127 @@ public class PersistentSettingsManager { } } - @SuppressWarnings("unchecked") - /** Converts the stored value from a String to its expected type */ - private T convertToT(String val) { - // TODO: Add other types? Float, etc - if (String.class.isAssignableFrom(myClazz)) { - return (T) val; - } - else if (Integer.class.isAssignableFrom(myClazz)) { - return (T) Integer.valueOf(val); - } - else if (Boolean.class.isAssignableFrom(myClazz)) { - return (T) Boolean.valueOf(val); - } + /** For list parameters, converts the restored values from String + * to its expected generic type */ + private List convertToT(List vals) { + List convertedList = new ArrayList<>(); - return null; + for(String val : vals) { + convertedList.add(convertToT(val)); + } + return convertedList; + } + + /** Converts a list of generic type to a list of String */ + private List convertTListToStringList(List tList) { + List strList = new ArrayList<>(); + // convert list to list of String + for(T elem : tList) { + strList.add(elem.toString()); + } + return strList; + } + } + + + /** Class for a persistent {@literal Map} parameter */ + public class PersistentMapParameter extends AbstractPersistentParameter { + private Map m_value; + private Map m_defaultValue; + + public PersistentMapParameter(Class clazz, boolean perInstance, String storeKey) { + super(clazz, perInstance, storeKey); + } + + /** Sets the default value to use if no persistent + * value is found for this parameter */ + public void setDefault(Map defaultValues) { + m_defaultValue = defaultValues; + } + + /** Sets the value to persist */ + public void set(Map values) { + m_value = values; + // save value in preference store + persistParameter(values); + } + + /** Returns the persistent value, if found, else the default value */ + public Map value() { + if (m_value == null) { + // attempt to get the value from the preference store + m_value = restoreParameter(); + } + // parameter has one value for any/all instances + else if(!isPerInstance()) { + // do not rely on cached value, since another instance might have + // changed it - reread from data store + m_value = restoreParameter(); + } + return (m_value == null)? m_defaultValue : m_value ; + } + + /** + * Gets the persistent value, optionally forcing re-reading stored value + * @param forceRefresh whether to force to re-read memento in case value changed + */ + public Map value(boolean forceRefresh) { + if (forceRefresh) { + m_value = null; + } + return value(); + } + + /** Attempts to find the parameter in the preference store. Returns null if not found */ + private Map restoreParameter() { + IEclipsePreferences store = MulticoreVisualizerUIPlugin.getEclipsePreferenceStore(); + String memento = store.get(getStoreKey(), null); + if (memento == null) return null; + + Map vals = MementoUtils.decodeMapFromMemento(memento); + // convert from Map to Map + Map convertedVal = convertToT(vals); + return convertedVal; + } + + /** Saves parameter's value in preference store */ + private void persistParameter(Map values) { + // Convert Map to Map + Map strMap = convertTMapToStringMap(values); + // create memento from Map + String memento = MementoUtils.encodeMapIntoMemento(strMap); + + // save memento in store + if (memento != null) { + IEclipsePreferences store = MulticoreVisualizerUIPlugin.getEclipsePreferenceStore(); + store.put(getStoreKey(), memento); + try { + store.flush(); + } catch (BackingStoreException e) { + e.printStackTrace(); + } + } + } + + /** For Map parameters, converts the restored values from {@literal Map} + * to {@literal Map} */ + private Map convertToT(Map vals) { + Map convertedMap = new HashMap<>(); + + for(String key : vals.keySet()) { + convertedMap.put(key, convertToT(vals.get(key))); + } + return convertedMap; + } + + /** Converts a {@literal Map} to a {@literal Map} */ + private Map convertTMapToStringMap(Map map) { + Map strMap = new HashMap<>(); + // convert each entry + for(String key : map.keySet()) { + strMap.put(key, map.get(key).toString()); + } + return strMap; } } @@ -165,17 +429,12 @@ public class PersistentSettingsManager { * @param defaultValue: default value to use (mandatory) */ public PersistentParameter getNewParameter(Class clazz, String label, boolean perInstance, T defaultValue) { + PersistentParameter setting; + // build the final store key with category, parameter label and specific instance, if applicable + setting = new PersistentParameter(clazz, perInstance, getStorageKey(perInstance) + "." + label); //$NON-NLS-1$ // check that we're dealing with one of a few supported types - // TODO: Add other types? Float, etc - if (String.class.isAssignableFrom(clazz) || - Integer.class.isAssignableFrom(clazz) || - Boolean.class.isAssignableFrom(clazz)) - { - PersistentParameter setting; - // build the final store key with category, parameter label and specific instance, if applicable - setting = new PersistentParameter(clazz, perInstance, getStorageKey(perInstance) + "." + label); //$NON-NLS-1$ + if (setting.isTypeSupported(clazz)) { setting.setDefault(defaultValue); - return setting; } else { @@ -183,11 +442,55 @@ public class PersistentSettingsManager { } } + /** + * Creates a new persistent {@literal List} parameter, using the namespace and instance id of this manager. + * @param clazz: the class of the persistent parameter List (e.g. List of that type). Supported types: String, Integer, Boolean + * @param label: unique label that identifies this parameter. + * @param perInstance: whether the parameter's value should be persisted per client instance or + * globally (one common shared stored value for all instances) + * @param defaultValue: default value to use (mandatory). + */ + public PersistentListParameter getNewListParameter(Class clazz, String label, boolean perInstance, List defaultValue) { + PersistentListParameter setting; + // build the final store key with category, parameter label and specific instance, if applicable + setting = new PersistentListParameter(clazz, perInstance, getStorageKey(perInstance) + "." + label); //$NON-NLS-1$ + // check that we're dealing with one of a few supported types + if (setting.isTypeSupported(clazz)) { + setting.setDefault(defaultValue); + return setting; + } + else { + throw new InvalidParameterException("Unsupported class type: " + clazz.toString()); //$NON-NLS-1$ + } + } + + /** + * Creates a new persistent {@literal Map} parameter, using the namespace and instance id of this manager. + * @param clazz: the class of the persistent parameter List (e.g. List of that type). Supported types: String, Integer, Boolean + * @param label: unique label that identifies this parameter. + * @param perInstance: whether the parameter's value should be persisted per client instance or + * globally (one common shared stored value for all instances) + * @param defaultValue: default value to use (mandatory). + */ + public PersistentMapParameter getNewMapParameter(Class clazz, String label, boolean perInstance, Map defaultValue) { + PersistentMapParameter setting; + // build the final store key with category, parameter label and specific instance, if applicable + setting = new PersistentMapParameter(clazz, perInstance, getStorageKey(perInstance) + "." + label); //$NON-NLS-1$ + // check that we're dealing with one of a few supported types + if (setting.isTypeSupported(clazz)) { + setting.setDefault(defaultValue); + return setting; + } + else { + throw new InvalidParameterException("Unsupported class type: " + clazz.toString()); //$NON-NLS-1$ + } + } + + // ---- misc ---- /** Returns the key to be used to save parameter, taking into account the * instance id, if applicable */ private String getStorageKey(boolean perInstance) { return (perInstance ? m_instance : "") + (!m_category.equals("") ? "." + m_category : ""); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$ } - -} +} \ No newline at end of file diff --git a/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/META-INF/MANIFEST.MF b/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/META-INF/MANIFEST.MF index 4dcd889e9ed..e899b29594d 100644 --- a/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/META-INF/MANIFEST.MF +++ b/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/META-INF/MANIFEST.MF @@ -8,3 +8,4 @@ Bundle-RequiredExecutionEnvironment: JavaSE-1.7 Require-Bundle: org.junit;bundle-version="4.8.1", org.eclipse.cdt.visualizer.ui, org.eclipse.swt +Import-Package: org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.utils diff --git a/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/src/org/eclipse/cdt/dsf/gdb/tests/AllTests.java b/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/src/org/eclipse/cdt/dsf/gdb/tests/AllTests.java index eab4d940408..4d97e35db42 100644 --- a/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/src/org/eclipse/cdt/dsf/gdb/tests/AllTests.java +++ b/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/src/org/eclipse/cdt/dsf/gdb/tests/AllTests.java @@ -1,5 +1,5 @@ /******************************************************************************* - * Copyright (c) 2014 Ericsson AB and others. + * Copyright (c) 2014, 2015 Ericsson AB and others. * All rights reserved. This program and the accompanying materials * are made available under the terms of the Eclipse Public License v1.0 * which accompanies this distribution, and is available at @@ -28,6 +28,7 @@ import org.junit.runners.Suite.SuiteClasses; MIStringHandlerTests.class, ProcStatParserTest.class, VisualizerVirtualBoundsGraphicObjectTest.class, + PersistentSettingsManagerTest.class, }) public class AllTests { // Often overriding BeforeClass method here diff --git a/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/src/org/eclipse/cdt/dsf/gdb/tests/PersistentSettingsManagerTest.java b/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/src/org/eclipse/cdt/dsf/gdb/tests/PersistentSettingsManagerTest.java new file mode 100644 index 00000000000..0e5b892c27c --- /dev/null +++ b/dsf-gdb/org.eclipse.cdt.dsf.gdb.tests/src/org/eclipse/cdt/dsf/gdb/tests/PersistentSettingsManagerTest.java @@ -0,0 +1,436 @@ +/******************************************************************************* + * Copyright (c) 2015 Ericsson + * All rights reserved. This program and the accompanying materials + * are made available under the terms of the Eclipse Public License v1.0 + * which accompanies this distribution, and is available at + * http://www.eclipse.org/legal/epl-v10.html + * + * Contributors: + * Marc Dumais (Ericsson) - Initial API and implementation + *******************************************************************************/ + +package org.eclipse.cdt.dsf.gdb.tests; + +import static org.junit.Assert.assertEquals; + +import java.util.ArrayList; +import java.util.HashMap; +import java.util.List; +import java.util.Map; +import java.util.Random; + +import org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.utils.PersistentSettingsManager; +import org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.utils.PersistentSettingsManager.PersistentListParameter; +import org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.utils.PersistentSettingsManager.PersistentMapParameter; +import org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.utils.PersistentSettingsManager.PersistentParameter; +import org.junit.Test; + +@SuppressWarnings("restriction") +public class PersistentSettingsManagerTest { + + // ---- variables ---- + + /** persistent settings manager */ + private PersistentSettingsManager m_persistentSettingsManager1; + private PersistentSettingsManager m_persistentSettingsManager2; + private PersistentSettingsManager m_persistentSettingsManager3; + + // set of persistent parameters + private PersistentParameter m_stringParam; + private PersistentParameter m_booleanParam; + private PersistentParameter m_integerParam; + private PersistentListParameter m_listOfStringParam; + private PersistentListParameter m_listOfBooleanParam; + private PersistentListParameter m_listOfIntegerParam; + private PersistentMapParameter m_mapOfStringParam; + private PersistentMapParameter m_mapOfBooleanParam; + private PersistentMapParameter m_mapOfIntegerParam; + + // set of per-instance parameters + private PersistentParameter m_stringParamInstance1; + private PersistentParameter m_stringParamInstance2; + private PersistentParameter m_stringParamInstance3; + + // set of global parameters + private PersistentParameter m_stringGlobalParamInstance1; + private PersistentParameter m_stringGlobalParamInstance2; + private PersistentParameter m_stringGlobalParamInstance3; + + // random number generator + private Random m_random = new Random(); + + + // ---- constants ---- + private final static int NUM_LIST_ELEMENTS = 10; + + private final static String INSTANCE_ID_1 = "instance1"; + private final static String INSTANCE_ID_2 = "instance2"; + private final static String INSTANCE_ID_3 = "instance3"; + + private final static String DEFAULT_STRING = "Default String"; + private final static Boolean DEFAULT_BOOLEAN = false; + private final static Integer DEFAULT_INTEGER = 1234321; + + private final static String DEFAULT_STRING_VAL_INSTANCE1 = "Default String Instance 1"; + private final static String DEFAULT_STRING_VAL_INSTANCE2 = "Default String Instance 2"; + private final static String DEFAULT_STRING_VAL_INSTANCE3 = "Default String Instance 3"; + private final static String DEFAULT_STRING_VAL_SHARED = "Default String Shared Instance"; + + + public PersistentSettingsManagerTest() { + // 3 instances of managers - to simulate, for example, 3 views each having one manager + m_persistentSettingsManager1 = new PersistentSettingsManager("PersistentSettingsManagerTest", INSTANCE_ID_1); + m_persistentSettingsManager2 = new PersistentSettingsManager("PersistentSettingsManagerTest", INSTANCE_ID_2); + m_persistentSettingsManager3 = new PersistentSettingsManager("PersistentSettingsManagerTest", INSTANCE_ID_3); + + // one persistent parameter for each supported type: + // simple types + m_stringParam = m_persistentSettingsManager1.getNewParameter(String.class, "String Parameter", false, DEFAULT_STRING); + m_booleanParam = m_persistentSettingsManager1.getNewParameter(Boolean.class, "Boolean Parameter", false, DEFAULT_BOOLEAN); + m_integerParam = m_persistentSettingsManager1.getNewParameter(Integer.class, "Integer Parameter", false, DEFAULT_INTEGER); + // List + m_listOfStringParam = m_persistentSettingsManager1.getNewListParameter(String.class, "List of String Parameter", false, new ArrayList()); + m_listOfBooleanParam = m_persistentSettingsManager1.getNewListParameter(Boolean.class, "List of Boolean Parameter", false, new ArrayList()); + m_listOfIntegerParam = m_persistentSettingsManager1.getNewListParameter(Integer.class, "List of Integer Parameter", false, new ArrayList()); + // Map + m_mapOfStringParam = m_persistentSettingsManager1.getNewMapParameter(String.class, "Map of String Parameter", true, new HashMap()); + m_mapOfBooleanParam = m_persistentSettingsManager1.getNewMapParameter(Boolean.class, "Map of Boolean Parameter", true, new HashMap()); + m_mapOfIntegerParam = m_persistentSettingsManager1.getNewMapParameter(Integer.class, "Map of Integer Parameter", true, new HashMap()); + + // simulate 3 instances using the same parameter, using "per instance" persistence (i.e. they'll be persisted independently) + m_stringParamInstance1 = m_persistentSettingsManager1.getNewParameter(String.class, "Per-instance String Parameter", true, DEFAULT_STRING_VAL_INSTANCE1); + m_stringParamInstance2 = m_persistentSettingsManager2.getNewParameter(String.class, "Per-instance String Parameter", true, DEFAULT_STRING_VAL_INSTANCE2); + m_stringParamInstance3 = m_persistentSettingsManager3.getNewParameter(String.class, "Per-instance String Parameter", true, DEFAULT_STRING_VAL_INSTANCE3); + + // This is to simulate a persistent parameter, being "shared" by 3 instances (e.g. views). So, the 3 instances are persisted as a single parameter. + m_stringGlobalParamInstance1 = m_persistentSettingsManager1.getNewParameter(String.class, "Global String Parameter", false, DEFAULT_STRING_VAL_SHARED); + m_stringGlobalParamInstance2 = m_persistentSettingsManager2.getNewParameter(String.class, "Global String Parameter", false, DEFAULT_STRING_VAL_SHARED); + m_stringGlobalParamInstance3 = m_persistentSettingsManager3.getNewParameter(String.class, "Global String Parameter", false, DEFAULT_STRING_VAL_SHARED); + } + + // testcases + + /** Test Un-supported base type - we expect an Exception */ + @Test(expected = Exception.class) + public void testUnsupportedBaseType() throws Exception { + m_persistentSettingsManager1.getNewParameter(Float.class, "Float Parameter", false, 1.0f); + } + + /** Test Un-supported List type - we expect an Exception */ + @Test(expected = Exception.class) + public void testUnsupportedListType() throws Exception { + m_persistentSettingsManager1.getNewListParameter(Float.class, "List of Float Parameter", false, new ArrayList()); + } + + /** Test Un-supported Map type - we expect an Exception */ + @Test(expected = Exception.class) + public void testUnsupportedMapType() throws Exception { + m_persistentSettingsManager1.getNewMapParameter(Float.class, "Map of Float Parameter", false, new HashMap()); + } + + /** Test persisting one String value */ + @Test + public void testPersistentParamString() throws Exception { + // no value persisted yet - should return default + assertEquals(DEFAULT_STRING, m_stringParam.value()); + + // set a value + String randomString = getRandomString(); + m_stringParam.set(randomString); + // get cached value + assertEquals(randomString, m_stringParam.value()); + // force re-read from storage + assertEquals(randomString, m_stringParam.value(true)); + + // set a different value + randomString = getRandomString(); + m_stringParam.set(randomString); + // get cached value + assertEquals(randomString, m_stringParam.value()); + // force re-read from storage + assertEquals(randomString, m_stringParam.value(true)); + } + + /** Test persisting one Boolean value */ + @Test + public void testPersistentParamBoolean() throws Exception { + // no value persisted yet - should return default + assertEquals(DEFAULT_BOOLEAN, m_booleanParam.value()); + + // set a value + m_booleanParam.set(!DEFAULT_BOOLEAN); + // get cached value + assertEquals(!DEFAULT_BOOLEAN, m_booleanParam.value()); + // force re-read from storage + assertEquals(!DEFAULT_BOOLEAN, m_booleanParam.value(true)); + } + + /** Test persisting one Integer value */ + @Test + public void testPersistentParamInteger() throws Exception { + // no value persisted yet - should return default + assertEquals(DEFAULT_INTEGER, m_integerParam.value()); + + // set a value + Integer randomInt = getRandomInt(); + m_integerParam.set(randomInt); + // get cached value + assertEquals(randomInt, m_integerParam.value()); + // force re-read from storage + assertEquals(randomInt, m_integerParam.value(true)); + + // set a different value + randomInt = getRandomInt(); + m_integerParam.set(randomInt); + // get cached value + assertEquals(randomInt, m_integerParam.value()); + // force re-read from storage + assertEquals(randomInt, m_integerParam.value(true)); + } + + /** Test persisting a List of String */ + @Test + public void testPersistentParamListOfString() throws Exception { + // no value persisted yet - should return default (empty list) + List list = m_listOfStringParam.value(); + assertEquals(0,list.size()); + // generate list of random elements + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + list.add(getRandomString()); + } + m_listOfStringParam.set(list); + + // get cached value + List list2 = m_listOfStringParam.value(); + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + assertEquals(list.get(i), list2.get(i)); + } + // force re-read from storage + list2 = m_listOfStringParam.value(true); + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + assertEquals(list.get(i), list2.get(i)); + } + } + + /** Test persisting a List of Boolean */ + @Test + public void testPersistentParamListOfBoolean() throws Exception { + // no value persisted yet - should return default (empty list) + List list = m_listOfBooleanParam.value(); + assertEquals(0,list.size()); + // generate list of random elements + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + list.add(getRandomBoolean()); + } + m_listOfBooleanParam.set(list); + + // get cached value + List list2 = m_listOfBooleanParam.value(); + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + assertEquals(list.get(i), list2.get(i)); + } + // force re-read from storage + list2 = m_listOfBooleanParam.value(true); + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + assertEquals(list.get(i), list2.get(i)); + } + } + + /** Test persisting a List of Integer */ + @Test + public void testPersistentParamListofInteger() throws Exception { + // no value persisted yet - should return default (empty list) + List list = m_listOfIntegerParam.value(); + assertEquals(0,list.size()); + // generate list of random elements + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + list.add(getRandomInt()); + } + m_listOfIntegerParam.set(list); + + List list2; + // get cached value + list2 = m_listOfIntegerParam.value(); + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + assertEquals(list.get(i), list2.get(i)); + } + // force re-read from storage + list2 = m_listOfIntegerParam.value(true); + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + assertEquals(list.get(i), list2.get(i)); + } + } + + + /** Test persisting a Map of String */ + @Test + public void testPersistentParamMapOfString() throws Exception { + // no value persisted yet - should return default (empty Map) + Map map = m_mapOfStringParam.value(); + assertEquals(0,map.size()); + + // generate random elements + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + map.put(getRandomString(), getRandomString()); + } + m_mapOfStringParam.set(map); + + // get cached value + Map map2 = m_mapOfStringParam.value(); + assertEquals(map.size(), map2.size()); + for(String key : map2.keySet()) { + assertEquals(map2.get(key), map.get(key)); + } + + // force re-read from storage + map2 = m_mapOfStringParam.value(true); + assertEquals(map.size(), map2.size()); + for(String key : map2.keySet()) { + assertEquals(map2.get(key), map.get(key)); + } + } + + /** Test persisting a Map of Boolean*/ + @Test + public void testPersistentParamMapOfBoolean() throws Exception { + // no value persisted yet - should return default (empty Map) + Map map = m_mapOfBooleanParam.value(); + assertEquals(0,map.size()); + + // generate random elements + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + map.put(getRandomString(), getRandomBoolean()); + } + m_mapOfBooleanParam.set(map); + + // get cached value + Map map2 = m_mapOfBooleanParam.value(); + assertEquals(map.size(), map2.size()); + for(String key : map2.keySet()) { + assertEquals(map2.get(key), map.get(key)); + } + + // force re-read from storage + map2 = m_mapOfBooleanParam.value(true); + assertEquals(map.size(), map2.size()); + for(String key : map2.keySet()) { + assertEquals(map2.get(key), map.get(key)); + } + } + + /** Test persisting a List of Integer */ + @Test + public void testPersistentParamMapOfInteger() throws Exception { + // no value persisted yet - should return default (empty Map) + Map map = m_mapOfIntegerParam.value(); + assertEquals(0,map.size()); + + // generate random elements + for(int i = 0; i < NUM_LIST_ELEMENTS; i++) { + map.put(getRandomString(), getRandomInt()); + } + m_mapOfIntegerParam.set(map); + + // get cached value + Map map2 = m_mapOfIntegerParam.value(); + assertEquals(map.size(), map2.size()); + for(String key : map2.keySet()) { + assertEquals(map2.get(key), map.get(key)); + } + + // force re-read from storage + map2 = m_mapOfIntegerParam.value(true); + assertEquals(map.size(), map2.size()); + for(String key : map2.keySet()) { + assertEquals(map2.get(key), map.get(key)); + } + } + + /** This test simulates 3 different instances (e.g. views) writing and reading the same persistent + * parameter. In this case the parameter is defined as "per-instance" (vs "global"), so each + * instance has it's own independent copy of the persistent parameter. */ + @Test + public void testMultipleInstances() throws Exception { + // no values persisted yet - should return defaults + assertEquals(DEFAULT_STRING_VAL_INSTANCE1, m_stringParamInstance1.value()); + assertEquals(DEFAULT_STRING_VAL_INSTANCE2, m_stringParamInstance2.value()); + assertEquals(DEFAULT_STRING_VAL_INSTANCE3, m_stringParamInstance3.value()); + + // set values - since the parameters were defined to save values per-instance, they should be + // persisted independently (i.e. not overwrite each other) + String randomString1 = getRandomString(); + String randomString2 = getRandomString(); + String randomString3 = getRandomString(); + m_stringParamInstance1.set(randomString1); + m_stringParamInstance2.set(randomString2); + m_stringParamInstance3.set(randomString3); + assertEquals(randomString1, m_stringParamInstance1.value(true)); + assertEquals(randomString2, m_stringParamInstance2.value(true)); + assertEquals(randomString3, m_stringParamInstance3.value(true)); + + // set different values + randomString1 = getRandomString(); + randomString2 = getRandomString(); + randomString3 = getRandomString(); + m_stringParamInstance1.set(randomString1); + m_stringParamInstance2.set(randomString2); + m_stringParamInstance3.set(randomString3); + assertEquals(randomString1, m_stringParamInstance1.value(true)); + assertEquals(randomString2, m_stringParamInstance2.value(true)); + assertEquals(randomString3, m_stringParamInstance3.value(true)); + } + + + /** This test simulates 3 different instances (e.g. views) writing and reading the same persistent + * parameter. In this case the parameter is defined as "global" (vs "per-instance"), so only one + * copy is shared between the instances. */ + @Test + public void testGlobalParamsWithMultipleInstances() throws Exception { + // no values persisted yet - should return defaults + assertEquals(DEFAULT_STRING_VAL_SHARED, m_stringGlobalParamInstance1.value()); + assertEquals(DEFAULT_STRING_VAL_SHARED, m_stringGlobalParamInstance2.value()); + assertEquals(DEFAULT_STRING_VAL_SHARED, m_stringGlobalParamInstance3.value()); + + // set values - since the parameters were defined to save values per-instance, they should be + // persisted independently + String randomString1 = getRandomString(); + String randomString2 = getRandomString(); + String randomString3 = getRandomString(); + m_stringGlobalParamInstance1.set(randomString1); + m_stringGlobalParamInstance2.set(randomString2); + m_stringGlobalParamInstance3.set(randomString3); + // since the parameters are global, they share the same storage... So the last value written + // will be persisted + assertEquals(randomString3, m_stringGlobalParamInstance1.value(true)); + assertEquals(randomString3, m_stringGlobalParamInstance2.value(true)); + assertEquals(randomString3, m_stringGlobalParamInstance3.value(true)); + + // set different values + randomString1 = getRandomString(); + randomString2 = getRandomString(); + randomString3 = getRandomString(); + m_stringGlobalParamInstance1.set(randomString1); + m_stringGlobalParamInstance2.set(randomString2); + m_stringGlobalParamInstance3.set(randomString3); + // since the parameters are global, they share the same storage... So the last value written + // will be persisted + assertEquals(randomString3, m_stringGlobalParamInstance1.value(true)); + assertEquals(randomString3, m_stringGlobalParamInstance2.value(true)); + assertEquals(randomString3, m_stringGlobalParamInstance3.value(true)); + } + + // utility methods + + private int getRandomInt() { + return m_random.nextInt(); + } + + private String getRandomString() { + return Integer.toString(getRandomInt(), 16) + Integer.toString(getRandomInt(), 16); + } + + private boolean getRandomBoolean() { + return getRandomInt() % 2 == 0; + } + +}