Don't add errors to ExecutionResult if are empty. Fixed #111

Author: syrusakbary

graphql/execution/executor.py

@@ -75,6 +75,8 @@ def on_rejected(error):
         return None
 
     def on_resolve(data):
+        if not context.errors:
+            return ExecutionResult(data=data)
         return ExecutionResult(data=data, errors=context.errors)
 
     promise = Promise(executor).catch(on_rejected).then(on_resolve)

graphql/execution/experimental/executor.py

@@ -45,6 +45,8 @@ def on_rejected(error):
         return None
 
     def on_resolve(data):
+        if not context.errors:
+            return ExecutionResult(data=data)
         return ExecutionResult(data=data, errors=context.errors)
 
     promise = Promise(executor).catch(on_rejected).then(on_resolve)