PhpGitHooks\Module\Configuration\Contract\Query\ConfigurationDataFinderHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	81
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	4	1
A	handle()	0	6	1
A	getConfigurationDataResponse()	0	51	1

<?php

namespace PhpGitHooks\Module\Configuration\Contract\Query;

use Bruli\EventBusBundle\QueryBus\QueryHandlerInterface;
use Bruli\EventBusBundle\QueryBus\QueryInterface;
use PhpGitHooks\Infrastructure\Hook\PrePush;
use PhpGitHooks\Module\Configuration\Contract\Response\ConfigurationDataResponse;
use PhpGitHooks\Module\Configuration\Domain\CommitMsg;
use PhpGitHooks\Module\Configuration\Domain\Config;
use PhpGitHooks\Module\Configuration\Domain\PhpCs;
use PhpGitHooks\Module\Configuration\Domain\PhpCsFixer;
use PhpGitHooks\Module\Configuration\Domain\PhpMd;
use PhpGitHooks\Module\Configuration\Domain\PhpUnit;
use PhpGitHooks\Module\Configuration\Domain\PhpUnitGuardCoverage;
use PhpGitHooks\Module\Configuration\Domain\PhpUnitStrictCoverage;
use PhpGitHooks\Module\Configuration\Domain\PreCommit;
use PhpGitHooks\Module\Configuration\Model\ConfigurationFileReaderInterface;
use PhpGitHooks\Module\Configuration\Service\ConfigurationDataResponseFactory;

class ConfigurationDataFinderHandler implements QueryHandlerInterface
{
    /**
     * @var ConfigurationFileReaderInterface
     */
    private $configurationFileReader;

    /**
     * ConfigurationDataFinder constructor.
     *
     * @param ConfigurationFileReaderInterface $configurationFileReader
     */
    public function __construct(ConfigurationFileReaderInterface $configurationFileReader)
    {
        $this->configurationFileReader = $configurationFileReader;
    }

    private function getConfigurationDataResponse(Config $data)
    {
        /** @var PreCommit $preCommit */
        $preCommit = $data->getPreCommit();
        /** @var CommitMsg $commitMsg */
        $commitMsg = $data->getCommitMsg();
        $tools = $preCommit->getExecute()->execute();
        /** @var PrePush $prePush */
        $prePush = $data->getPrePush();
        $prePushTools = $prePush->getExecute()->execute();

The method getExecute() does not seem to exist on object<PhpGitHooks\Infrastructure\Hook\PrePush>.

        $composer = $tools[0];
        $jsonLint = $tools[1];
        $phpLint = $tools[2];
        /** @var PhpMd $phpMd */
        $phpMd = $tools[3];
        /** @var PhpCs $phpCs */
        $phpCs = $tools[4];
        /** @var PhpCsFixer $phpCsFixer */
        $phpCsFixer = $tools[5];
        /** @var PhpUnit $phpUnit */
        $phpUnit = $tools[6];
        /** @var PhpUnitStrictCoverage $phpUnitStrictCoverage */
        $phpUnitStrictCoverage = $tools[7];
        /** @var PhpUnitGuardCoverage $phpUnitGuardCoverage */
        $phpUnitGuardCoverage = $tools[8];
        /** @var PhpUnit $prePushPhpUnit */
        $prePushPhpUnit = $prePushTools[0];
        /** @var PhpUnitStrictCoverage $prePushStrictCoverage */
        $prePushStrictCoverage = $prePushTools[1];
        /** @var PhpUnitGuardCoverage $prePushGuardCoverage */
        $prePushGuardCoverage = $prePushTools[2];

        return ConfigurationDataResponseFactory::build(
            $preCommit,
            $composer,

$composer 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.

            $jsonLint,

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

            $phpLint,

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

            $phpMd,
            $phpCs,
            $phpCsFixer,
            $phpUnit,
            $phpUnitStrictCoverage,
            $phpUnitGuardCoverage,
            $commitMsg,
            $prePush,

$prePush is of type object<PhpGitHooks\Infrastructure\Hook\PrePush>, but the function expects a object<PhpGitHooks\Module\Configuration\Domain\PrePush>.

            $prePushPhpUnit,
            $prePushStrictCoverage,
            $prePushGuardCoverage
        );
    }

    /**
     * @param QueryInterface $query
     *
     * @return ConfigurationDataResponse
     */
    public function handle(QueryInterface $query)
    {
        $data = $this->configurationFileReader->getData();

        return $this->getConfigurationDataResponse($data);
    }
}