Merge pull request #110 from femesq/patch-1

syrusakbary · web-flow · commit b10adc4ee3a2 · 2017-03-16T22:43:20.000-07:00
Accept custom middleware on experimental executor
diff --git a/graphql/execution/experimental/executor.py b/graphql/execution/experimental/executor.py
@@ -16,6 +16,8 @@ def execute(schema, document_ast, root_value=None, context_value=None,
         'not multiple versions of GraphQL installed in your node_modules directory.'
     )
     if middleware:
+        if not isinstance(middleware, MiddlewareManager):
+            middleware = MiddlewareManager(*middleware)
         assert isinstance(middleware, MiddlewareManager), (
             'middlewares have to be an instance'
             ' of MiddlewareManager. Received "{}".'.format(middleware)

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,8 @@ def execute(schema, document_ast, root_value=None, context_value=None,`
`16`	`16`	`'not multiple versions of GraphQL installed in your node_modules directory.'`
`17`	`17`	`)`
`18`	`18`	`if middleware:`
	`19`	`+ if not isinstance(middleware, MiddlewareManager):`
	`20`	`+ middleware = MiddlewareManager(*middleware)`
`19`	`21`	`assert isinstance(middleware, MiddlewareManager), (`
`20`	`22`	`'middlewares have to be an instance'`
`21`	`23`	`' of MiddlewareManager. Received "{}".'.format(middleware)`