GraphQLError: don't wrap single node in array

Replicates graphql/graphql-js@fbd9c47

Author: Cito

graphql/execution/values.py

@@ -58,7 +58,7 @@ def get_variable_values(
                     f"Variable '${var_name}' expected value of type"
                     f" '{print_ast(var_def_node.type)}'"
                     " which cannot be used as an input type.",
-                    [var_def_node.type],
+                    var_def_node.type,
                 )
             )
         else:
@@ -79,7 +79,7 @@ def get_variable_values(
                         if has_value
                         else f"Variable '${var_name}' of required type"
                         f" '{var_type}' was not provided.",
-                        [var_def_node],
+                        var_def_node,
                     )
                 )
             elif has_value:
@@ -146,21 +146,21 @@ def get_argument_values(
                 raise GraphQLError(
                     f"Argument '{name}' of non-null type"
                     f" '{arg_type}' must not be null.",
-                    [argument_node.value],
+                    argument_node.value,
                 )
             elif argument_node and isinstance(argument_node.value, VariableNode):
                 raise GraphQLError(
                     f"Argument '{name}' of required type"
                     f" '{arg_type}' was provided the variable"
                     f" '${variable_name}'"
                     " which was not provided a runtime value.",
-                    [argument_node.value],
+                    argument_node.value,
                 )
             else:
                 raise GraphQLError(
                     f"Argument '{name}' of required type '{arg_type}'"
                     " was not provided.",
-                    [node],
+                    node,
                 )
         elif has_value:
             if isinstance(argument_node.value, NullValueNode):
@@ -183,7 +183,7 @@ def get_argument_values(
                     raise GraphQLError(
                         f"Argument '{name}'"
                         f" has invalid value {print_ast(value_node)}.",
-                        [argument_node.value],
+                        argument_node.value,
                     )
                 coerced_values[name] = coerced_value
     return coerced_values

graphql/type/validate.py

@@ -93,9 +93,7 @@ def report_error(
         message: str,
         nodes: Union[Optional[Node], Sequence[Optional[Node]]] = None,
     ):
-        if isinstance(nodes, Node):
-            nodes = [nodes]
-        if nodes:
+        if nodes and not isinstance(nodes, Node):
             nodes = [node for node in nodes if node]
         nodes = cast(Optional[Sequence[Node]], nodes)
         self.add_error(GraphQLError(message, nodes))

graphql/utilities/find_deprecated_usages.py

@@ -42,7 +42,7 @@ def enter_field(self, node, *_args):
                     GraphQLError(
                         f"The field {parent_type.name}.{field_name}"
                         " is deprecated." + (f" {reason}" if reason else ""),
-                        [node],
+                        node,
                     )
                 )
 
@@ -57,6 +57,6 @@ def enter_enum_value(self, node, *_args):
                     GraphQLError(
                         f"The enum value {type_.name}.{enum_val_name}"
                         " is deprecated." + (f" {reason}" if reason else ""),
-                        [node],
+                        node,
                     )
                 )

graphql/utilities/get_operation_root_type.py

@@ -21,22 +21,20 @@ def get_operation_root_type(
         query_type = schema.query_type
         if not query_type:
             raise GraphQLError(
-                "Schema does not define the required query root type.", [operation]
+                "Schema does not define the required query root type.", operation
             )
         return query_type
     elif operation_type == OperationType.MUTATION:
         mutation_type = schema.mutation_type
         if not mutation_type:
-            raise GraphQLError("Schema is not configured for mutations.", [operation])
+            raise GraphQLError("Schema is not configured for mutations.", operation)
         return mutation_type
     elif operation_type == OperationType.SUBSCRIPTION:
         subscription_type = schema.subscription_type
         if not subscription_type:
-            raise GraphQLError(
-                "Schema is not configured for subscriptions.", [operation]
-            )
+            raise GraphQLError("Schema is not configured for subscriptions.", operation)
         return subscription_type
     else:
         raise GraphQLError(
-            "Can only have query, mutation and subscription operations.", [operation]
+            "Can only have query, mutation and subscription operations.", operation
         )

graphql/validation/rules/executable_definitions.py

@@ -40,7 +40,7 @@ def enter_document(self, node: DocumentNode, *_args):
                                 definition,
                             ).name.value
                         ),
-                        [definition],
+                        definition,
                     )
                 )
         return self.SKIP

graphql/validation/rules/fields_on_correct_type.py

@@ -63,7 +63,7 @@ def enter_field(self, node: FieldNode, *_args):
                 undefined_field_message(
                     field_name, type_.name, suggested_type_names, suggested_field_names
                 ),
-                [node],
+                node,
             )
         )
 

graphql/validation/rules/fragments_on_composite_types.py

@@ -37,7 +37,7 @@ def enter_inline_fragment(self, node: InlineFragmentNode, *_args):
                         inline_fragment_on_non_composite_error_message(
                             print_ast(type_condition)
                         ),
-                        [type_condition],
+                        type_condition,
                     )
                 )
 
@@ -50,6 +50,6 @@ def enter_fragment_definition(self, node: FragmentDefinitionNode, *_args):
                     fragment_on_non_composite_error_message(
                         node.name.value, print_ast(type_condition)
                     ),
-                    [type_condition],
+                    type_condition,
                 )
             )

graphql/validation/rules/known_directives.py

@@ -64,12 +64,12 @@ def enter_directive(self, node: DirectiveNode, _key, _parent, _path, ancestors):
                         misplaced_directive_message(
                             node.name.value, candidate_location.value
                         ),
-                        [node],
+                        node,
                     )
                 )
         else:
             self.report_error(
-                GraphQLError(unknown_directive_message(node.name.value), [node])
+                GraphQLError(unknown_directive_message(node.name.value), node)
             )
 
 

graphql/validation/rules/known_fragment_names.py

@@ -21,5 +21,5 @@ def enter_fragment_spread(self, node: FragmentSpreadNode, *_args):
         fragment = self.context.get_fragment(fragment_name)
         if not fragment:
             self.report_error(
-                GraphQLError(unknown_fragment_message(fragment_name), [node.name])
+                GraphQLError(unknown_fragment_message(fragment_name), node.name)
             )

graphql/validation/rules/lone_anonymous_operation.py

@@ -30,5 +30,5 @@ def enter_document(self, node: DocumentNode, *_args):
     def enter_operation_definition(self, node: OperationDefinitionNode, *_args):
         if not node.name and self.operation_count > 1:
             self.report_error(
-                GraphQLError(anonymous_operation_not_alone_message(), [node])
+                GraphQLError(anonymous_operation_not_alone_message(), node)
             )