Remove oneOf validation from values of correct type

Author: JoviDeCroock

src/validation/__tests__/ValuesOfCorrectTypeRule-test.ts

@@ -1094,22 +1094,6 @@ describe('Validate: Values of correct type', () => {
       ]);
     });
 
-    it('Exactly one nullable variable', () => {
-      expectErrors(`
-        query ($string: String) {
-          complicatedArgs {
-            oneOfArgField(oneOfArg: { stringField: $string })
-          }
-        }
-      `).toDeepEqual([
-        {
-          message:
-            'Variable "string" must be non-nullable to be used for OneOf Input Object "OneOfInput".',
-          locations: [{ line: 4, column: 37 }],
-        },
-      ]);
-    });
-
     it('More than one field', () => {
       expectErrors(`
         {

src/validation/rules/ValuesOfCorrectTypeRule.ts

@@ -87,7 +87,6 @@ export function ValuesOfCorrectTypeRule(
           node,
           type,
           fieldNodeMap,
-          variableDefinitions,
         );
       }
     },
@@ -185,7 +184,6 @@ function validateOneOfInputObject(
   node: ObjectValueNode,
   type: GraphQLInputObjectType,
   fieldNodeMap: ObjMap<ObjectFieldNode>,
-  variableDefinitions: { [key: string]: VariableDefinitionNode },
 ): void {
   const keys = Object.keys(fieldNodeMap);
   const isNotExactlyOneField = keys.length !== 1;
@@ -202,29 +200,12 @@ function validateOneOfInputObject(
 
   const value = fieldNodeMap[keys[0]]?.value;
   const isNullLiteral = !value || value.kind === Kind.NULL;
-  const isVariable = value?.kind === Kind.VARIABLE;
 
   if (isNullLiteral) {
     context.reportError(
       new GraphQLError(`Field "${type.name}.${keys[0]}" must be non-null.`, {
         nodes: [node],
       }),
     );
-    return;
-  }
-
-  if (isVariable) {
-    const variableName = value.name.value;
-    const definition = variableDefinitions[variableName];
-    const isNullableVariable = definition.type.kind !== Kind.NON_NULL_TYPE;
-
-    if (isNullableVariable) {
-      context.reportError(
-        new GraphQLError(
-          `Variable "${variableName}" must be non-nullable to be used for OneOf Input Object "${type.name}".`,
-          { nodes: [node] },
-        ),
-      );
-    }
   }
 }