move unit test to the correct class, add positive test case FasterXML…

…#366
njank · Oct 4, 2024 · 671fa21 · 671fa21
1 parent 05c42bf
commit 671fa21
Show file tree

Hide file tree

Showing 2 changed files with 50 additions and 20 deletions.
diff --git a/.../src/test/java/com/fasterxml/jackson/dataformat/yaml/failing/SimpleGeneration215Test.java b/.../src/test/java/com/fasterxml/jackson/dataformat/yaml/failing/SimpleGeneration215Test.java
@@ -1,8 +1,6 @@
 package com.fasterxml.jackson.dataformat.yaml.failing;
 
 import java.util.Collections;
-import java.util.HashMap;
-import java.util.Map;
 
 import com.fasterxml.jackson.dataformat.yaml.ModuleTestBase;
 import com.fasterxml.jackson.dataformat.yaml.YAMLFactory;
@@ -34,22 +32,4 @@ public void testStartMarkerViaMapper() throws Exception
             .trim();
         assertEquals("key: \"value\"", output);
     }
-
-    public void testLiteralBlockStyleMultilineWithTrailingSpace() throws Exception
-    {
-        YAMLFactory f = new YAMLFactory();
-        // verify default settings
-        assertFalse(f.isEnabled(YAMLGenerator.Feature.LITERAL_BLOCK_STYLE));
-
-        YAMLMapper mapper = YAMLMapper.builder()
-                .configure(YAMLGenerator.Feature.LITERAL_BLOCK_STYLE, true)
-                .build();
-
-        Map<String, Object> content = new HashMap<String, Object>();
-        content.put("text", "Hello\nWorld ");
-        String yaml = mapper.writeValueAsString(content).trim();
-
-        assertEquals("---\n" +
-                "text: |-\n  Hello\n  World ", yaml);
-    }
 }
diff --git a/.../src/test/java/com/fasterxml/jackson/dataformat/yaml/failing/SimpleGeneration366Test.java b/.../src/test/java/com/fasterxml/jackson/dataformat/yaml/failing/SimpleGeneration366Test.java
@@ -0,0 +1,50 @@
+package com.fasterxml.jackson.dataformat.yaml.failing;
+
+import com.fasterxml.jackson.dataformat.yaml.ModuleTestBase;
+import com.fasterxml.jackson.dataformat.yaml.YAMLFactory;
+import com.fasterxml.jackson.dataformat.yaml.YAMLGenerator;
+import com.fasterxml.jackson.dataformat.yaml.YAMLMapper;
+
+import java.util.HashMap;
+import java.util.Map;
+
+public class SimpleGeneration366Test extends ModuleTestBase
+{
+    // [dataformats-text#366]: multiline literal block with trailing spaces does not work
+    public void testLiteralBlockStyleMultilineWithTrailingSpace() throws Exception
+    {
+        YAMLFactory f = new YAMLFactory();
+        // verify default settings
+        assertFalse(f.isEnabled(YAMLGenerator.Feature.LITERAL_BLOCK_STYLE));
+
+        YAMLMapper mapper = YAMLMapper.builder()
+                .configure(YAMLGenerator.Feature.LITERAL_BLOCK_STYLE, true)
+                .build();
+
+        Map<String, Object> content = new HashMap<String, Object>();
+        content.put("text", "Hello\nWorld ");
+        String yaml = mapper.writeValueAsString(content).trim();
+
+        assertEquals("---\n" +
+                "text: |-\n  Hello\n  World ", yaml);
+    }
+
+    // [dataformats-text#366]: multiline literal block without trailing spaces actually works
+    public void testLiteralBlockStyleMultiline() throws Exception
+    {
+        YAMLFactory f = new YAMLFactory();
+        // verify default settings
+        assertFalse(f.isEnabled(YAMLGenerator.Feature.LITERAL_BLOCK_STYLE));
+
+        YAMLMapper mapper = YAMLMapper.builder()
+                .configure(YAMLGenerator.Feature.LITERAL_BLOCK_STYLE, true)
+                .build();
+
+        Map<String, Object> content = new HashMap<String, Object>();
+        content.put("text", "Hello\nWorld");
+        String yaml = mapper.writeValueAsString(content).trim();
+
+        assertEquals("---\n" +
+                "text: |-\n  Hello\n  World", yaml);
+    }
+}