[jbosscache-commits] JBoss Cache SVN: r4630 - core/trunk/src/test/java/org/jboss/cache/api.

[jbosscache-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2007-10-17 10:56:50 -0400 (Wed, 17 Oct 2007)
New Revision: 4630

Modified:
   core/trunk/src/test/java/org/jboss/cache/api/CacheAPITest.java
Log:
Add minor assertion that Cache.removeNode removes nodes that have children

Modified: core/trunk/src/test/java/org/jboss/cache/api/CacheAPITest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/api/CacheAPITest.java	2007-10-17 03:33:42 UTC (rev 4629)
+++ core/trunk/src/test/java/org/jboss/cache/api/CacheAPITest.java	2007-10-17 14:56:50 UTC (rev 4630)
@@ -184,7 +184,7 @@
    /**
     * Another convenience method that tests node removal
     */
-   public void nodeConvenienceNodeRemoval()
+   public void testNodeConvenienceNodeRemoval()
    {
       // this fqn is relative, but since it is from the root it may as well be absolute
       Fqn<String> fqn = Fqn.fromString("/test/fqn");
@@ -194,6 +194,15 @@
       assertEquals(true, cache.removeNode(fqn));
       assertFalse(cache.getRoot().hasChild(fqn));
       assertEquals(false, cache.removeNode(fqn));
+      
+      // Check that it's removed if it has a child
+      Fqn<String> child = Fqn.fromString("/test/fqn/child");
+      cache.getRoot().addChild(child);
+      assertTrue(cache.getRoot().hasChild(child));
+
+      assertEquals(true, cache.removeNode(fqn));
+      assertFalse(cache.getRoot().hasChild(fqn));
+      assertEquals(false, cache.removeNode(fqn));
    }
 
    /**