PhpGitHooks\Module\Configuration\Service\PreCommitProcessor

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	20.97 %

Coupling/Cohesion

Components	1
Dependencies	13

Test Coverage

Coverage

100%

Importance

Changes	4
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	configTools()	0	15	1
A	process()	13	13	3

<?php

namespace PhpGitHooks\Module\Configuration\Service;

use Composer\IO\IOInterface;
use PhpGitHooks\Module\Configuration\Domain\Execute;
use PhpGitHooks\Module\Configuration\Domain\PreCommit;
use PhpGitHooks\Module\Configuration\Model\ExecuteInterface;

class PreCommitProcessor
{
    /**
     * @var IOInterface
     */
    private $io;
    /**
     * @var PhpUnitGuardCoverageConfigurator
     */
    private $phpUnitGuardCoverageConfigurator;

    /**
     * PreCommitProcessor constructor.
     *
     * @param PhpUnitGuardCoverageConfigurator $phpUnitGuardCoverageConfigurator
     */
    public function __construct(PhpUnitGuardCoverageConfigurator $phpUnitGuardCoverageConfigurator)
    {
        $this->phpUnitGuardCoverageConfigurator = $phpUnitGuardCoverageConfigurator;
    }

    /**
     * @param PreCommit   $preCommitData
     * @param IOInterface $input
     *
     * @return PreCommit
     */
    public function process(PreCommit $preCommitData, IOInterface $input)

This method seems to be duplicated in your project.

    {
        $this->io = $input;
        if (true === $preCommitData->isUndefined()) {
            $preCommitData = PreCommitConfigurator::configure($this->io, $preCommitData);
        }

        if (true === $preCommitData->isEnabled()) {
            $preCommitData = $preCommitData->setExecute($this->configTools($preCommitData->getExecute()));
        }

        return $preCommitData;
    }

    /**
     * @param ExecuteInterface $execute
     *
     * @return ExecuteInterface
     */
    private function configTools(ExecuteInterface $execute)
    {
        $tools = $execute->execute();
        $tools[0] = ComposerConfigurator::configure($this->io, $tools[0]);

$tools[0] of type object<PhpGitHooks\Module\Configuration\Model\ToolInterface> is not a sub-type of object<PhpGitHooks\Module\Configuration\Domain\Composer>. It seems like you assume a concrete implementation of the interface PhpGitHooks\Module\Configuration\Model\ToolInterface to be always present.

        $tools[1] = JsonLintConfigurator::configure($this->io, $tools[1]);
        $tools[2] = PhpLintConfigurator::configure($this->io, $tools[2]);
        $tools[3] = PhpMdConfigurator::configure($this->io, $tools[3]);
        $tools[4] = PhpCsConfigurator::configure($this->io, $tools[4]);
        $tools[5] = PhpCsFixerConfigurator::configure($this->io, $tools[5]);
        $tools[6] = PhpUnitConfigurator::configure($this->io, $tools[6]);
        $tools[7] = PhpUnitStrictCoverageConfigurator::configure($this->io, $tools[7]);
        $tools[8] = $this->phpUnitGuardCoverageConfigurator->configure($this->io, $tools[8]);

        return new Execute($tools);
    }
}