From d8613d228588ed53b3dd1c4fc5073dcd20fc0b37 Mon Sep 17 00:00:00 2001
From: zab <zab@mail.i2p>
Date: Fri, 23 Nov 2012 07:06:01 +0000
Subject: [PATCH] more appropriate junit test

---
 .../i2p/util/CachedIteratorArrayListTest.java | 36 ++++++++++++-------
 1 file changed, 23 insertions(+), 13 deletions(-)

diff --git a/core/java/test/junit/net/i2p/util/CachedIteratorArrayListTest.java b/core/java/test/junit/net/i2p/util/CachedIteratorArrayListTest.java
index 4ae8e182ef..3d6a1ce16b 100644
--- a/core/java/test/junit/net/i2p/util/CachedIteratorArrayListTest.java
+++ b/core/java/test/junit/net/i2p/util/CachedIteratorArrayListTest.java
@@ -10,33 +10,43 @@ import org.junit.Test;
 
 public class CachedIteratorArrayListTest {
 
-    private List<Integer> l;
+    private List<Character> l;
     
     @Before
     public void setUp() {
-        l = new CachedIteratorArrayList<Integer>();
-        l.add(1);
-        l.add(2);
-        l.add(3);
+        l = new CachedIteratorArrayList<Character>();
+        l.add('a');
+        l.add('b');
+        l.add('c');
     }
     
+    /** test iterations work */
     @Test
     public void test() {
-        // test for-each worx
-        int total = 0;
-        for (int i : l)
+        String total = "";
+        
+        // two full for-each iterations
+        for (char i : l)
             total += i;
-        assertEquals(6, total);
-        for (int i : l)
+        assertEquals("abc", total);
+        for (char i : l)
             total += i;
-        assertEquals(12, total);
+        assertEquals("abcabc", total);
         
+        // and one partial
+        total = "";
+        Iterator<Character> iter = l.iterator();
+        total += iter.next();
+        total += iter.next();
+        iter = l.iterator();
+        total += iter.next();
+        assertEquals("aba",total);
     }
     
     @Test
     public void testSameness() {
-        Iterator<Integer> two = l.iterator();
-        Iterator<Integer> one = l.iterator();
+        Iterator<Character> two = l.iterator();
+        Iterator<Character> one = l.iterator();
         assertSame(one, two);
     }
 
-- 
GitLab