weaver 2004/10/29 07:02:28 Modified: components/prefs/src/test/org/apache/jetspeed/prefs TestPropertyManager.java TestPreferences.java Added: components/prefs/src/test/org/apache/jetspeed/prefs TestPreferencesNoPropManager.java Log: see: http://nagoya.apache.org/jira/browse/JS2-144 - Refactored prefs Revision Changes Path 1.19 +26 -31 jakarta-jetspeed-2/components/prefs/src/test/org/apache/jetspeed/prefs/TestPropertyManager.java Index: TestPropertyManager.java =================================================================== RCS file: /home/cvs/jakarta-jetspeed-2/components/prefs/src/test/org/apache/jetspeed/prefs/TestPropertyManager.java,v retrieving revision 1.18 retrieving revision 1.19 diff -u -r1.18 -r1.19 --- TestPropertyManager.java 1 Aug 2004 20:23:00 -0000 1.18 +++ TestPropertyManager.java 29 Oct 2004 14:02:28 -0000 1.19 @@ -22,9 +22,7 @@ import junit.framework.Test; import junit.framework.TestSuite; -import org.apache.jetspeed.components.persistence.store.util.PersistenceSupportedTestCase; -import org.apache.jetspeed.prefs.impl.PreferencesProviderImpl; -import org.apache.jetspeed.prefs.impl.PropertyManagerImpl; +import org.apache.jetspeed.components.util.DatasourceEnabledSpringTestCase; import org.apache.jetspeed.prefs.om.Property; /** @@ -32,7 +30,7 @@ * * @author <a href="mailto:[EMAIL PROTECTED]">David Le Strat</a> */ -public class TestPropertyManager extends PersistenceSupportedTestCase +public class TestPropertyManager extends DatasourceEnabledSpringTestCase { /** The property manager. */ @@ -44,28 +42,8 @@ private final static int USER_PROPERTY_SET_TYPE = 0; private final static int SYSTEM_PROPERTY_SET_TYPE = 1; - private PreferencesProviderImpl provider; + private PreferencesProvider provider; - /** - * <p>Defines the test case name for junit.</p> - * @param testName The test case name. - */ - public TestPropertyManager(String testName) - { - super(testName); - } - - /** - * @see junit.framework.TestCase#setUp() - */ - public void setUp() throws Exception - { - super.setUp(); - pms =new PropertyManagerImpl(persistenceStore); - provider = new PreferencesProviderImpl(persistenceStore, "org.apache.jetspeed.prefs.impl.PreferencesFactoryImpl", true); - // Class.forName("org.apache.jetspeed.prefs.impl.PreferencesImpl"); - provider.start(); - } /** * @see junit.framework.TestCase#tearDown() @@ -86,7 +64,7 @@ /** * <p>Test add property keys to a [EMAIL PROTECTED] Preferences} node.</p> */ - public void testAddPropertyKeys() throws PropertyException + public void testAddPropertyKeys() throws Exception { Map propertyKeys = initPropertyKeysMap(); Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); @@ -102,8 +80,9 @@ /** * <p>Test get property key.</p> + * @throws Exception */ - public void testGetPropertyKeys() + public void testGetPropertyKeys() throws Exception { initPropertyKeys(); Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); @@ -114,7 +93,7 @@ /** * <p>Test update property key.</p> */ - public void testUpdatePropertyKey() throws PropertyException + public void testUpdatePropertyKey() throws Exception { initPropertyKeys(); Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); @@ -139,7 +118,7 @@ /** * <p>Test remove property keys.</p> */ - public void testRemovePropertyKeys() throws PropertyException + public void testRemovePropertyKeys() throws Exception { initPropertyKeys(); Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); @@ -174,7 +153,7 @@ /** * <p>Init property property keys.</p> */ - protected void initPropertyKeys() + protected void initPropertyKeys() throws Exception { Map propertyKeys = initPropertyKeysMap(); Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); @@ -191,7 +170,7 @@ /** * <p>Clean properties.</p> */ - protected void clean() + protected void clean() throws Exception { Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); try @@ -207,6 +186,22 @@ { System.out.println("BackingStoreException" + bse); } + } + + protected String[] getConfigurations() + { + return new String[]{"META-INF/prefs-dao.xml", "META-INF/transaction.xml"}; + } + + /** + * @see junit.framework.TestCase#setUp() + */ + public void setUp() throws Exception + { + super.setUp(); + provider = (PreferencesProvider) ctx.getBean("prefsProvider"); + + pms = (PropertyManager) ctx.getBean("propertyManager"); } } 1.20 +17 -50 jakarta-jetspeed-2/components/prefs/src/test/org/apache/jetspeed/prefs/TestPreferences.java Index: TestPreferences.java =================================================================== RCS file: /home/cvs/jakarta-jetspeed-2/components/prefs/src/test/org/apache/jetspeed/prefs/TestPreferences.java,v retrieving revision 1.19 retrieving revision 1.20 diff -u -r1.19 -r1.20 --- TestPreferences.java 12 Oct 2004 19:46:30 -0000 1.19 +++ TestPreferences.java 29 Oct 2004 14:02:28 -0000 1.20 @@ -22,9 +22,7 @@ import junit.framework.Test; import junit.framework.TestSuite; -import org.apache.jetspeed.components.persistence.store.util.PersistenceSupportedTestCase; -import org.apache.jetspeed.prefs.impl.PreferencesProviderImpl; -import org.apache.jetspeed.prefs.impl.PropertyManagerImpl; +import org.apache.jetspeed.components.util.DatasourceEnabledSpringTestCase; /** * <p> @@ -33,7 +31,7 @@ * * @author <a href="[EMAIL PROTECTED]">David Le Strat </a> */ -public class TestPreferences extends PersistenceSupportedTestCase +public class TestPreferences extends DatasourceEnabledSpringTestCase { /** The property manager. */ @@ -47,20 +45,9 @@ private final static int USER_PROPERTY_SET_TYPE = 0; private final static int SYSTEM_PROPERTY_SET_TYPE = 1; - private PreferencesProviderImpl provider; + private PreferencesProvider provider; - /** - * <p> - * Defines the test case name for junit. - * </p> - * - * @param testName - * The test case name. - */ - public TestPreferences( String testName ) - { - super(testName); - } + private PreferencesProvider providerNoProp; /** * @see junit.framework.TestCase#setUp() @@ -68,10 +55,10 @@ public void setUp() throws Exception { super.setUp(); - pms = new PropertyManagerImpl(persistenceStore); - provider = new PreferencesProviderImpl(persistenceStore, - "org.apache.jetspeed.prefs.impl.PreferencesFactoryImpl", true); - provider.start(); + provider = (PreferencesProvider) ctx.getBean("prefsProvider"); + + pms = (PropertyManager) ctx.getBean("propertyManager"); + // Make sure we are starting with a clean slate clearChildren(Preferences.userRoot()); clearChildren(Preferences.systemRoot()); @@ -100,9 +87,7 @@ */ public void testUserRoot() { - pms = new PropertyManagerImpl(persistenceStore); - provider = new PreferencesProviderImpl(persistenceStore, - "org.apache.jetspeed.prefs.impl.PreferencesFactoryImpl", true); + Preferences prefs = Preferences.userRoot(); if (null != prefs) { @@ -114,20 +99,6 @@ } } - public void testSansPropertyManager() - { - pms = new PropertyManagerImpl(persistenceStore); - provider = new PreferencesProviderImpl(persistenceStore, - "org.apache.jetspeed.prefs.impl.PreferencesFactoryImpl", false); - Preferences pref0 = Preferences.userRoot(); - // Test that the property manager is off - Preferences pref1 = pref0.node("testOpenNode"); - pref1.put("0", "I am 0 key"); - - assertNotNull(pref1.get("0", null)); - - } - /** * <p> * Test system root. @@ -135,9 +106,6 @@ */ public void testSystemRoot() { - pms = new PropertyManagerImpl(persistenceStore); - provider = new PreferencesProviderImpl(persistenceStore, - "org.apache.jetspeed.prefs.impl.PreferencesFactoryImpl", true); Preferences prefs = Preferences.systemRoot(); if (null != prefs) { @@ -156,9 +124,6 @@ */ public void testNodeAndChildrenNames() { - pms = new PropertyManagerImpl(persistenceStore); - provider = new PreferencesProviderImpl(persistenceStore, - "org.apache.jetspeed.prefs.impl.PreferencesFactoryImpl", true); Preferences prefs = Preferences.userRoot(); // Test without children. try @@ -220,12 +185,10 @@ * Test adding properties to a property set node and get property keys for a * given node. * </p> + * @throws Exception */ - public void testPropertyAndPropertyKeys() + public void testPropertyAndPropertyKeys() throws Exception { - pms = new PropertyManagerImpl(persistenceStore); - provider = new PreferencesProviderImpl(persistenceStore, - "org.apache.jetspeed.prefs.impl.PreferencesFactoryImpl", true); // 1. Current node does not have any property associated to it. // No property has been defined nor added to the node. There should be @@ -339,7 +302,7 @@ * Init property property keys. * </p> */ - protected void initPropertyKeys() + protected void initPropertyKeys() throws Exception { Map propertyKeys = initPropertyKeysMap(); Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); @@ -358,7 +321,7 @@ * Clean properties. * </p> */ - protected void clean() + protected void clean() throws Exception { Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); try @@ -390,4 +353,8 @@ } } + protected String[] getConfigurations() + { + return new String[]{"META-INF/prefs-dao.xml", "META-INF/transaction.xml"}; + } } 1.1 jakarta-jetspeed-2/components/prefs/src/test/org/apache/jetspeed/prefs/TestPreferencesNoPropManager.java Index: TestPreferencesNoPropManager.java =================================================================== /* * Created on Oct 21, 2004 * * TODO To change the template for this generated file go to * Window - Preferences - Java - Code Generation - Code and Comments */ package org.apache.jetspeed.prefs; import java.util.Map; import java.util.prefs.BackingStoreException; import java.util.prefs.Preferences; import org.apache.jetspeed.components.util.DatasourceEnabledSpringTestCase; /** * <p> * TestPreferencesNoPropManager * </p> * <p> * * </p> * @author <a href="mailto:[EMAIL PROTECTED]">Scott T. Weaver</a> * @version $Id: TestPreferencesNoPropManager.java,v 1.1 2004/10/29 14:02:28 weaver Exp $ * */ public class TestPreferencesNoPropManager extends DatasourceEnabledSpringTestCase { /** * The property manager. */ private static PropertyManager pms; private PreferencesProvider provider; /** * @see junit.framework.TestCase#setUp() */ public void setUp() throws Exception { super.setUp(); provider = (PreferencesProvider) ctx.getBean("prefsProvider"); pms = (PropertyManager) ctx.getBean("propertyManager"); // Make sure we are starting with a clean slate clearChildren(Preferences.userRoot()); clearChildren(Preferences.systemRoot()); } /** * @see junit.framework.TestCase#tearDown() */ public void tearDown() throws Exception { clean(); // super.tearDown(); } protected String[] getConfigurations() { return new String[]{"META-INF/prefs-noprop-dao.xml", "META-INF/transaction.xml"}; } protected void clearChildren(Preferences node) throws Exception { String[] names = node.childrenNames(); for(int i=0; i < names.length; i++) { node.node(names[i]).removeNode(); } } /** * <p> * Clean properties. * </p> */ protected void clean() throws Exception { Preferences pref = Preferences.userRoot().node("/user/principal1/propertyset1"); try { Map propertyKeys = pms.getPropertyKeys(pref); pms.removePropertyKeys(pref, propertyKeys.keySet()); Preferences.userRoot().node("/user").removeNode(); Preferences.userRoot().node("/an1").removeNode(); Preferences.userRoot().node("/rn1").removeNode(); Preferences.userRoot().node("/testOpenNode").removeNode(); Preferences.userRoot().node("/removeTest").removeNode(); } catch (PropertyException pex) { System.out.println("PropertyException" + pex); } catch (BackingStoreException bse) { System.out.println("BackingStoreException" + bse); } } public void testSansPropertyManager() throws Exception { // Make sure we are starting with a clean slate clearChildren(Preferences.userRoot()); clearChildren(Preferences.systemRoot()); Preferences pref0 = Preferences.userRoot(); // Test that the property manager is off Preferences pref1 = pref0.node("testOpenNode"); pref1.put("0", "I am 0 key"); assertNotNull(pref1.get("0", null)); } }
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]