Overblog\GraphQLBundle\Command\ValidateCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

86.36%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	configure()	0	13	1
A	execute()	0	14	2

<?php

declare(strict_types=1);

namespace Overblog\GraphQLBundle\Command;

use GraphQL\Error\InvariantViolation;
use Overblog\GraphQLBundle\Resolver\TypeResolver;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

final class ValidateCommand extends Command
{
    use RequestExecutorLazyLoaderTrait;

    private $typeResolver;

    public function __construct(TypeResolver $typeResolver)
    {
        parent::__construct();
        $this->typeResolver = $typeResolver;
    }

    protected function configure(): void
    {
        $this
            ->setName('graphql:validate')
            ->setDescription('Validate schema')
            ->addOption(
                'schema',
                null,
                InputOption::VALUE_OPTIONAL,
                'The schema name to validate.'
            )
        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output): void
    {
        $schemaName = $input->getOption('schema');
        $schema = $this->getRequestExecutor()->getSchema($schemaName);

        try {
            @$schema->assertValid();

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        } catch (InvariantViolation $e) {

The class GraphQL\Error\InvariantViolation does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $output->writeln('<comment>'.$e->getMessage().'</comment>');

            return;
        }
        $output->writeln('<info>No error</info>');
    }
}