Do not return empty arrays with GraphQL filter

ttu · Sep 30, 2018 · 221dd91 · 221dd91
1 parent 65afa4b
commit 221dd91
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,7 @@
 * Fix for HEAD method use and ETag to support HEAD method
 * .NET Core version to 2.1
 * Support for objects
+* Do not return empty arrays with GraphQL filter
 
 ### [0.6.0] - 2017-11-30
 * Caching of unchanged resources with ETag and If-None-Match headers

diff --git a/FakeServer.Test/GraphQLTests.cs b/FakeServer.Test/GraphQLTests.cs
@@ -33,6 +33,8 @@ public void Query_Families()
 
             Assert.NotEmpty(js);
 
+            Assert.Equal(4, results.Data["families"].Count);
+
             UTHelpers.Down(filePath);
         }
 

diff --git a/FakeServer/GraphQL/GraphQL.cs b/FakeServer/GraphQL/GraphQL.cs
@@ -284,7 +284,7 @@ private static ExecutionResult ResolveField(dynamic source, dynamic target, Fiel
                         var r = ResolveField(item, rootObject, i.Value);
                         rootObject = r.Data;
                     }
-
+                    
                     if (rootObject != null)
                         result.Add(rootObject);
                 }
@@ -332,6 +332,9 @@ private static ExecutionResult ResolveField(dynamic source, dynamic target, Fiel
                         if (rootObject != null)
                             newArray.Add(rootObject);
                     }
+
+                    if (f.Arguments.Any() && newArray.Count == 0)
+                        target = null;
                 }
                 else
                 {