Add flake8 pre-commit hook and manually edit files to pass flake8 validation (#746)

Add flake8 pre-commit hook and manually edit files to pass flake8 validation

Author: dan98765

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
 -   repo: git://github.com/pre-commit/pre-commit-hooks
-    rev: v1.2.3
+    rev: v1.3.0
     hooks:
     -   id: autopep8-wrapper
         args:
@@ -16,6 +16,7 @@ repos:
     -   id: pretty-format-json
         args:
         - --autofix
+    -   id: flake8
 -   repo: https://github.com/asottile/seed-isort-config
     rev: v1.0.0
     hooks:

graphene/relay/tests/test_connection_query.py

@@ -56,8 +56,7 @@ def resolve_connection_letters(self, info, **args):
 
 letters = OrderedDict()
 for i, letter in enumerate(letter_chars):
-    l = Letter(id=i, letter=letter)
-    letters[letter] = l
+    letters[letter] = Letter(id=i, letter=letter)
 
 
 def edges(selected_letters):
@@ -74,8 +73,8 @@ def edges(selected_letters):
 
 
 def cursor_for(ltr):
-    l = letters[ltr]
-    return base64('arrayconnection:%s' % l.id)
+    letter = letters[ltr]
+    return base64('arrayconnection:%s' % letter.id)
 
 
 def execute(args=''):

graphene/tests/issues/test_313.py

@@ -1,7 +1,6 @@
 # https://github.com/graphql-python/graphene/issues/313
 
 import graphene
-from graphene import resolve_only_args
 
 
 class Query(graphene.ObjectType):

graphene/tests/issues/test_356.py

@@ -21,10 +21,13 @@ class Meta:
 
 
 def test_issue():
-    with pytest.raises(Exception) as exc_info:
-        class Query(graphene.ObjectType):
-            things = relay.ConnectionField(MyUnion)
+    class Query(graphene.ObjectType):
+        things = relay.ConnectionField(MyUnion)
 
-        schema = graphene.Schema(query=Query)
+    with pytest.raises(Exception) as exc_info:
+        graphene.Schema(query=Query)
 
-    assert str(exc_info.value) == 'IterableConnectionField type have to be a subclass of Connection. Received "MyUnion".'
+    assert str(exc_info.value) == (
+        'IterableConnectionField type have to be a subclass of Connection. '
+        'Received "MyUnion".'
+    )

graphene/tests/issues/test_425.py

@@ -2,8 +2,7 @@
 # Adapted for Graphene 2.0
 
 from graphene.types.enum import Enum, EnumOptions
-from graphene.types.inputobjecttype import (InputObjectType,
-                                            InputObjectTypeOptions)
+from graphene.types.inputobjecttype import InputObjectType
 from graphene.types.objecttype import ObjectType, ObjectTypeOptions
 
 

graphene/types/tests/test_abstracttype.py

@@ -1,5 +1,3 @@
-import pytest
-
 from .. import abstracttype
 from ..abstracttype import AbstractType
 from ..field import Field

graphene/types/tests/test_argument.py

@@ -51,7 +51,10 @@ def test_to_arguments_raises_if_field():
     with pytest.raises(ValueError) as exc_info:
         to_arguments(args)
 
-    assert str(exc_info.value) == 'Expected arg_string to be Argument, but received Field. Try using Argument(String).'
+    assert str(exc_info.value) == (
+        'Expected arg_string to be Argument, but received Field. Try using '
+        'Argument(String).'
+    )
 
 
 def test_to_arguments_raises_if_inputfield():
@@ -62,7 +65,10 @@ def test_to_arguments_raises_if_inputfield():
     with pytest.raises(ValueError) as exc_info:
         to_arguments(args)
 
-    assert str(exc_info.value) == 'Expected arg_string to be Argument, but received InputField. Try using Argument(String).'
+    assert str(exc_info.value) == (
+        'Expected arg_string to be Argument, but received InputField. Try '
+        'using Argument(String).'
+    )
 
 
 def test_argument_with_lazy_type():

graphene/types/tests/test_base.py

@@ -1,5 +1,3 @@
-import pytest
-
 from ..base import BaseOptions, BaseType
 
 

graphene/types/tests/test_datetime.py

@@ -62,7 +62,7 @@ def test_bad_datetime_query():
 
     assert len(result.errors) == 1
     assert isinstance(result.errors[0], GraphQLError)
-    assert result.data == None
+    assert result.data is None
 
 
 def test_bad_date_query():
@@ -72,7 +72,7 @@ def test_bad_date_query():
 
     assert len(result.errors) == 1
     assert isinstance(result.errors[0], GraphQLError)
-    assert result.data == None
+    assert result.data is None
 
 
 def test_bad_time_query():
@@ -82,7 +82,7 @@ def test_bad_time_query():
 
     assert len(result.errors) == 1
     assert isinstance(result.errors[0], GraphQLError)
-    assert result.data == None
+    assert result.data is None
 
 
 def test_datetime_query_variable():

graphene/types/tests/test_definition.py

@@ -288,7 +288,14 @@ def test_stringifies_simple_types():
 #     ]
 #     for x in bad_union_types:
 #         with raises(Exception) as excinfo:
-#             GraphQLSchema(GraphQLObjectType('Root', fields={'union': GraphQLField(GraphQLUnionType('BadUnion', [x]))}))
+#             GraphQLSchema(
+#                 GraphQLObjectType(
+#                     'Root',
+#                     fields={
+#                         'union': GraphQLField(GraphQLUnionType('BadUnion', [x]))
+#                     }
+#                 )
+#             )
 
 #         assert 'BadUnion may only contain Object types, it cannot contain: ' + str(x) + '.' \
 #                == str(excinfo.value)