removed Iterator.toString() as it changes the iterator state, the default is fine and folks should be explicit about iterating over an iterator. Also disabled the default limit on the iterator on join(); if folks want to limit the join they should be explicit

2012-03-30 10:24:27 +01:00
parent 25c0ed3279
commit fbd1279cc9
12 changed files with 45 additions and 24 deletions
@@ -52,8 +52,8 @@ class IteratorsTest {
    }

    Test fun toStringJoinsNoMoreThanTheFirstTenElements() {
-        assertEquals("0, 1, 1, 2, 3, 5, 8, 13, 21, 34, ...", fibonacci().toString())
-        assertEquals("13, 21, 34, 55, 89, 144, 233, 377, 610, 987, ...", fibonacci().filter {  it > 10 }.toString())
-        assertEquals("144, 233, 377, 610, 987", fibonacci().filter { it > 100 }.takeWhile { it < 1000 }.toString())
+        assertEquals("0, 1, 1, 2, 3, 5, 8, 13, 21, 34, ...", fibonacci().join(limit = 10))
+        assertEquals("13, 21, 34, 55, 89, 144, 233, 377, 610, 987, ...", fibonacci().filter {  it > 10 }.join(limit = 10))
+        assertEquals("144, 233, 377, 610, 987", fibonacci().filter { it > 100 }.takeWhile { it < 1000 }.join())
    }
 }