PhpGitHooks\Module\PhpLint\Contract\Command\PhpLintToolHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	execute()	0	22	3
A	handle()	0	4	1

<?php

namespace PhpGitHooks\Module\PhpLint\Contract\Command;

use Bruli\EventBusBundle\CommandBus\CommandHandlerInterface;
use Bruli\EventBusBundle\CommandBus\CommandInterface;
use PhpGitHooks\Module\Git\Contract\Response\BadJobLogoResponse;
use PhpGitHooks\Module\Git\Service\PreCommitOutputWriter;
use PhpGitHooks\Module\PhpLint\Contract\Exception\PhpLintViolationsException;
use PhpGitHooks\Module\PhpLint\Model\PhpLintToolProcessorInterface;
use Symfony\Component\Console\Output\OutputInterface;

class PhpLintToolHandler implements CommandHandlerInterface
{
    const RUNNING_PHPLINT = 'Running PHPLINT';
    /**
     * @var PhpLintToolProcessorInterface
     */
    private $phpLintTool;
    /**
     * @var OutputInterface
     */
    private $output;

    /**
     * PhpLintTool constructor.
     *
     * @param PhpLintToolProcessorInterface $phpLintTool
     * @param OutputInterface               $output
     */
    public function __construct(PhpLintToolProcessorInterface $phpLintTool, OutputInterface $output)
    {
        $this->phpLintTool = $phpLintTool;
        $this->output = $output;
    }

    /**
     * @param array $files
     * @param string $errorMessage
     *
     * @throws PhpLintViolationsException
     */
    private function execute(array $files, $errorMessage)
    {
        $outputMessage = new PreCommitOutputWriter(self::RUNNING_PHPLINT);
        $this->output->write($outputMessage->getMessage());

        $errors = [];
        foreach ($files as $file) {
            $errors[] = $this->phpLintTool->process($file);
        }

        $errors = array_filter($errors);

        if (!empty($errors)) {
            $this->output->writeln($outputMessage->getFailMessage());
            $this->output->writeln($outputMessage->setError(implode('', $errors)));
            $this->output->writeln(BadJobLogoResponse::paint($errorMessage));

            throw new PhpLintViolationsException();
        }

        $this->output->writeln($outputMessage->getSuccessfulMessage());
    }

    /**
     * @param CommandInterface|PhpLintTool $command
     */
    public function handle(CommandInterface $command)
    {
        $this->execute($command->getFiles(), $command->getErrorMessage());

It seems like you code against a concrete implementation and not the interface Bruli\EventBusBundle\CommandBus\CommandInterface as the method getFiles() does only exist in the following implementations of said interface: PhpGitHooks\Module\Composer\Contract\Command\ComposerTool, PhpGitHooks\Module\JsonLint\Contract\Command\JsonLintTool, PhpGitHooks\Module\PhpCsFixer\Contract\Command\PhpCsFixerTool, PhpGitHooks\Module\PhpCs\Contract\Command\PhpCsTool, PhpGitHooks\Module\PhpLint\Contract\Command\PhpLintTool, PhpGitHooks\Module\PhpMd\Contract\Command\PhpMdTool.

It seems like you code against a concrete implementation and not the interface Bruli\EventBusBundle\CommandBus\CommandInterface as the method getErrorMessage() does only exist in the following implementations of said interface: PhpGitHooks\Module\Composer\Contract\Command\ComposerTool, PhpGitHooks\Module\JsonLint\Contract\Command\JsonLintTool, PhpGitHooks\Module\PhpCsFixer\Contract\Command\PhpCsFixerTool, PhpGitHooks\Module\PhpCs\Contract\Command\PhpCsTool, PhpGitHooks\Module\PhpLint\Contract\Command\PhpLintTool, PhpGitHooks\Module\PhpMd\Contract\Command\PhpMdTool, PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitTool, PhpGitHooks\Module\PhpUnit\Contract\Command\StrictCoverage.

    }
}