#255: SplitText fixed

yegor256 · Jun 28, 2017 · 8af1906 · 8af1906
1 parent 0b97734
commit 8af1906
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 6 deletions.
diff --git a/src/main/java/org/cactoos/text/SplitText.java b/src/main/java/org/cactoos/text/SplitText.java
@@ -26,6 +26,7 @@
 import java.util.Iterator;
 import org.cactoos.Text;
 import org.cactoos.list.ArrayAsIterable;
+import org.cactoos.list.FilteredIterator;
 
 /**
  * Split the Text.
@@ -98,11 +99,14 @@ public SplitText(final UncheckedText text, final UncheckedText rgx) {
 
     @Override
     public Iterator<String> iterator() {
-        return new ArrayAsIterable<>(
-            this.origin.asString().split(
-                this.regex.asString()
-            )
-        ).iterator();
+        return new FilteredIterator<>(
+            new ArrayAsIterable<>(
+                this.origin.asString().split(
+                    this.regex.asString()
+                )
+            ).iterator(),
+            input -> !input.isEmpty()
+        );
     }
 
 }
diff --git a/src/test/java/org/cactoos/text/SplitTextTest.java b/src/test/java/org/cactoos/text/SplitTextTest.java
@@ -31,7 +31,7 @@
  * Test case for {@link SplitText}.
  * @author Alexey Semenyuk (semenyukalexey@gmail.com)
  * @version $Id$
- * @since 0.1
+ * @since 0.9
  * @checkstyle JavadocMethodCheck (500 lines)
  */
 public final class SplitTextTest {
@@ -50,4 +50,13 @@ public void splitText() throws Exception {
         );
     }
 
+    @Test
+    public void splitEmptyText() throws Exception {
+        MatcherAssert.assertThat(
+            "Can't split an empty text",
+            new SplitText("", "\n"),
+            Matchers.emptyIterable()
+        );
+    }
+
 }