format_error should return locations as a list of {"line", "column"}

Author: ktosiek

src/graphql/error/format_error.py

@@ -17,7 +17,11 @@ def format_error(error: "GraphQLError") -> Dict[str, Any]:
         raise TypeError("Received no error object.")
     formatted: Dict[str, Any] = dict(  # noqa: E701 (pycqa/flake8#394)
         message=error.message or "An unknown error occurred.",
-        locations=error.locations,
+        locations=(
+            [{"line": l.line, "column": l.column} for l in error.locations]
+            if error.locations is not None
+            else None
+        ),
         path=error.path,
     )
     if error.extensions:

tests/error/test_format_error.py

@@ -30,9 +30,10 @@ def format_graphql_error():
             ValueError("original"),
             extensions=extensions,
         )
-        assert error == {
+        formatted = format_error(error)
+        assert formatted == {
             "message": "test message",
-            "locations": [(2, 14), (3, 20)],
+            "locations": [{"line": 2, "column": 14}, {"line": 3, "column": 20}],
             "path": path,
             "extensions": extensions,
         }

tests/execution/test_abstract.py

@@ -254,7 +254,7 @@ def resolve_type_on_interface_yields_useful_error():
         assert format_error(result.errors[0]) == {
             "message": "Runtime Object type 'Human'"
             " is not a possible type for 'Pet'.",
-            "locations": [(3, 15)],
+            "locations": [{"line": 3, "column": 15}],
             "path": ["pets", 2],
         }
 
@@ -330,7 +330,7 @@ def resolve_type_on_union_yields_useful_error():
         assert format_error(result.errors[0]) == {
             "message": "Runtime Object type 'Human'"
             " is not a possible type for 'Pet'.",
-            "locations": [(3, 15)],
+            "locations": [{"line": 3, "column": 15}],
             "path": ["pets", 2],
         }
 

tests/execution/test_abstract_async.py

@@ -182,12 +182,12 @@ async def is_type_of_with_async_error():
         assert list(map(format_error, result.errors)) == [  # type: ignore
             {
                 "message": "We are testing this error",
-                "locations": [(3, 15)],
+                "locations": [{"line": 3, "column": 15}],
                 "path": ["pets", 0],
             },
             {
                 "message": "We are testing this error",
-                "locations": [(3, 15)],
+                "locations": [{"line": 3, "column": 15}],
                 "path": ["pets", 1],
             },
         ]
@@ -334,7 +334,7 @@ async def resolve_type_on_interface_yields_useful_error():
         assert format_error(result.errors[0]) == {
             "message": "Runtime Object type 'Human'"
             " is not a possible type for 'Pet'.",
-            "locations": [(3, 15)],
+            "locations": [{"line": 3, "column": 15}],
             "path": ["pets", 2],
         }
 
@@ -411,7 +411,7 @@ async def resolve_type_on_union_yields_useful_error():
         assert format_error(result.errors[0]) == {
             "message": "Runtime Object type 'Human'"
             " is not a possible type for 'Pet'.",
-            "locations": [(3, 15)],
+            "locations": [{"line": 3, "column": 15}],
             "path": ["pets", 2],
         }