Unexpected input field value causes TypeError

[mrix]

Hi. I have this code in the userEdit field:

/**
 * @inheritdoc
 */
public function getName()
{
    return 'userEdit';
}
/**
 * @inheritdoc
 */
public function getType()
{
    return new UserType();
}
/**
 * @inheritdoc
 */
public function build(FieldConfig $config)
{
    $this->addArgument(
        new InputField([
            'name' => 'id',
            'type' => new NonNullType(new IntType()),
        ])
    )->addArgument(
        new InputField([
            'name' => 'data',
            'type' => new NonNullType(new InputObjectType([
                'name' => 'data',
                'fields' => [
                    [
                        'name' => 'fullName',
                        'type' => new NonNullType(new StringType()),
                    ],
                ],
            ])),
        ])
    );
    parent::build($config);
}

and the command:

mutation m {
  userEdit(id: 1000000, data: 0) {
    fullName
  }
}

returns:

{
  "errors": {
    "message": "Type error: Argument 1 passed to Youshido\\GraphQL\\Validator\\ResolveValidator\\ResolveValidator::processInputObject() must be an instance of Youshido\\GraphQL\\Parser\\Ast\\ArgumentValue\\InputObject, instance of Youshido\\GraphQL\\Parser\\Ast\\ArgumentValue\\Literal given, called in /var/www/symfony/vendor/youshido/graphql/src/Validator/ResolveValidator/ResolveValidator.php on line 101"
  }
}

Any unexpected value ([], 0, "str") for the data argument is not validated and causes to TypeError.

Also this request for the same schema:

mutation m {
  userEdit(id: 1000000, data: {unknownField: "a"}) {
    fullName
  }
}

returns only [] without any errors/exceptions.

[portey]

@mrix, thank's for you interesting. We've already fixed it in last release.