Spec compliant Int sizing

Related GraphQL-js commit graphql/graphql-js@06f97b6

Author: syrusakbary

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):

tests/core_type/test_serialization.py

@@ -10,6 +10,8 @@ def test_serializes_output_int():
     assert GraphQLInt.serialize(1.1) == 1
     assert GraphQLInt.serialize(-1.1) == -1
     assert GraphQLInt.serialize(1e5) == 100000
+    assert GraphQLInt.serialize(9876504321) is None
+    assert GraphQLInt.serialize(-9876504321) is None
     assert GraphQLInt.serialize(1e100) is None
     assert GraphQLInt.serialize(-1e100) is None
     assert GraphQLInt.serialize('-1.1') == -1