readsBlockSequence test

decorators-squad · May 29, 2018 · 2f784ac · 2f784ac
1 parent cf89666
commit 2f784ac
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 5 deletions.
diff --git a/src/main/java/com/amihaiemil/camel/AbstractYamlSequence.java b/src/main/java/com/amihaiemil/camel/AbstractYamlSequence.java
@@ -102,10 +102,10 @@ public int compareTo(final YamlNode other) {
             } else if (nodes.size() < others.size()) {
                 result = -1;
             } else {
-                final Iterator<YamlNode> ietrator = others.iterator();
+                final Iterator<YamlNode> iterator = others.iterator();
                 final Iterator<YamlNode> here = nodes.iterator();
-                while(ietrator.hasNext()) {
-                    result = here.next().compareTo(ietrator.next());
+                while(iterator.hasNext()) {
+                    result = here.next().compareTo(iterator.next());
                     if(result != 0) {
                         break;
                     }

diff --git a/src/test/java/com/amihaiemil/camel/RtYamlInputTest.java b/src/test/java/com/amihaiemil/camel/RtYamlInputTest.java
@@ -1,5 +1,6 @@
 package com.amihaiemil.camel;
 
+import java.io.ByteArrayInputStream;
 import java.io.File;
 import java.io.FileInputStream;
 
@@ -67,4 +68,25 @@ public void readsMappingWithoutComments() throws Exception {
         );
     }
 
+    /**
+     * A YamlSequence in block style can be read.
+     * @throws Exception If something goes wrong.
+     */
+    @Test
+    public void readsBlockSequence() throws Exception {
+        YamlSequence expected = Yaml.createYamlSequenceBuilder()
+            .add("apples")
+            .add("pears")
+            .add("peaches")
+            .build();
+
+        YamlSequence read = new RtYamlInput(
+            new ByteArrayInputStream(
+                "- apples\n- pears\n- peaches".getBytes()
+            )
+        ).readYamlSequence();
+
+        MatcherAssert.assertThat(read, Matchers.equalTo(expected));
+    }
+
 }
diff --git a/src/test/java/com/amihaiemil/camel/RtYamlSequenceTest.java b/src/test/java/com/amihaiemil/camel/RtYamlSequenceTest.java
@@ -35,7 +35,6 @@
 import java.util.Iterator;
 import java.util.LinkedList;
 import java.util.List;
-
 import org.apache.commons.io.IOUtils;
 import org.hamcrest.MatcherAssert;
 import org.hamcrest.Matchers;
@@ -63,7 +62,10 @@ public void fetchesChildren() {
         MatcherAssert.assertThat(
             seq.children(), Matchers.not(Matchers.emptyIterable())
         );
-        MatcherAssert.assertThat(seq.children().size(), Matchers.equalTo(3));
+        MatcherAssert.assertThat(
+            seq.children().size(),
+            Matchers.equalTo(3)
+        );        
     }
 
     /**