PhpGitHooks\Module\Configuration\Service\PrePushProcessor

Complexity

Total Complexity

4

Size/Duplication

Total Lines	41
Duplicated Lines	31.71 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	process()	13	13	3
A	configTools()	0	8	1

<?php

namespace PhpGitHooks\Module\Configuration\Service;

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

class PrePushProcessor
{
    /**
     * @var IOInterface
     */
    private $input;

    /**
     * @param PrePush     $prePushData
     * @param IOInterface $input
     *
     * @return PrePush
     */
    public function process(PrePush $prePushData, IOInterface $input)

This method seems to be duplicated in your project.

    {
        $this->input = $input;
        if (true === $prePushData->isUndefined()) {
            $prePushData = PrePushConfigurator::configure($this->input, $prePushData);
        }

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

        return $prePushData;
    }

    /**
     * @param ExecuteInterface $execute
     *
     * @return Execute
     */
    protected function configTools(ExecuteInterface $execute)
    {
        $tools = $execute->execute();

        $tools[0] = PhpUnitConfigurator::configure($this->input, $tools[0]);

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

        return new  Execute($tools);
    }
}