GraphQLError: fix empty locations if error got nodes without locations

Replicates graphql/graphql-js@8ba5c56

Author: Cito

src/graphql/error/graphql_error.py

@@ -2,7 +2,7 @@
 from typing import Any, Collection, Dict, List, Optional, Union, TYPE_CHECKING
 
 if TYPE_CHECKING:
-    from ..language.ast import Node  # noqa: F401
+    from ..language.ast import Node, Location  # noqa: F401
     from ..language.location import SourceLocation  # noqa: F401
     from ..language.source import Source  # noqa: F401
 
@@ -95,27 +95,24 @@ def __init__(
         if nodes and not isinstance(nodes, list):
             nodes = [nodes]  # type: ignore
         self.nodes = nodes or None  # type: ignore
+        node_locations = (
+            [node.loc for node in nodes if node.loc] if nodes else []  # type: ignore
+        )
         self.source = source
-        if not source and nodes:
-            node = nodes[0]  # type: ignore
-            if node and node.loc and node.loc.source:
-                self.source = node.loc.source
-        if not positions and nodes:
-            positions = [node.loc.start for node in nodes if node.loc]  # type: ignore
+        if not source and node_locations:
+            loc = node_locations[0]
+            if loc.source:  # pragma: no cover else
+                self.source = loc.source
+        if not positions and node_locations:
+            positions = [loc.start for loc in node_locations]
         self.positions = positions or None
         if positions and source:
             locations: Optional[List["SourceLocation"]] = [
                 source.get_location(pos) for pos in positions
             ]
-        elif nodes:
-            locations = [
-                node.loc.source.get_location(node.loc.start)
-                for node in nodes  # type: ignore
-                if node.loc
-            ]
         else:
-            locations = None
-        self.locations = locations
+            locations = [loc.source.get_location(loc.start) for loc in node_locations]
+        self.locations = locations or None
         if path and not isinstance(path, list):
             path = list(path)
         self.path = path or None  # type: ignore

tests/error/test_graphql_error.py

@@ -110,6 +110,15 @@ def converts_node_with_loc_start_zero_to_positions_and_locations():
         assert e.positions == [0]
         assert e.locations == [(1, 1)]
 
+    def converts_node_without_location_to_source_positions_and_locations_as_none():
+        document_node = parse("{ foo }", no_location=True)
+
+        e = GraphQLError("msg", document_node)
+        assert e.nodes == [document_node]
+        assert e.source is None
+        assert e.positions is None
+        assert e.locations is None
+
     def converts_source_and_positions_to_locations():
         e = GraphQLError("msg", None, source, [6])
         assert e.nodes is None

tests/validation/test_validation.py

@@ -155,7 +155,6 @@ def _validate_document(max_errors=None):
     def _invalid_field_error(field_name: str):
         return {
             "message": f"Cannot query field '{field_name}' on type 'QueryRoot'.",
-            "locations": [],
         }
 
     def when_max_errors_is_equal_to_number_of_errors():