Author: sback
Date: 2007-07-05 07:49:24 +0000 (Thu, 05 Jul 2007)
New Revision: 13944

Modified:
   trunk/freenet/test/freenet/support/SimpleFieldSetTest.java
Log:
Removed some obscurity from the code, thanks to toad

Modified: trunk/freenet/test/freenet/support/SimpleFieldSetTest.java
===================================================================
--- trunk/freenet/test/freenet/support/SimpleFieldSetTest.java  2007-07-05 
07:41:11 UTC (rev 13943)
+++ trunk/freenet/test/freenet/support/SimpleFieldSetTest.java  2007-07-05 
07:49:24 UTC (rev 13944)
@@ -481,13 +481,15 @@

        /**
         * Tests directSubsetNameIterator() method.
+        * It uses SAMPLE_STRING_PAIRS and for this reason
+        * the expected subset is "foo".
         */
        public void testDirectSubsetNameIterator() {
                SimpleFieldSet methodSFS = sfsFromSampleStringPairs();
+               String expectedSubset = SAMPLE_STRING_PAIRS[0][0];      //"foo"
                Iterator methodIter = methodSFS.directSubsetNameIterator();
                while (methodIter.hasNext())
-                       
((String)methodIter.next()).equals(SAMPLE_STRING_PAIRS[0][0]);
-
+                       ((String)methodIter.next()).equals(expectedSubset);
                methodSFS = new SimpleFieldSet(true);
                methodIter = methodSFS.directSubsetNameIterator();
                assertNull(methodIter);
@@ -505,11 +507,6 @@
                assertNull(methodSFS.namesOfDirectSubsets());
        }

-       public void testOrderedString() {
-               SimpleFieldSet methodSFS = sfsFromSampleStringPairs();
-               System.out.println(methodSFS.toOrderedString());
-       }
-       
        /**
         * Test the putOverwrite(String,String) method.
         */


Reply via email to