bit more testing wrt #888

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/databind/ser/BasicSerializerFactory.java

@@ -869,14 +869,13 @@ protected MapSerializer _checkMapContentInclusion(SerializerProvider prov,
         case NON_NULL:
             valueToSuppress = null;
             suppressNulls = true;
-            // fall through
+            break;
         case ALWAYS: // default
         default:
             valueToSuppress = null;
             suppressNulls = !prov.isEnabled(SerializationFeature.WRITE_NULL_MAP_VALUES);
             break;
         }
-
         return mapSer.withContentInclusion(valueToSuppress, suppressNulls);
     }
 

src/main/java/com/fasterxml/jackson/databind/ser/std/MapSerializer.java

@@ -281,12 +281,12 @@ public MapSerializer withFilterId(Object filterId) {
      * @since 2.9
      */
     public MapSerializer withContentInclusion(Object suppressableValue, boolean suppressNulls) {
-        if ((suppressableValue == _suppressableValue) && (_suppressNulls == suppressNulls)) {
+        if ((suppressableValue == _suppressableValue) && (suppressNulls == _suppressNulls)) {
             return this;
         }
         _ensureOverride();
         return new MapSerializer(this, _valueTypeSerializer, suppressableValue, suppressNulls);
-    }                
+    }
 
     /**
      * @since 2.8
@@ -496,12 +496,12 @@ public JsonSerializer<?> createContextual(SerializerProvider provider,
                     case NON_NULL:
                         valueToSuppress = null;
                         suppressNulls = true;
-                        // fall through
+                        break;
                     case ALWAYS: // default
                     default:
                         valueToSuppress = null;
-                        // 30-Sep-2016, tatu: Should not check global flags here, if inclusion forced
-                        //    to be ALWAYS
+                        // 30-Sep-2016, tatu: Should not need to check global flags here,
+                        //   if inclusion forced to be ALWAYS
                         suppressNulls = false;
                         break;
                     }

src/test/java/com/fasterxml/jackson/databind/filter/TestMapFiltering.java

@@ -243,4 +243,31 @@ public void testMapWithOnlyEmptyValues() throws IOException
             .add("b", null)));
         assertEquals(aposToQuotes("{}"), json);
     }
+
+    public void testMapViaGlobalNonEmpty() throws Exception
+    {
+        // basic Map<String,String> subclass:
+        ObjectMapper mapper = new ObjectMapper();
+        mapper.setPropertyInclusion(JsonInclude.Value.empty()
+                .withContentInclusion(JsonInclude.Include.NON_EMPTY));
+        assertEquals(aposToQuotes("{'a':'b'}"), mapper.writeValueAsString(
+                new StringMap497()
+                    .add("x", "")
+                    .add("a", "b")
+                    ));
+    }
+
+    public void testMapViaTypeOverride() throws Exception
+    {
+        // basic Map<String,String> subclass:
+        ObjectMapper mapper = new ObjectMapper();
+        mapper.configOverride(Map.class)
+            .setInclude(JsonInclude.Value.empty()
+                .withContentInclusion(JsonInclude.Include.NON_EMPTY));
+        assertEquals(aposToQuotes("{'a':'b'}"), mapper.writeValueAsString(
+                new StringMap497()
+                    .add("foo", "")
+                    .add("a", "b")
+                    ));
+    }
 }