fix!: lines should be empty in case of no output. closes #157

Signed-off-by: Joke de Buhr <[email protected]>

examples/slf4j/src/test/groovy/io/github/joke/test/EmptyLogs.groovy

@@ -0,0 +1,20 @@
+package io.github.joke.test
+
+import io.github.joke.spockoutputcapture.CapturedOutput
+import io.github.joke.spockoutputcapture.OutputCapture
+import spock.lang.Shared
+import spock.lang.Specification
+import spock.lang.Stepwise
+
+@Stepwise
+class EmptyLogs extends Specification {
+
+    @Shared @OutputCapture CapturedOutput globalLogs
+    @OutputCapture CapturedOutput localLogs
+
+    def 'empty logs'() {
+        expect:
+        globalLogs.lines.empty
+        localLogs.lines.empty
+    }
+}

spock-outputcapture/src/main/java/io/github/joke/spockoutputcapture/CapturedOutputImpl.java

@@ -1,11 +1,16 @@
 package io.github.joke.spockoutputcapture;
 
 import java.util.List;
+import java.util.regex.Pattern;
+import java.util.stream.Collectors;
 
 import static java.util.Arrays.asList;
+import static java.util.stream.Collectors.toList;
 
 public class CapturedOutputImpl implements CapturedOutput {
 
+    private static final Pattern NEWLINES = Pattern.compile("(\\r\\n|\\n|\\r)");
+
     private final StringBuffer buffer = new StringBuffer();
 
     @Override
@@ -15,7 +20,8 @@ public String toString() {
 
     @Override
     public List<String> getLines() {
-        return asList(buffer.toString().split("(\\r\\n|\\n|\\r)"));
+        return NEWLINES.splitAsStream(buffer.toString())
+                .collect(toList());
     }
 
     @Override