Backout inappropriate reformatting

Author: alexchamberlain

src/graphql/validation/rules/lone_schema_definition.py

@@ -17,14 +17,23 @@ def __init__(self, context: SDLValidationContext):
         super().__init__(context)
         old_schema = context.schema
         self.already_defined = old_schema and (
-            old_schema.ast_node or old_schema.query_type or old_schema.mutation_type or old_schema.subscription_type
+            old_schema.ast_node
+            or old_schema.query_type
+            or old_schema.mutation_type
+            or old_schema.subscription_type
         )
         self.schema_definitions_count = 0
 
     def enter_schema_definition(self, node: SchemaDefinitionNode, *_args: Any) -> None:
         if self.already_defined:
-            self.report_error(GraphQLError("Cannot define a new schema within a schema extension.", node))
+            self.report_error(
+                GraphQLError(
+                    "Cannot define a new schema within a schema extension.", node
+                )
+            )
         else:
             if self.schema_definitions_count:
-                self.report_error(GraphQLError("Must provide only one schema definition.", node))
+                self.report_error(
+                    GraphQLError("Must provide only one schema definition.", node)
+                )
             self.schema_definitions_count += 1

src/graphql/validation/rules/no_undefined_variables.py

@@ -21,7 +21,9 @@ def __init__(self, context: ValidationContext):
     def enter_operation_definition(self, *_args: Any) -> None:
         self.defined_variable_names.clear()
 
-    def leave_operation_definition(self, operation: OperationDefinitionNode, *_args: Any) -> None:
+    def leave_operation_definition(
+        self, operation: OperationDefinitionNode, *_args: Any
+    ) -> None:
         usages = self.context.get_recursive_variable_usages(operation)
         defined_variables = self.defined_variable_names
         for usage in usages:
@@ -30,12 +32,15 @@ def leave_operation_definition(self, operation: OperationDefinitionNode, *_args:
             if var_name not in defined_variables:
                 self.report_error(
                     GraphQLError(
-                        f"Variable '${var_name}' is not defined" f" by operation '{operation.name.value}'."
+                        f"Variable '${var_name}' is not defined"
+                        f" by operation '{operation.name.value}'."
                         if operation.name
                         else f"Variable '${var_name}' is not defined.",
                         [node, operation],
                     )
                 )
 
-    def enter_variable_definition(self, node: VariableDefinitionNode, *_args: Any) -> None:
+    def enter_variable_definition(
+        self, node: VariableDefinitionNode, *_args: Any
+    ) -> None:
         self.defined_variable_names.add(node.variable.name.value)

src/graphql/validation/validation_context.py

@@ -56,7 +56,9 @@ def enter_variable_definition(self, *_args: Any) -> VisitorAction:
 
     def enter_variable(self, node: VariableNode, *_args: Any) -> VisitorAction:
         type_info = self._type_info
-        usage = VariableUsage(node, type_info.get_input_type(), type_info.get_default_value())
+        usage = VariableUsage(
+            node, type_info.get_input_type(), type_info.get_default_value()
+        )
         self._append_usage(usage)
         return None
 
@@ -73,9 +75,13 @@ class ASTValidationContext:
 
     _fragments: Optional[Dict[str, FragmentDefinitionNode]]
     _fragment_spreads: Dict[SelectionSetNode, List[FragmentSpreadNode]]
-    _recursively_referenced_fragments: Dict[OperationDefinitionNode, List[FragmentDefinitionNode]]
+    _recursively_referenced_fragments: Dict[
+        OperationDefinitionNode, List[FragmentDefinitionNode]
+    ]
 
-    def __init__(self, ast: DocumentNode, on_error: Callable[[GraphQLError], None]) -> None:
+    def __init__(
+        self, ast: DocumentNode, on_error: Callable[[GraphQLError], None]
+    ) -> None:
         self.document = ast
         self.on_error = on_error  # type: ignore
         self._fragments = None
@@ -112,13 +118,17 @@ def get_fragment_spreads(self, node: SelectionSetNode) -> List[FragmentSpreadNod
                     if isinstance(selection, FragmentSpreadNode):
                         append_spread(selection)
                     else:
-                        set_to_visit = cast(NodeWithSelectionSet, selection).selection_set
+                        set_to_visit = cast(
+                            NodeWithSelectionSet, selection
+                        ).selection_set
                         if set_to_visit:
                             append_set(set_to_visit)
             self._fragment_spreads[node] = spreads
         return spreads
 
-    def get_recursively_referenced_fragments(self, operation: OperationDefinitionNode) -> List[FragmentDefinitionNode]:
+    def get_recursively_referenced_fragments(
+        self, operation: OperationDefinitionNode
+    ) -> List[FragmentDefinitionNode]:
         fragments = self._recursively_referenced_fragments.get(operation)
         if fragments is None:
             fragments = []
@@ -200,7 +210,9 @@ def get_variable_usages(self, node: NodeWithSelectionSet) -> List[VariableUsage]
             self._variable_usages[node] = usages
         return usages
 
-    def get_recursive_variable_usages(self, operation: OperationDefinitionNode) -> List[VariableUsage]:
+    def get_recursive_variable_usages(
+        self, operation: OperationDefinitionNode
+    ) -> List[VariableUsage]:
         usages = self._recursive_variable_usages.get(operation)
         if usages is None:
             get_variable_usages = self.get_variable_usages