commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ohe...@apache.org
Subject svn commit: r1225652 - /commons/proper/configuration/trunk/src/test/java/org/apache/commons/configuration/event/TestEventSource.java
Date Thu, 29 Dec 2011 21:00:58 GMT
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)
         {



Mime
View raw message