Update mypy and check tests

Author: Cito

.mypy.ini

@@ -8,9 +8,9 @@ include .bumpversion.cfg
 include .coveragerc
 include .editorconfig
 include .flake8
-include .mypy.ini
 
 include codecov.yml
+include mypy.ini
 include tox.ini
 
 include poetry.lock

MANIFEST.in

@@ -0,0 +1,6 @@
+[mypy]
+python_version = 3.7
+warn_redundant_casts = true
+warn_unused_ignores = True
+# https://github.com/python/mypy/issues/7203
+new_semantic_analyzer = False

mypy.ini

@@ -36,7 +36,7 @@ pytest-describe = ">=0.12"
 pyyaml = "^5.1"
 black = ">=19.3b0"
 flake8 = "^3.7"
-mypy = ">=0.711"
+mypy = ">=0.720"
 codecov = "^2"
 sphinx = "^2.1"
 sphinx_rtd_theme = ">=0.4"

poetry.lock

@@ -62,7 +62,6 @@ async def subscribe(
         return ExecutionResult(data=None, errors=[error])
     if isinstance(result_or_stream, ExecutionResult):
         return result_or_stream
-    result_or_stream = cast(AsyncIterable, result_or_stream)
 
     async def map_source_to_response(payload):
         """Map source to response.
@@ -153,7 +152,6 @@ async def create_source_event_stream(
     # Call the `subscribe()` resolver or the default resolver to produce an
     # AsyncIterable yielding raw payloads.
     resolve_fn = field_def.subscribe or context.field_resolver
-    resolve_fn = cast(GraphQLFieldResolver, resolve_fn)  # help mypy
 
     path = add_path(None, response_name)
 
@@ -172,7 +170,7 @@ async def create_source_event_stream(
 
     # Assert field returned an event stream, otherwise yield an error.
     if isinstance(event_stream, AsyncIterable):
-        return cast(AsyncIterable, event_stream)
+        return event_stream
     raise TypeError(
         f"Subscription field must return AsyncIterable. Received: {event_stream!r}"
     )

pyproject.toml

@@ -449,7 +449,7 @@ def __init__(
             )
         else:
             args = {
-                name: cast(GraphQLArgument, value)
+                name: value
                 if isinstance(value, GraphQLArgument)
                 else GraphQLArgument(cast(GraphQLInputType, value))
                 for name, value in args.items()

src/graphql/subscription/subscribe.py

@@ -75,7 +75,7 @@ def __init__(
             )
         else:
             args = {
-                name: cast(GraphQLArgument, value)
+                name: value
                 if isinstance(value, GraphQLArgument)
                 else GraphQLArgument(cast(GraphQLInputType, value))
                 for name, value in args.items()

src/graphql/type/definition.py

@@ -71,11 +71,9 @@ def ast_from_value(value: Any, type_: GraphQLInputType) -> Optional[ValueNode]:
         type_ = cast(GraphQLList, type_)
         item_type = type_.of_type
         if isinstance(value, Iterable) and not isinstance(value, str):
-            value_nodes = [
-                ast_from_value(item, item_type) for item in value  # type: ignore
-            ]
+            value_nodes = [ast_from_value(item, item_type) for item in value]
             return ListValueNode(values=value_nodes)
-        return ast_from_value(value, item_type)  # type: ignore
+        return ast_from_value(value, item_type)
 
     # Populate the fields of the input object by creating ASTs from each value in the
     # Python dict according to the fields in the input type.

src/graphql/type/directives.py

@@ -103,14 +103,13 @@ def build_ast_schema(
         if isinstance(def_, SchemaDefinitionNode):
             schema_def = def_
         elif isinstance(def_, TypeDefinitionNode):
-            def_ = cast(TypeDefinitionNode, def_)
             type_defs.append(def_)
         elif isinstance(def_, DirectiveDefinitionNode):
             append_directive_def(def_)
 
     def resolve_type(type_name: str) -> GraphQLNamedType:
         type_ = type_map.get(type_name)
-        if not type:
+        if not type_:
             raise TypeError(f"Type '{type_name}' not found in document.")
         return type_