Updated resolver to use (root, info, **args)

Author: syrusakbary

graphql/error/tests/test_base.py

@@ -44,15 +44,15 @@ def resolver(context, *_):
         ('test_reraise', 'result.errors[0].reraise()'),
         ('reraise', 'six.reraise(type(self), self, self.stack)'),
         # ('reraise', 'raise value.with_traceback(tb)'),
-        ('resolve_or_error', 'return executor.execute(resolve_fn, source, args, context, info)'),
+        ('resolve_or_error', 'return executor.execute(resolve_fn, source, info, **args)'),
         ('execute', 'return fn(*args, **kwargs)'), ('resolver', "raise Exception('Failed')")
     ]
     # assert formatted_tb == [
     #     ('test_reraise', 'result.errors[0].reraise()'),
     #     ('reraise', 'six.reraise(type(self), self, self.stack)'),
     #     ('on_complete_resolver', 'result = __resolver(*args, **kwargs)'),
     #     # ('reraise', 'raise value.with_traceback(tb)'),
-    #     # ('resolve_or_error', 'return executor.execute(resolve_fn, source, args, context, info)'),
+    #     # ('resolve_or_error', 'return executor.execute(resolve_fn, source, info, **args)'),
     #     # ('execute', 'return fn(*args, **kwargs)'),
     #     ('resolver', "raise Exception('Failed')")
     # ]

graphql/execution/base.py

@@ -288,7 +288,7 @@ def __init__(self, field_name, field_asts, return_type, parent_type,
         self.context = context
 
 
-def default_resolve_fn(source, args, context, info):
+def default_resolve_fn(source, info, **args):
     """If a resolve function is not given, then a default resolve behavior is used which takes the property of the source object
     of the same name as the field and returns it as the result, or if it's a function, returns the result of calling that function."""
     name = info.field_name

graphql/execution/executor.py

@@ -17,25 +17,14 @@
                    collect_fields, default_resolve_fn, get_field_def,
                    get_operation_root_type)
 from .executors.sync import SyncExecutor
-from .experimental.executor import execute as experimental_execute
 from .middleware import MiddlewareManager
 
 logger = logging.getLogger(__name__)
 
 
-use_experimental_executor = False
-
-
 def execute(schema, document_ast, root_value=None, context_value=None,
             variable_values=None, operation_name=None, executor=None,
             return_promise=False, middleware=None):
-    if use_experimental_executor:
-        return experimental_execute(
-            schema, document_ast, root_value, context_value,
-            variable_values, operation_name, executor,
-            return_promise, middleware
-        )
-
     assert schema, 'Must provide schema'
     assert isinstance(schema, GraphQLSchema), (
         'Schema must be an instance of GraphQLSchema. Also ensure that there are ' +
@@ -186,7 +175,7 @@ def resolve_field(exe_context, parent_type, source, field_asts):
     )
 
     executor = exe_context.executor
-    result = resolve_or_error(resolve_fn_middleware, source, args, context, info, executor)
+    result = resolve_or_error(resolve_fn_middleware, source, info, args, executor)
 
     return complete_value_catching_error(
         exe_context,
@@ -197,9 +186,9 @@ def resolve_field(exe_context, parent_type, source, field_asts):
     )
 
 
-def resolve_or_error(resolve_fn, source, args, context, info, executor):
+def resolve_or_error(resolve_fn, source, info, args, executor):
     try:
-        return executor.execute(resolve_fn, source, args, context, info)
+        return executor.execute(resolve_fn, source, info, **args)
     except Exception as e:
         logger.exception("An error occurred while resolving field {}.{}".format(
             info.parent_type.name, info.field_name