Author: mgrigorov
Date: Sun Jul 10 10:31:15 2011
New Revision: 1144807

URL: http://svn.apache.org/viewvc?rev=1144807&view=rev
Log:
WICKET-3789 childFirst issue in "visitChildren and friends require inner 
classes. Replace with iterators"

Make it easier to read the expected hierarchy.



Modified:
    
wicket/trunk/wicket-core/src/test/java/org/apache/wicket/util/iterator/AbstractHierarchyIteratorTest.java

Modified: 
wicket/trunk/wicket-core/src/test/java/org/apache/wicket/util/iterator/AbstractHierarchyIteratorTest.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket-core/src/test/java/org/apache/wicket/util/iterator/AbstractHierarchyIteratorTest.java?rev=1144807&r1=1144806&r2=1144807&view=diff
==============================================================================
--- 
wicket/trunk/wicket-core/src/test/java/org/apache/wicket/util/iterator/AbstractHierarchyIteratorTest.java
 (original)
+++ 
wicket/trunk/wicket-core/src/test/java/org/apache/wicket/util/iterator/AbstractHierarchyIteratorTest.java
 Sun Jul 10 10:31:15 2011
@@ -313,18 +313,22 @@ public class AbstractHierarchyIteratorTe
 
                // Filter leaf components only
                int count = 0;
-               String buf = "";
+               StringBuilder buf = new StringBuilder();
                ComponentHierarchyIterator iter = new 
ComponentHierarchyIterator(page);
                iter.setChildFirst(true);
                while (iter.hasNext())
                {
                        Component component = iter.next();
                        count += 1;
-                       buf += component.getId();
+                       if (buf.length() > 0)
+                       {
+                               buf.append(Component.PATH_SEPARATOR);
+                       }
+                       buf.append(component.getId());
                }
 
                assertEquals(6, count);
-               assertEquals("ab121b12b1b2b", buf);
+               assertEquals("a:b121:b12:b1:b2:b", buf.toString());
        }
 
        /** */


Reply via email to