Fix Python 3.7 deprecation warnings

Importing ABCs directly from the `collections` module is deprecated
in Python 3.7.

Author: elprans

graphql/execution/executor.py

@@ -1,4 +1,5 @@
 import collections
+import collections.abc
 import functools
 import logging
 import sys
@@ -580,7 +581,7 @@ def complete_list_value(
     """
     Complete a list value by completing each item in the list with the inner type
     """
-    assert isinstance(result, collections.Iterable), (
+    assert isinstance(result, collections.abc.Iterable), (
         "User Error: expected iterable, but did not find one " + "for field {}.{}."
     ).format(info.parent_type, info.field_name)
 

graphql/execution/values.py

@@ -1,4 +1,4 @@
-import collections
+import collections.abc
 import json
 
 from six import string_types
@@ -160,7 +160,7 @@ def coerce_value(type, value):
     if isinstance(type, GraphQLList):
         item_type = type.of_type
         if not isinstance(value, string_types) and isinstance(
-            value, collections.Iterable
+            value, collections.abc.Iterable
         ):
             return [coerce_value(item_type, item) for item in value]
         else:

graphql/type/definition.py

@@ -1,4 +1,5 @@
 import collections
+import collections.abc
 import copy
 
 from ..language import ast
@@ -234,7 +235,7 @@ def define_field_map(
     if callable(field_map):
         field_map = field_map()
 
-    assert isinstance(field_map, collections.Mapping) and len(field_map) > 0, (
+    assert isinstance(field_map, collections.abc.Mapping) and len(field_map) > 0, (
         "{} fields must be a mapping (dict / OrderedDict) with field names as keys or a "
         "function which returns such a mapping."
     ).format(type)
@@ -248,7 +249,7 @@ def define_field_map(
 
         if field_args:
             assert isinstance(
-                field_args, collections.Mapping
+                field_args, collections.abc.Mapping
             ), "{}.{} args must be a mapping (dict / OrderedDict) with argument names as keys.".format(
                 type, field_name
             )
@@ -520,15 +521,15 @@ def serialize(self, value):
         if isinstance(value, PyEnum):
             # We handle PyEnum values
             value = value.value
-        if isinstance(value, collections.Hashable):
+        if isinstance(value, collections.abc.Hashable):
             enum_value = self._value_lookup.get(value)
             if enum_value:
                 return enum_value.name
 
         return None
 
     def parse_value(self, value):
-        if isinstance(value, collections.Hashable):
+        if isinstance(value, collections.abc.Hashable):
             enum_value = self._name_lookup.get(value)
 
             if enum_value:
@@ -555,7 +556,7 @@ def _name_lookup(self):
 
 def define_enum_values(type, value_map):
     assert (
-        isinstance(value_map, collections.Mapping) and len(value_map) > 0
+        isinstance(value_map, collections.abc.Mapping) and len(value_map) > 0
     ), "{} values must be a mapping (dict / OrderedDict) with value names as keys.".format(
         type
     )
@@ -661,7 +662,7 @@ def _define_field_map(self):
         else:
             fields = self._fields
 
-        assert isinstance(fields, collections.Mapping) and len(fields) > 0, (
+        assert isinstance(fields, collections.abc.Mapping) and len(fields) > 0, (
             "{} fields must be a mapping (dict / OrderedDict) with field names as keys or a "
             "function which returns such a mapping."
         ).format(self)

graphql/type/directives.py

@@ -1,4 +1,4 @@
-import collections
+import collections.abc
 
 from ..pyutils.ordereddict import OrderedDict
 from ..utils.assert_valid_name import assert_valid_name
@@ -43,7 +43,7 @@ def __init__(self, name, description=None, args=None, locations=None):
         assert name, "Directive must be named."
         assert_valid_name(name)
         assert isinstance(
-            locations, collections.Iterable
+            locations, collections.abc.Iterable
         ), "Must provide locations for directive."
 
         self.name = name
@@ -52,7 +52,7 @@ def __init__(self, name, description=None, args=None, locations=None):
 
         if args:
             assert isinstance(
-                args, collections.Mapping
+                args, collections.abc.Mapping
             ), "{} args must be a dict with argument names as keys.".format(name)
             for arg_name, _arg in args.items():
                 assert_valid_name(arg_name)

graphql/type/schema.py

@@ -1,4 +1,4 @@
-from collections import Iterable
+from collections.abc import Iterable
 
 from .definition import GraphQLObjectType
 from .directives import GraphQLDirective, specified_directives

graphql/type/typemap.py

@@ -1,4 +1,5 @@
-from collections import OrderedDict, Sequence, defaultdict
+from collections import OrderedDict, defaultdict
+from collections.abc import Sequence
 from functools import reduce
 
 from ..utils.type_comparators import is_equal_type, is_type_sub_type_of

graphql/utils/is_valid_value.py

@@ -2,7 +2,7 @@
     Implementation of isValidJSValue from graphql.s
 """
 
-import collections
+import collections.abc
 import json
 
 from six import string_types
@@ -38,7 +38,7 @@ def is_valid_value(value, type):
     if isinstance(type, GraphQLList):
         item_type = type.of_type
         if not isinstance(value, string_types) and isinstance(
-            value, collections.Iterable
+            value, collections.abc.Iterable
         ):
             errors = []
             for i, item in enumerate(value):
@@ -52,7 +52,7 @@ def is_valid_value(value, type):
             return is_valid_value(value, item_type)
 
     if isinstance(type, GraphQLInputObjectType):
-        if not isinstance(value, collections.Mapping):
+        if not isinstance(value, collections.abc.Mapping):
             return [u'Expected "{}", found not an object.'.format(type)]
 
         fields = type.fields