Author: jcarman
Date: Thu Jul 22 19:40:20 2010
New Revision: 966821

URL: http://svn.apache.org/viewvc?rev=966821&view=rev
Log:
Adding test cases for null checks.

Modified:
    
commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/event/EventListenerSupportTest.java

Modified: 
commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/event/EventListenerSupportTest.java
URL: 
http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/event/EventListenerSupportTest.java?rev=966821&r1=966820&r2=966821&view=diff
==============================================================================
--- 
commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/event/EventListenerSupportTest.java
 (original)
+++ 
commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/event/EventListenerSupportTest.java
 Thu Jul 22 19:40:20 2010
@@ -29,6 +29,34 @@ import java.util.List;
  */
 public class EventListenerSupportTest extends TestCase
 {
+    public void testAddNullListener()
+    {
+        EventListenerSupport<ActionListener> listenerSupport = 
EventListenerSupport.create(ActionListener.class);
+        try
+        {
+            listenerSupport.addListener(null);
+            fail("Should not be able to add a null listener.");
+        }
+        catch (NullPointerException e)
+        {
+
+        }
+    }
+
+    public void testRemoveNullListener()
+    {
+        EventListenerSupport<ActionListener> listenerSupport = 
EventListenerSupport.create(ActionListener.class);
+        try
+        {
+            listenerSupport.removeListener(null);
+            fail("Should not be able to remove a null listener.");
+        }
+        catch (NullPointerException e)
+        {
+
+        }
+    }
+
     public void testEventDispatchOrder()
     {
         EventListenerSupport<ActionListener> listenerSupport = 
EventListenerSupport.create(ActionListener.class);
@@ -51,9 +79,9 @@ public class EventListenerSupportTest ex
             EventListenerSupport.create(String.class);
             fail("Should not be able to create using non-interface class.");
         }
-        catch(IllegalArgumentException e)
+        catch (IllegalArgumentException e)
         {
-            
+
         }
     }
 
@@ -64,12 +92,12 @@ public class EventListenerSupportTest ex
             EventListenerSupport.create(null);
             fail("Should not be able to create using null class.");
         }
-        catch(NullPointerException e)
+        catch (NullPointerException e)
         {
 
         }
     }
-    
+
     public void testRemoveListenerDuringEvent()
     {
         final EventListenerSupport<ActionListener> listenerSupport = 
EventListenerSupport.create(ActionListener.class);


Reply via email to