Better type checking when creating GraphQL types

Also add tests for the type checks, improving coverage.

Author: Cito

README.md

@@ -13,7 +13,7 @@ a query language for APIs created by Facebook.
 [![Code Style](https://img.shields.io/badge/code%20style-black-000000.svg)](https://github.com/ambv/black)
 
 The current version 1.0.5 of GraphQL-core-next is up-to-date with GraphQL.js version
-14.3.1. All parts of the API are covered by an extensive test suite of currently 1825
+14.3.1. All parts of the API are covered by an extensive test suite of currently 1877
 unit tests.
 
 

graphql/__init__.py

@@ -53,7 +53,6 @@
     GraphQLScalarType,
     GraphQLObjectType,
     GraphQLInterfaceType,
-    GraphQLInterfaceList,
     GraphQLUnionType,
     GraphQLEnumType,
     GraphQLInputObjectType,
@@ -134,7 +133,6 @@
     assert_valid_schema,
     # Types
     GraphQLType,
-    GraphQLTypeList,
     GraphQLInputType,
     GraphQLOutputType,
     GraphQLLeafType,
@@ -406,7 +404,6 @@
     "GraphQLScalarType",
     "GraphQLObjectType",
     "GraphQLInterfaceType",
-    "GraphQLInterfaceList",
     "GraphQLUnionType",
     "GraphQLEnumType",
     "GraphQLInputObjectType",
@@ -476,7 +473,6 @@
     "validate_schema",
     "assert_valid_schema",
     "GraphQLType",
-    "GraphQLTypeList",
     "GraphQLInputType",
     "GraphQLOutputType",
     "GraphQLLeafType",

graphql/error/invalid.py

@@ -10,6 +10,9 @@ def __repr__(self):
     def __str__(self):
         return "INVALID"
 
+    def __hash__(self):
+        return hash(InvalidType)
+
     def __bool__(self):
         return False
 

graphql/pyutils/frozen_dict.py

@@ -1,9 +1,14 @@
-__all__ = ["FrozenDict"]
+from typing import Dict, TypeVar
 
 from .frozen_error import FrozenError
 
+__all__ = ["FrozenDict"]
+
+K = TypeVar("K")
+T = TypeVar("T", covariant=True)
+
 
-class FrozenDict(dict):
+class FrozenDict(Dict[K, T]):
     """Dictionary that can only be read, but not changed."""
 
     def __delitem__(self, key):

graphql/pyutils/frozen_list.py

@@ -1,9 +1,14 @@
-__all__ = ["FrozenList"]
+from typing import List, TypeVar
 
 from .frozen_error import FrozenError
 
+__all__ = ["FrozenList"]
+
+
+T = TypeVar("T", covariant=True)
+
 
-class FrozenList(list):
+class FrozenList(List[T]):
     """List that can only be read, but not changed."""
 
     def __delitem__(self, key):

graphql/type/__init__.py

@@ -58,7 +58,6 @@
     GraphQLScalarType,
     GraphQLObjectType,
     GraphQLInterfaceType,
-    GraphQLInterfaceList,
     GraphQLUnionType,
     GraphQLEnumType,
     GraphQLInputObjectType,
@@ -67,7 +66,6 @@
     GraphQLNonNull,
     # Types
     GraphQLType,
-    GraphQLTypeList,
     GraphQLInputType,
     GraphQLOutputType,
     GraphQLLeafType,
@@ -187,7 +185,6 @@
     "GraphQLScalarType",
     "GraphQLObjectType",
     "GraphQLInterfaceType",
-    "GraphQLInterfaceList",
     "GraphQLUnionType",
     "GraphQLEnumType",
     "GraphQLInputObjectType",
@@ -196,7 +193,6 @@
     "GraphQLList",
     "GraphQLNonNull",
     "GraphQLType",
-    "GraphQLTypeList",
     "GraphQLInputType",
     "GraphQLOutputType",
     "GraphQLLeafType",