Fix filter/args parsing for nested objects

Author: Jakub Knejzlik

index.js

@@ -43,7 +43,9 @@ function parceFind(_levelA) {
 //=====================================================
 
 function getGraphQLValue(value) {
-  if ("string" === typeof value) {
+  if (value === null) {
+    return value;
+  } else if ("string" === typeof value) {
     value = JSON.stringify(value);
   } else if (Array.isArray(value)) {
     value = value
@@ -57,6 +59,7 @@ function getGraphQLValue(value) {
             value = value.toSource().slice(2,-2);
         else*/
     value = objectToString(value);
+    if (value === "{}") return undefined;
     //console.error("No toSource!!",value);
   }
   return value;
@@ -70,7 +73,10 @@ function objectToString(obj) {
       continue;
     }
     // if ("object" === typeof obj[prop]) {
-    sourceA.push(`${prop}:${getGraphQLValue(obj[prop])}`);
+    let value = getGraphQLValue(obj[prop]);
+    if (typeof value !== "undefined") {
+      sourceA.push(`${prop}:${value}`);
+    }
     // } else {
     //      sourceA.push(`${prop}:${obj[prop]}`);
     // }
@@ -95,7 +101,9 @@ function Query(_fnNameS, _aliasS_OR_Filter) {
       if ("{}" === val) {
         continue;
       }
-      this.headA.push(`${propS}:${val}`);
+      if (typeof val !== "undefined") {
+        this.headA.push(`${propS}:${val}`);
+      }
     }
     return this;
   };

test.js

@@ -169,6 +169,22 @@ describe("graphql query builder", function() {
     expect(removeSpaces(expeted)).to.equal(removeSpaces(ItemQuery));
   });
 
+  it("should skip empty objects in filter/args for nested objects", function() {
+    let expeted =
+      'inventory(toy:"jack in the box",input:{blah:"foo",nullTest:null})  { id }';
+
+    let ChildsToy = {
+      toy: "jack in the box",
+      utils: {},
+      input: { blah: "foo", nullTest: null, objectTest: {} }
+    };
+
+    let ItemQuery = new Query("inventory", ChildsToy);
+    ItemQuery.find("id");
+
+    expect(removeSpaces(expeted)).to.equal(removeSpaces(ItemQuery));
+  });
+
   it("should throw Error if find input items have zero props", function() {
     expect(() => new Query("x").find({})).to.throw(Error);
   });