Author: oheger
Date: Thu Dec 29 21:00:57 2011
New Revision: 1225652
URL: http://svn.apache.org/viewvc?rev=1225652&view=rev
Log:
Converted tests to JUnit 4, fixed warnings.
Modified:
commons/proper/configuration/trunk/src/test/java/org/apache/commons/configuration/event/TestEventSource.java
Modified:
commons/proper/configuration/trunk/src/test/java/org/apache/commons/configuration/event/TestEventSource.java
URL:
http://svn.apache.org/viewvc/commons/proper/configuration/trunk/src/test/java/org/apache/commons/configuration/event/TestEventSource.java?rev=1225652&r1=1225651&r2=1225652&view=diff
==============================================================================
---
commons/proper/configuration/trunk/src/test/java/org/apache/commons/configuration/event/TestEventSource.java
(original)
+++
commons/proper/configuration/trunk/src/test/java/org/apache/commons/configuration/event/TestEventSource.java
Thu Dec 29 21:00:57 2011
@@ -16,16 +16,21 @@
*/
package org.apache.commons.configuration.event;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
import java.util.Collection;
-import junit.framework.TestCase;
+import org.junit.Before;
+import org.junit.Test;
/**
* Test class for EventSource.
*
* @version $Id$
*/
-public class TestEventSource extends TestCase
+public class TestEventSource
{
/** Constant for the event type used for testing. */
static final int TEST_TYPE = 42;
@@ -39,15 +44,16 @@ public class TestEventSource extends Tes
/** The object under test. */
CountingEventSource source;
- protected void setUp() throws Exception
+ @Before
+ public void setUp() throws Exception
{
- super.setUp();
source = new CountingEventSource();
}
/**
* Tests a newly created source object.
*/
+ @Test
public void testInit()
{
assertTrue("Listeners list is not empty", source
@@ -62,11 +68,12 @@ public class TestEventSource extends Tes
/**
* Tests registering a new listener.
*/
+ @Test
public void testAddConfigurationListener()
{
TestListener l = new TestListener();
source.addConfigurationListener(l);
- Collection listeners = source.getConfigurationListeners();
+ Collection<ConfigurationListener> listeners =
source.getConfigurationListeners();
assertEquals("Wrong number of listeners", 1, listeners.size());
assertTrue("Listener not in list", listeners.contains(l));
}
@@ -75,22 +82,16 @@ public class TestEventSource extends Tes
* Tests adding an undefined configuration listener. This should cause an
* exception.
*/
+ @Test(expected = IllegalArgumentException.class)
public void testAddNullConfigurationListener()
{
- try
- {
- source.addConfigurationListener(null);
- fail("Could add null listener!");
- }
- catch (IllegalArgumentException iex)
- {
- // ok
- }
+ source.addConfigurationListener(null);
}
/**
* Tests removing a listener.
*/
+ @Test
public void testRemoveConfigurationListener()
{
TestListener l = new TestListener();
@@ -109,6 +110,7 @@ public class TestEventSource extends Tes
/**
* Tests if a null listener can be removed. This should be a no-op.
*/
+ @Test
public void testRemoveNullConfigurationListener()
{
source.addConfigurationListener(new TestListener());
@@ -121,28 +123,22 @@ public class TestEventSource extends Tes
/**
* Tests whether the listeners list is read only.
*/
+ @Test(expected = UnsupportedOperationException.class)
public void testGetConfigurationListenersUpdate()
{
source.addConfigurationListener(new TestListener());
- Collection list = source.getConfigurationListeners();
- try
- {
- list.add("test");
- fail("Could manipulate list!");
- }
- catch (Exception ex)
- {
- // ok
- }
+ Collection<ConfigurationListener> list =
source.getConfigurationListeners();
+ list.clear();
}
/**
* Tests that the collection returned by getConfigurationListeners() is
* really a snapshot. A later added listener must not be visible.
*/
+ @Test
public void testGetConfigurationListenersAddNew()
{
- Collection list = source.getConfigurationListeners();
+ Collection<ConfigurationListener> list =
source.getConfigurationListeners();
source.addConfigurationListener(new TestListener());
assertTrue("Listener snapshot not empty", list.isEmpty());
}
@@ -150,6 +146,7 @@ public class TestEventSource extends Tes
/**
* Tests enabling and disabling the detail events flag.
*/
+ @Test
public void testSetDetailEvents()
{
source.setDetailEvents(true);
@@ -164,6 +161,7 @@ public class TestEventSource extends Tes
/**
* Tests delivering an event to a listener.
*/
+ @Test
public void testFireEvent()
{
TestListener l = new TestListener();
@@ -180,8 +178,9 @@ public class TestEventSource extends Tes
}
/**
- * Tests firering an event if there are no listeners.
+ * Tests firing an event if there are no listeners.
*/
+ @Test
public void testFireEventNoListeners()
{
source.fireEvent(TEST_TYPE, TEST_PROPNAME, TEST_PROPVALUE, false);
@@ -191,6 +190,7 @@ public class TestEventSource extends Tes
/**
* Tests generating a detail event if detail events are not allowed.
*/
+ @Test
public void testFireEventNoDetails()
{
TestListener l = new TestListener();
@@ -205,6 +205,7 @@ public class TestEventSource extends Tes
* Tests whether an event listener can deregister itself in reaction of a
* delivered event.
*/
+ @Test
public void testRemoveListenerInFireEvent()
{
ConfigurationListener lstRemove = new ConfigurationListener()
@@ -227,11 +228,12 @@ public class TestEventSource extends Tes
/**
* Tests registering a new error listener.
*/
+ @Test
public void testAddErrorListener()
{
TestListener l = new TestListener();
source.addErrorListener(l);
- Collection listeners = source.getErrorListeners();
+ Collection<ConfigurationErrorListener> listeners =
source.getErrorListeners();
assertEquals("Wrong number of listeners", 1, listeners.size());
assertTrue("Listener not in list", listeners.contains(l));
}
@@ -239,22 +241,16 @@ public class TestEventSource extends Tes
/**
* Tests adding an undefined error listener. This should cause an
exception.
*/
+ @Test(expected = IllegalArgumentException.class)
public void testAddNullErrorListener()
{
- try
- {
- source.addErrorListener(null);
- fail("Could add null error listener!");
- }
- catch (IllegalArgumentException iex)
- {
- // ok
- }
+ source.addErrorListener(null);
}
/**
* Tests removing an error listener.
*/
+ @Test
public void testRemoveErrorListener()
{
TestListener l = new TestListener();
@@ -271,6 +267,7 @@ public class TestEventSource extends Tes
/**
* Tests if a null error listener can be removed. This should be a no-op.
*/
+ @Test
public void testRemoveNullErrorListener()
{
source.addErrorListener(new TestListener());
@@ -283,24 +280,18 @@ public class TestEventSource extends Tes
/**
* Tests whether the listeners list is read only.
*/
+ @Test(expected = UnsupportedOperationException.class)
public void testGetErrorListenersUpdate()
{
source.addErrorListener(new TestListener());
- Collection list = source.getErrorListeners();
- try
- {
- list.add("test");
- fail("Could manipulate list!");
- }
- catch (Exception ex)
- {
- // ok
- }
+ Collection<ConfigurationErrorListener> list =
source.getErrorListeners();
+ list.clear();
}
/**
* Tests delivering an error event to a listener.
*/
+ @Test
public void testFireError()
{
TestListener l = new TestListener();
@@ -321,8 +312,9 @@ public class TestEventSource extends Tes
}
/**
- * Tests firering an error event if there are no error listeners.
+ * Tests firing an error event if there are no error listeners.
*/
+ @Test
public void testFireErrorNoListeners()
{
source.fireError(TEST_TYPE, TEST_PROPNAME, TEST_PROPVALUE,
@@ -334,6 +326,7 @@ public class TestEventSource extends Tes
* Tests cloning an event source object. The registered listeners should
not
* be registered at the clone.
*/
+ @Test
public void testClone() throws CloneNotSupportedException
{
source.addConfigurationListener(new TestListener());
@@ -373,7 +366,7 @@ public class TestEventSource extends Tes
/**
* A specialized event source implementation that counts the number of
* created event objects. It is used to test whether the
- * <code>fireEvent()</code> methods only creates event objects if
+ * {@code fireEvent()} methods only creates event objects if
* necessary. It also allows testing the clone() operation.
*/
static class CountingEventSource extends EventSource implements Cloneable
@@ -382,6 +375,7 @@ public class TestEventSource extends Tes
int errorCount;
+ @Override
protected ConfigurationEvent createEvent(int type, String propName,
Object propValue, boolean before)
{
@@ -389,6 +383,7 @@ public class TestEventSource extends Tes
return super.createEvent(type, propName, propValue, before);
}
+ @Override
protected ConfigurationErrorEvent createErrorEvent(int type,
String propName, Object value, Throwable ex)
{
