Improve return data and errors

Author: no1seman

graphql/execute.lua

@@ -313,7 +313,6 @@ end
 
 evaluateSelections = function(objectType, object, selections, context)
   local result = {}
-  local errors
   local err
   local fields = collectFields(objectType, selections, {}, {}, context)
   for _, field in ipairs(fields) do
@@ -322,14 +321,14 @@ evaluateSelections = function(objectType, object, selections, context)
     result[field.name], err = getFieldEntry(objectType, object, {field.selection},
                                        context)
     if err ~= nil then
-        errors = errors or {}
-        table.insert(errors, err)
+        context.errors = context.errors or {}
+        table.insert(context.errors, err)
     end
     if result[field.name] == nil then
         result[field.name] = box.NULL
     end
   end
-  return result, errors
+  return result, context.errors
 end
 
 local function execute(schema, tree, rootValue, variables, operationName)

test/integration/graphql_test.lua

@@ -1158,4 +1158,43 @@ function g.test_both_data_and_error_result()
         {message = 'Simple error A'},
         {message = 'Simple error B'},
     })
+
+    query = [[{
+        prefix {
+            test_A: test(arg: "A")
+            test_B: test(arg: "B")
+        }
+    }]]
+
+    query_schema = {
+        ['prefix'] = {
+            kind = {
+                __type = 'Object',
+                name = 'prefix',
+                fields = {
+                    ['test'] = {
+                        kind = types.string.nonNull,
+                        arguments = {
+                            arg = types.string.nonNull,
+                            arg2 = types.string,
+                            arg3 = types.int,
+                            arg4 = types.long,
+                        },
+                        resolve = callback,
+                    }
+                },
+            },
+            arguments = {},
+            resolve = function()
+                return {}
+            end,
+        }
+    }
+
+    data, errors = check_request(query, query_schema)
+    t.assert_equals(data, {prefix = {test_A = "A", test_B = "B"}})
+    t.assert_equals(errors,  {
+        {message = 'Simple error A'},
+        {message = 'Simple error B'},
+    })
 end