Merge pull request #18 from dotarj/fix-empty-array

fix empty array is removed from json response #11

Author: dotarj

src/PartialResponse.AspNetCore.Mvc.Formatters.Json/Internal/JsonSerializerExtensions.cs

@@ -62,11 +62,17 @@ public static void Serialize(this JsonSerializer jsonSerializer, JsonWriter json
 
         private static void RemoveArrayElements(JArray array, string currentPath, SerializerContext context)
         {
+            var containsChildItems = array.Count > 0;
+
             array.OfType<JObject>()
                 .ToList()
                 .ForEach(childObject => RemoveObjectProperties(childObject, currentPath, context));
 
-            RemoveArrayIfEmpty(array);
+            // PartialResponse should only remove an array when it initially contained items, but was empty after filtering.
+            if (containsChildItems)
+            {
+                RemoveArrayIfEmpty(array);
+            }
         }
 
         private static void RemoveArrayIfEmpty(JArray array)

test/PartialResponse.AspNetCore.Mvc.Formatters.Json.Test/JsonSerializerExtensionsTests.cs

@@ -174,5 +174,18 @@ public void TheSerializeMethodFilterCorrectlyUsingCache()
             // Assert
             Assert.Equal("[{\"foo\":\"bar\"},{\"foo\":\"bar\"}]", result.ToString());
         }
+
+        [Fact]
+        public void TheSerializeMethodShouldFilterArrayElementInsideArrayElement2()
+        {
+            // Arrange
+            var value = new { totalCount = 0, data = new object[0] };
+
+            // Act
+            jsonSerializer.Serialize(this.jsonWriter, value, _ => true);
+
+            // Assert
+            Assert.Equal("{\"totalCount\":0,\"data\":[]}", result.ToString());
+        }
     }
 }