Remove oneof validation from values of correct type

Replicates graphql/graphql-js@7134cab

Author: Cito

src/graphql/validation/rules/values_of_correct_type.py

@@ -1,20 +1,18 @@
-from typing import cast, Any, Dict, Mapping
+from typing import cast, Any, Mapping
 
 from ...error import GraphQLError
 from ...language import (
     BooleanValueNode,
     EnumValueNode,
     FloatValueNode,
     IntValueNode,
-    NonNullTypeNode,
     NullValueNode,
     ListValueNode,
     ObjectFieldNode,
     ObjectValueNode,
     StringValueNode,
     ValueNode,
     VariableDefinitionNode,
-    VariableNode,
     VisitorAction,
     SKIP,
     print_ast,
@@ -86,9 +84,7 @@ def enter_object_value(self, node: ObjectValueNode, *_args: Any) -> VisitorActio
                     )
                 )
         if type_.is_one_of:
-            validate_one_of_input_object(
-                self.context, node, type_, field_node_map, self.variable_definitions
-            )
+            validate_one_of_input_object(self.context, node, type_, field_node_map)
         return None
 
     def enter_object_field(self, node: ObjectFieldNode, *_args: Any) -> None:
@@ -187,7 +183,6 @@ def validate_one_of_input_object(
     node: ObjectValueNode,
     type_: GraphQLInputObjectType,
     field_node_map: Mapping[str, ObjectFieldNode],
-    variable_definitions: Dict[str, VariableDefinitionNode],
 ) -> None:
     keys = list(field_node_map)
     is_not_exactly_one_filed = len(keys) != 1
@@ -212,19 +207,3 @@ def validate_one_of_input_object(
                 node,
             )
         )
-        return
-
-    is_variable = value and isinstance(value, VariableNode)
-    if is_variable:
-        variable_name = cast(VariableNode, value).name.value
-        definition = variable_definitions[variable_name]
-        is_nullable_variable = not isinstance(definition.type, NonNullTypeNode)
-
-        if is_nullable_variable:
-            context.report_error(
-                GraphQLError(
-                    f"Variable '{variable_name}' must be non-nullable"
-                    f" to be used for OneOf Input Object '{type_.name}'.",
-                    node,
-                )
-            )

tests/validation/test_values_of_correct_type.py

@@ -1085,24 +1085,6 @@ def exactly_one_null_field():
                 ],
             )
 
-        def exactly_one_nullable_variable():
-            assert_errors(
-                """
-                query ($string: String) {
-                complicatedArgs {
-                    oneOfArgField(oneOfArg: { stringField: $string })
-                }
-                }
-                """,
-                [
-                    {
-                        "message": "Variable 'string' must be non-nullable to be used"
-                        " for OneOf Input Object 'OneOfInput'.",
-                        "locations": [(4, 45)],
-                    },
-                ],
-            )
-
         def more_than_one_field():
             assert_errors(
                 """

tests/validation/test_variables_in_allowed_position.py

@@ -344,6 +344,18 @@ def allows_exactly_one_non_nullable_variable():
                 }
                 """)
 
+        def undefined_variable_in_one_of_input_object():
+            assert_errors(
+                """
+                {
+                  complicatedArgs {
+                    oneOfArgField(oneOfArg: { stringField: $undefinedVariable })
+                  }
+                }
+                """,
+                [],
+            )
+
         def forbids_one_nullable_variable():
             assert_errors(
                 """