You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by tn...@apache.org on 2013/04/25 22:48:08 UTC

svn commit: r1475953 - /commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java

Author: tn
Date: Thu Apr 25 20:48:08 2013
New Revision: 1475953

URL: http://svn.apache.org/r1475953
Log:
Reorder tests, remove TODO as it should be fine imho.

Modified:
    commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java

Modified: commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java?rev=1475953&r1=1475952&r2=1475953&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java (original)
+++ commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/bag/TreeBagTest.java Thu Apr 25 20:48:08 2013
@@ -37,17 +37,6 @@ public class TreeBagTest<T> extends Abst
         return new TreeBag<T>();
     }
 
-   // TODO: Generics (for example... is this even needed?)
-   public void testCollections265() {
-       final Bag<Object> bag = new TreeBag<Object>();
-       try {
-           bag.add(new Object());
-           fail("IllegalArgumentException expected");
-       } catch(final IllegalArgumentException iae) {
-           // expected;
-       }
-   }
-   
     @SuppressWarnings("unchecked")
     public SortedBag<T> setupBag() {
         final SortedBag<T> bag = makeObject();
@@ -58,6 +47,16 @@ public class TreeBagTest<T> extends Abst
         return bag;
     }
 
+    public void testCollections265() {
+        final Bag<Object> bag = new TreeBag<Object>();
+        try {
+            bag.add(new Object());
+            fail("IllegalArgumentException expected");
+        } catch(final IllegalArgumentException iae) {
+            // expected;
+        }
+    }
+    
     public void testOrdering() {
         final Bag<T> bag = setupBag();
         assertEquals("Should get elements in correct order", "A", bag.toArray()[0]);