Query validation should pass if empty array of rules is provided

Author: vladar

src/Validator/DocumentValidator.php

@@ -118,8 +118,11 @@ public static function addRule($name, callable $rule)
 
     public static function validate(Schema $schema, DocumentNode $ast, array $rules = null, TypeInfo $typeInfo = null)
     {
+        if (null === $rules) {
+            $rules = static::allRules();
+        }
         $typeInfo = $typeInfo ?: new TypeInfo($schema);
-        $errors = static::visitUsingRules($schema, $typeInfo, $ast, $rules ?: static::allRules());
+        $errors = static::visitUsingRules($schema, $typeInfo, $ast, $rules);
         return $errors;
     }
 

tests/Validator/ValidationTest.php

@@ -23,4 +23,16 @@ public function testValidatesQueries()
           }
         ');
     }
+
+    public function testPassesValidationWithEmptyRules()
+    {
+        $query = '{invalid}';
+
+        $expectedError = [
+            'message' => 'Cannot query field "invalid" on type "QueryRoot".',
+            'locations' => [ ['line' => 1, 'column' => 2] ]
+        ];
+        $this->expectFailsCompleteValidation($query, [$expectedError]);
+        $this->expectValid($this->getDefaultSchema(), [], $query);
+    }
 }