Merge pull request #51 from graphql-python/releases/0.4.14

Way to 0.4.14

Author: syrusakbary

graphql/core/language/visitor.py

@@ -5,8 +5,17 @@
 from . import ast
 from .visitor_meta import QUERY_DOCUMENT_KEYS, VisitorMeta
 
+
+class Falsey(object):
+    def __nonzero__(self):
+        return False
+
+    def __bool__(self):
+        return False
+
+
 BREAK = object()
-REMOVE = object()
+REMOVE = Falsey()
 
 
 class Stack(object):
@@ -90,7 +99,7 @@ def visit(root, visitor, key_map=None):
                 key = None
                 node = new_root
 
-            if node is None:
+            if node is REMOVE or node is None:
                 continue
 
             if parent:
@@ -162,3 +171,55 @@ def leave(self, node, key, parent, path, ancestors):
         method = self._get_leave_handler(type(node))
         if method:
             return method(self, node, key, parent, path, ancestors)
+
+
+class ParallelVisitor(Visitor):
+    __slots__ = 'skipping', 'visitors'
+
+    def __init__(self, visitors):
+        self.visitors = visitors
+        self.skipping = [None] * len(visitors)
+
+    def enter(self, node, key, parent, path, ancestors):
+        for i, visitor in enumerate(self.visitors):
+            if not self.skipping[i]:
+                result = visitor.enter(node, key, parent, path, ancestors)
+                if result is False:
+                    self.skipping[i] = node
+                elif result is BREAK:
+                    self.skipping[i] = BREAK
+                elif result is not None:
+                    return result
+
+    def leave(self, node, key, parent, path, ancestors):
+        for i, visitor in enumerate(self.visitors):
+            if not self.skipping[i]:
+                result = visitor.leave(node, key, parent, path, ancestors)
+                if result is BREAK:
+                    self.skipping[i] = BREAK
+                elif result is not None and result is not False:
+                    return result
+            elif self.skipping[i] == node:
+                self.skipping[i] = REMOVE
+
+
+class TypeInfoVisitor(Visitor):
+    __slots__ = 'visitor', 'type_info'
+
+    def __init__(self, type_info, visitor):
+        self.type_info = type_info
+        self.visitor = visitor
+
+    def enter(self, node, key, parent, path, ancestors):
+        self.type_info.enter(node)
+        result = self.visitor.enter(node, key, parent, path, ancestors)
+        if result is not None:
+            self.type_info.leave(node)
+            if isinstance(result, ast.Node):
+                self.type_info.enter(result)
+        return result
+
+    def leave(self, node, key, parent, path, ancestors):
+        result = self.visitor.leave(node, key, parent, path, ancestors)
+        self.type_info.leave(node)
+        return result

graphql/core/type/__init__.py

@@ -12,6 +12,7 @@
     GraphQLInputObjectField,
     GraphQLList,
     GraphQLNonNull,
+    get_named_type,
     is_abstract_type,
     is_composite_type,
     is_input_type,

graphql/core/type/scalars.py

@@ -3,11 +3,13 @@
 from ..language.ast import BooleanValue, FloatValue, IntValue, StringValue
 from .definition import GraphQLScalarType
 
-# Integers are only safe when between -(2^53 - 1) and 2^53 - 1 due to being
-# encoded in JavaScript and represented in JSON as double-precision floating
-# point numbers, as specified by IEEE 754.
-MAX_INT = 9007199254740991
-MIN_INT = -9007199254740991
+# As per the GraphQL Spec, Integers are only treated as valid when a valid
+# 32-bit signed integer, providing the broadest support across platforms.
+#
+# n.b. JavaScript's integers are safe between -(2^53 - 1) and 2^53 - 1 because
+# they are internally represented as IEEE 754 doubles.
+MAX_INT = 2147483647
+MIN_INT = -2147483648
 
 
 def coerce_int(value):

graphql/core/validation/__init__.py

@@ -1,9 +1,8 @@
-from ..language.visitor import visit
+from ..language.visitor import ParallelVisitor, TypeInfoVisitor, visit
 from ..type import GraphQLSchema
 from ..utils.type_info import TypeInfo
 from .context import ValidationContext
 from .rules import specified_rules
-from .visitor import ParallelVisitor, TypeInfoVisitor
 
 
 def validate(schema, ast, rules=specified_rules):

graphql/core/validation/context.py

@@ -1,7 +1,6 @@
 from ..language.ast import (FragmentDefinition, FragmentSpread,
                             OperationDefinition)
-from ..language.visitor import Visitor, visit
-from .visitor import TypeInfoVisitor
+from ..language.visitor import TypeInfoVisitor, Visitor, visit
 
 
 class VariableUsage(object):

graphql/core/validation/rules/known_type_names.py

@@ -4,6 +4,18 @@
 
 class KnownTypeNames(ValidationRule):
 
+    def enter_ObjectTypeDefinition(self, node, *args):
+        return False
+
+    def enter_InterfaceTypeDefinition(self, node, *args):
+        return False
+
+    def enter_UnionTypeDefinition(self, node, *args):
+        return False
+
+    def enter_InputObjectTypeDefinition(self, node, *args):
+        return False
+
     def enter_NamedType(self, node, *args):
         type_name = node.name.value
         type = self.context.get_schema().get_type(type_name)