Run pre-commit on all files

Author: Daniel Gallagher

graphql/backend/base.py

@@ -1,9 +1,10 @@
-from ..pyutils.cached_property import cached_property
-from ..language import ast
-
 from abc import ABCMeta, abstractmethod
+
 import six
 
+from ..language import ast
+from ..pyutils.cached_property import cached_property
+
 
 class GraphQLBackend(six.with_metaclass(ABCMeta)):
     @abstractmethod

graphql/backend/cache.py

@@ -1,7 +1,8 @@
 from hashlib import sha1
+
 from six import string_types
-from ..type import GraphQLSchema
 
+from ..type import GraphQLSchema
 from .base import GraphQLBackend
 
 _cached_schemas = {}

graphql/backend/compiled.py

@@ -1,4 +1,5 @@
 from six import string_types
+
 from .base import GraphQLDocument
 
 

graphql/backend/core.py

@@ -1,11 +1,11 @@
 from functools import partial
+
 from six import string_types
 
-from ..execution import execute, ExecutionResult
-from ..language.base import parse, print_ast
+from ..execution import ExecutionResult, execute
 from ..language import ast
+from ..language.base import parse, print_ast
 from ..validation import validate
-
 from .base import GraphQLBackend, GraphQLDocument
 
 

graphql/backend/quiver_cloud.py

@@ -6,12 +6,12 @@
         "You can install it using: pip install requests"
     )
 
-from ..utils.schema_printer import print_schema
+from six import urlparse
 
+from ..utils.schema_printer import print_schema
 from .base import GraphQLBackend
 from .compiled import GraphQLCompiledDocument
 
-from six import urlparse
 
 GRAPHQL_QUERY = """
 mutation($schemaDsl: String!, $query: String!) {

graphql/backend/tests/schema.py

@@ -1,6 +1,5 @@
-from graphql.type import (GraphQLField, GraphQLObjectType,
-                          GraphQLSchema, GraphQLString)
-
+from graphql.type import (GraphQLField, GraphQLObjectType, GraphQLSchema,
+                          GraphQLString)
 
 Query = GraphQLObjectType('Query', lambda: {
     'hello': GraphQLField(GraphQLString, resolver=lambda *_: "World"),

graphql/backend/tests/test_base.py

@@ -1,5 +1,6 @@
 import pytest
-from .. import get_default_backend, set_default_backend, GraphQLCoreBackend
+
+from .. import GraphQLCoreBackend, get_default_backend, set_default_backend
 
 
 def test_get_default_backend_returns_core_by_default():

graphql/backend/tests/test_cache.py

@@ -2,11 +2,12 @@
 # -*- coding: utf-8 -*-
 """Tests for `graphql.backend.cache` module."""
 
+from graphql.execution.executors.sync import SyncExecutor
+
 import pytest
 
-from ..core import GraphQLCoreBackend
 from ..cache import GraphQLCachedBackend
-from graphql.execution.executors.sync import SyncExecutor
+from ..core import GraphQLCoreBackend
 from .schema import schema
 
 

graphql/backend/tests/test_core.py

@@ -2,9 +2,10 @@
 # -*- coding: utf-8 -*-
 """Tests for `graphql.backend.core` module."""
 
-import pytest
 from graphql.execution.executors.sync import SyncExecutor
 
+import pytest
+
 from ..base import GraphQLBackend, GraphQLDocument
 from ..core import GraphQLCoreBackend
 from .schema import schema

graphql/backend/tests/test_decider.py

@@ -5,10 +5,9 @@
 import pytest
 
 from ..base import GraphQLBackend, GraphQLDocument
-from ..core import GraphQLCoreBackend
 from ..cache import GraphQLCachedBackend
+from ..core import GraphQLCoreBackend
 from ..decider import GraphQLDeciderBackend
-
 from .schema import schema
 
 

graphql/error/base.py

@@ -1,4 +1,5 @@
 import six
+
 from ..language.location import get_location
 
 

graphql/error/tests/test_base.py

@@ -1,11 +1,12 @@
-import pytest
 import traceback
 
 from graphql.execution import execute
 from graphql.language.parser import parse
 from graphql.type import (GraphQLField, GraphQLObjectType, GraphQLSchema,
                           GraphQLString)
 
+import pytest
+
 
 def test_raise():
     ast = parse('query Example { a }')

graphql/execution/base.py

@@ -1,15 +1,9 @@
 # We keep the following imports to preserve compatibility
-from .utils import (
-    ExecutionContext,
-    SubscriberExecutionContext,
-    get_operation_root_type,
-    collect_fields,
-    should_include_node,
-    does_fragment_condition_match,
-    get_field_entry_key,
-    default_resolve_fn,
-    get_field_def
-)
+from .utils import (ExecutionContext, SubscriberExecutionContext,
+                    collect_fields, default_resolve_fn,
+                    does_fragment_condition_match, get_field_def,
+                    get_field_entry_key, get_operation_root_type,
+                    should_include_node)
 
 
 class ExecutionResult(object):

graphql/execution/executor.py

@@ -3,21 +3,21 @@
 import logging
 import sys
 import warnings
-from rx import Observable
 
+from promise import Promise, is_thenable, promise_for_dict
+from rx import Observable
 from six import string_types
-from promise import Promise, promise_for_dict, is_thenable
 
 from ..error import GraphQLError, GraphQLLocatedError
 from ..pyutils.default_ordered_dict import DefaultOrderedDict
 from ..pyutils.ordereddict import OrderedDict
-from ..utils.undefined import Undefined
 from ..type import (GraphQLEnumType, GraphQLInterfaceType, GraphQLList,
                     GraphQLNonNull, GraphQLObjectType, GraphQLScalarType,
                     GraphQLSchema, GraphQLUnionType)
+from ..utils.undefined import Undefined
 from .base import (ExecutionContext, ExecutionResult, ResolveInfo,
-                   collect_fields, default_resolve_fn, get_field_def,
-                   get_operation_root_type, SubscriberExecutionContext)
+                   SubscriberExecutionContext, collect_fields,
+                   default_resolve_fn, get_field_def, get_operation_root_type)
 from .executors.sync import SyncExecutor
 from .middleware import MiddlewareManager
 
@@ -147,7 +147,7 @@ def execute_field_callback(results, response_name):
             source_value,
             field_asts,
             None,
-            path+[response_name]
+            path + [response_name]
         )
         if result is Undefined:
             return results

graphql/execution/executors/asyncio_utils.py

@@ -1,11 +1,10 @@
-from inspect import isasyncgen
 from asyncio import ensure_future
-from rx import Observable, AnonymousObserver
-from rx.core import ObservableBase, Disposable, ObserverBase
+from inspect import isasyncgen
 
+from rx import AnonymousObserver, Observable
 from rx.concurrency import current_thread_scheduler
-
-from rx.core import Observer, Observable, Disposable
+from rx.core import (Disposable, Observable, ObservableBase, Observer,
+                     ObserverBase)
 from rx.core.anonymousobserver import AnonymousObserver
 from rx.core.autodetachobserver import AutoDetachObserver
 

graphql/execution/tests/test_benchmark.py

@@ -3,6 +3,7 @@
 
 from graphql import (GraphQLField, GraphQLInt, GraphQLList, GraphQLObjectType,
                      GraphQLSchema, Source, execute, parse)
+
 # from graphql.execution import executor
 
 # executor.use_experimental_executor = True

graphql/execution/tests/test_dataloader.py

@@ -1,10 +1,11 @@
-import pytest
+from graphql import (GraphQLArgument, GraphQLField, GraphQLID, GraphQLNonNull,
+                     GraphQLObjectType, GraphQLSchema, execute, parse)
+from graphql.execution.executors.sync import SyncExecutor
+from graphql.execution.executors.thread import ThreadExecutor
 from promise import Promise
 from promise.dataloader import DataLoader
 
-from graphql import GraphQLObjectType, GraphQLField, GraphQLID, GraphQLArgument, GraphQLNonNull, GraphQLSchema, parse, execute
-from graphql.execution.executors.sync import SyncExecutor
-from graphql.execution.executors.thread import ThreadExecutor
+import pytest
 
 
 @pytest.mark.parametrize("executor", [

graphql/execution/tests/test_executor.py

@@ -1,15 +1,15 @@
 import json
 
-from pytest import raises
-
 from graphql.error import GraphQLError
 from graphql.execution import MiddlewareManager, execute
 from graphql.language.parser import parse
 from graphql.type import (GraphQLArgument, GraphQLBoolean, GraphQLField,
-                          GraphQLInt, GraphQLList, GraphQLObjectType,
-                          GraphQLSchema, GraphQLString, GraphQLNonNull, GraphQLID)
+                          GraphQLID, GraphQLInt, GraphQLList, GraphQLNonNull,
+                          GraphQLObjectType, GraphQLSchema, GraphQLString)
 from promise import Promise
 
+from pytest import raises
+
 
 def test_executes_arbitary_code():
     class Data(object):

graphql/execution/tests/test_executor_thread.py

@@ -206,13 +206,13 @@ def handle_results(result):
             'syncReturnErrorList': ['sync0', None, 'sync2', None]
         }
         assert sorted(list(map(format_error, result.errors)), key=sort_key) == sorted([
-            {'locations': [{'line': 4, 'column': 9}], 'path':['syncError'], 'message': 'Error getting syncError'},
-            {'locations': [{'line': 5, 'column': 9}], 'path':['syncReturnError'], 'message': 'Error getting syncReturnError'},
-            {'locations': [{'line': 6, 'column': 9}], 'path':['syncReturnErrorList', 1], 'message': 'Error getting syncReturnErrorList1'},
-            {'locations': [{'line': 6, 'column': 9}], 'path':['syncReturnErrorList', 3], 'message': 'Error getting syncReturnErrorList3'},
-            {'locations': [{'line': 8, 'column': 9}], 'path':['asyncReject'], 'message': 'Error getting asyncReject'},
-            {'locations': [{'line': 9, 'column': 9}], 'path':['asyncEmptyReject'], 'message': 'An unknown error occurred.'},
-            {'locations': [{'line': 10, 'column': 9}], 'path':['asyncReturnError'], 'message': 'Error getting asyncReturnError'}
+            {'locations': [{'line': 4, 'column': 9}], 'path': ['syncError'], 'message': 'Error getting syncError'},
+            {'locations': [{'line': 5, 'column': 9}], 'path': ['syncReturnError'], 'message': 'Error getting syncReturnError'},
+            {'locations': [{'line': 6, 'column': 9}], 'path': ['syncReturnErrorList', 1], 'message': 'Error getting syncReturnErrorList1'},
+            {'locations': [{'line': 6, 'column': 9}], 'path': ['syncReturnErrorList', 3], 'message': 'Error getting syncReturnErrorList3'},
+            {'locations': [{'line': 8, 'column': 9}], 'path': ['asyncReject'], 'message': 'Error getting asyncReject'},
+            {'locations': [{'line': 9, 'column': 9}], 'path': ['asyncEmptyReject'], 'message': 'An unknown error occurred.'},
+            {'locations': [{'line': 10, 'column': 9}], 'path': ['asyncReturnError'], 'message': 'Error getting asyncReturnError'}
         ], key=sort_key)
 
     handle_results(execute(schema, ast, Data(), executor=ThreadExecutor()))

graphql/execution/tests/test_located_error.py

@@ -1,11 +1,7 @@
 # coding: utf-8
 
-from graphql import GraphQLField
-from graphql import GraphQLObjectType
-from graphql import GraphQLSchema
-from graphql import GraphQLString
-from graphql import execute
-from graphql import parse
+from graphql import (GraphQLField, GraphQLObjectType, GraphQLSchema,
+                     GraphQLString, execute, parse)
 from graphql.error import GraphQLLocatedError
 
 

graphql/execution/tests/test_nonnull.py

@@ -320,27 +320,27 @@ def test_nulls_the_first_nullable_object_after_a_field_throws_in_a_long_chain_of
         'errors': [{'locations': [{'column': 19, 'line': 8}],
                     'path': [
                         'nest', 'nonNullNest', 'nonNullPromiseNest',
-                        'nonNullNest',  'nonNullPromiseNest', 'nonNullSync'
-                    ],
-                    'message': str(non_null_sync_error)},
-                   {'locations': [{'column': 19, 'line': 19}],
-                   'path': [
-                        'promiseNest', 'nonNullNest', 'nonNullPromiseNest',
-                        'nonNullNest',  'nonNullPromiseNest', 'nonNullSync'
-                    ],
-                    'message': str(non_null_sync_error)},
-                   {'locations': [{'column': 19, 'line': 30}],
-                   'path': [
-                        'anotherNest', 'nonNullNest', 'nonNullPromiseNest',
-                        'nonNullNest',  'nonNullPromiseNest', 'nonNullPromise'
-                    ],
-                    'message': str(non_null_promise_error)},
-                   {'locations': [{'column': 19, 'line': 41}],
-                   'path': [
-                        'anotherPromiseNest', 'nonNullNest', 'nonNullPromiseNest',
-                        'nonNullNest',  'nonNullPromiseNest', 'nonNullPromise'
-                    ],
-                    'message': str(non_null_promise_error)}]
+                        'nonNullNest', 'nonNullPromiseNest', 'nonNullSync'
+        ],
+            'message': str(non_null_sync_error)},
+            {'locations': [{'column': 19, 'line': 19}],
+             'path': [
+                'promiseNest', 'nonNullNest', 'nonNullPromiseNest',
+                'nonNullNest', 'nonNullPromiseNest', 'nonNullSync'
+            ],
+            'message': str(non_null_sync_error)},
+            {'locations': [{'column': 19, 'line': 30}],
+             'path': [
+                'anotherNest', 'nonNullNest', 'nonNullPromiseNest',
+                'nonNullNest', 'nonNullPromiseNest', 'nonNullPromise'
+            ],
+            'message': str(non_null_promise_error)},
+            {'locations': [{'column': 19, 'line': 41}],
+             'path': [
+                'anotherPromiseNest', 'nonNullNest', 'nonNullPromiseNest',
+                'nonNullNest', 'nonNullPromiseNest', 'nonNullPromise'
+            ],
+            'message': str(non_null_promise_error)}]
     })
 
 

graphql/execution/tests/test_subscribe.py

@@ -1,7 +1,10 @@
 from collections import OrderedDict, namedtuple
+
+from graphql import (GraphQLBoolean, GraphQLField, GraphQLInt, GraphQLList,
+                     GraphQLObjectType, GraphQLSchema, GraphQLString, graphql,
+                     parse, subscribe)
 from rx import Observable, Observer
 from rx.subjects import Subject
-from graphql import parse, GraphQLObjectType, GraphQLString, GraphQLBoolean, GraphQLInt, GraphQLField, GraphQLList, GraphQLSchema, graphql, subscribe
 
 Email = namedtuple('Email', 'from_,subject,message,unread')
 

graphql/execution/tests/test_variables.py

@@ -1,16 +1,16 @@
 import json
 from collections import OrderedDict
 
-from pytest import raises
-
 from graphql.error import GraphQLError, format_error
 from graphql.execution import execute
 from graphql.language.parser import parse
-from graphql.type import (GraphQLArgument, GraphQLField, GraphQLBoolean,
+from graphql.type import (GraphQLArgument, GraphQLBoolean, GraphQLField,
                           GraphQLInputObjectField, GraphQLInputObjectType,
                           GraphQLList, GraphQLNonNull, GraphQLObjectType,
                           GraphQLScalarType, GraphQLSchema, GraphQLString)
 
+from pytest import raises
+
 TestComplexScalar = GraphQLScalarType(
     name='ComplexScalar',
     serialize=lambda v: 'SerializedValue' if v == 'DeserializedValue' else None,

graphql/graphql.py

@@ -1,7 +1,8 @@
-from .execution import ExecutionResult
+from promise import promisify
+
 from .backend import get_default_backend
+from .execution import ExecutionResult
 
-from promise import promisify
 
 # This is the primary entry point function for fulfilling GraphQL operations
 # by parsing, validating, and executing a GraphQL document along side a

graphql/language/tests/test_lexer.py

@@ -1,9 +1,9 @@
-from pytest import raises
-
 from graphql.error import GraphQLSyntaxError
 from graphql.language.lexer import Lexer, Token, TokenKind
 from graphql.language.source import Source
 
+from pytest import raises
+
 
 def lex_one(s):
     return Lexer(Source(s)).next_token()