Optional JsonInclude.NON_NULL actually prints null on empty values.

nbauernfeind · nbauernfeind · commit c889d3f2fbb3 · 2016-04-18T10:11:34.000-04:00
See https://github.com/FasterXML/jackson-datatype-jdk8/issues/1 for more information.
diff --git a/src/test/scala/com/fasterxml/jackson/module/scala/ser/OptionSerializerTest.scala b/src/test/scala/com/fasterxml/jackson/module/scala/ser/OptionSerializerTest.scala
@@ -90,9 +90,10 @@ class OptionSerializerTest extends SerializerTest {
   }
 
   it should "honor JsonInclude.Include.NON_NULL" in {
-    val nonNullMapper = newMapper
-    nonNullMapper.setSerializationInclusion(JsonInclude.Include.NON_NULL)
-    nonNullMapper.writeValueAsString(new NonNullOption()) should be ("{}")
+    // See https://github.com/FasterXML/jackson-datatype-jdk8/issues/1 for more information.
+    newMapper
+      .setSerializationInclusion(JsonInclude.Include.NON_NULL)
+      .writeValueAsString(new NonNullOption()) should be ("""{"foo":null}""")
   }
 
   it should "generate correct schema for options" in {

Original file line number	Diff line number	Diff line change
`@@ -90,9 +90,10 @@ class OptionSerializerTest extends SerializerTest {`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`it should "honor JsonInclude.Include.NON_NULL" in {`
`93`		`- val nonNullMapper = newMapper`
`94`		`- nonNullMapper.setSerializationInclusion(JsonInclude.Include.NON_NULL)`
`95`		`- nonNullMapper.writeValueAsString(new NonNullOption()) should be ("{}")`
	`93`	`+ // See https://github.com/FasterXML/jackson-datatype-jdk8/issues/1 for more information.`
	`94`	`+ newMapper`
	`95`	`+ .setSerializationInclusion(JsonInclude.Include.NON_NULL)`
	`96`	`+ .writeValueAsString(new NonNullOption()) should be ("""{"foo":null}""")`
`96`	`97`	`}`
`97`	`98`
`98`	`99`	`it should "generate correct schema for options" in {`