Issue 77: respect DumperOptions if used with Dumper
diff --git a/src/main/java/org/yaml/snakeyaml/Yaml.java b/src/main/java/org/yaml/snakeyaml/Yaml.java
index a265245..338521f 100644
--- a/src/main/java/org/yaml/snakeyaml/Yaml.java
+++ b/src/main/java/org/yaml/snakeyaml/Yaml.java
@@ -499,7 +499,7 @@
* @deprecated use with Constructor instead of Loader
*/
public Yaml(Loader loader, Dumper dumper, Resolver resolver) {
- this(loader.constructor, dumper.representer, new DumperOptions(), resolver);
+ this(loader.constructor, dumper.representer, dumper.options, resolver);
}
/**
diff --git a/src/test/java/org/yaml/snakeyaml/DumperTest.java b/src/test/java/org/yaml/snakeyaml/DumperTest.java
index 5587b73..9806852 100644
--- a/src/test/java/org/yaml/snakeyaml/DumperTest.java
+++ b/src/test/java/org/yaml/snakeyaml/DumperTest.java
@@ -26,6 +26,8 @@
import junit.framework.TestCase;
+import org.yaml.snakeyaml.resolver.Resolver;
+
public class DumperTest extends TestCase {
public void testDump1() {
@@ -97,8 +99,17 @@
}
@SuppressWarnings("deprecation")
- public void testDeprecated() {
+ public void testDeprecated1() {
Yaml yaml = new Yaml(new Dumper());
yaml.dump("aaa1234567890");
}
+
+ @SuppressWarnings("deprecation")
+ public void testDeprecated2() {
+ DumperOptions options = new DumperOptions();
+ options.setCanonical(true);
+ Yaml yaml = new Yaml(new Loader(), new Dumper(options), new Resolver());
+ String doc = yaml.dump("aaa1234567890");
+ assertEquals("---\n!!str \"aaa1234567890\"\n", doc);
+ }
}
