Add failing tests wrt #245

Author: cowtowncoder

ion/src/main/java/com/fasterxml/jackson/dataformat/ion/IonGenerator.java

@@ -286,9 +286,7 @@ public void annotateNextValue(String annotation) {
         _writer.addTypeAnnotation(annotation);
     }
 
-    /* Ion Exentions
-     *
-     */
+    // // // Ion Extensions
 
     public void writeDate(Calendar value) throws JsonGenerationException, IOException {
         _verifyValueWrite("write date value");

ion/src/test/java/com/fasterxml/jackson/dataformat/ion/failing/PrettyPrintWriteTest.java

@@ -0,0 +1,35 @@
+package com.fasterxml.jackson.dataformat.ion.failing;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+import com.fasterxml.jackson.annotation.JsonPropertyOrder;
+import com.fasterxml.jackson.dataformat.ion.IonFactory;
+import com.fasterxml.jackson.dataformat.ion.IonObjectMapper;
+
+// For [dataformats-binary#245]: no pretty-printing for textual format
+public class PrettyPrintWriteTest
+{
+    @JsonPropertyOrder({ "x", "y" })
+    static class Point {
+        public int x = 1;
+        public int y = 2;
+    }
+
+    @Test
+    public void testBasicPrettyPrintTextual() throws Exception
+    {
+        IonObjectMapper mapper = IonObjectMapper.builder(IonFactory.forTextualWriters()).build();
+        Assert.assertEquals("{\n  x:1,\n  y:2\n}",
+                mapper.writerWithDefaultPrettyPrinter().writeValueAsString(new Point()));
+    }
+
+    // and with binary format, should simply be no-op
+    @Test
+    public void testIgnorePrettyPrintForBinary() throws Exception
+    {
+        IonObjectMapper mapper = IonObjectMapper.builder(IonFactory.forBinaryWriters()).build();
+        byte[] encoded = mapper.writerWithDefaultPrettyPrinter().writeValueAsBytes(new Point());
+        Assert.assertNotNull(encoded);
+    }
+}