PhpGitHooks\Module\PhpUnit\Tests\Behaviour\GuardCoverageToolCommandHandlerTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	66
Duplicated Lines	59.09 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	2
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	11	1
A	itShouldWorksFine()	16	16	1
A	itShouldShowWarningMessage()	23	23	1

<?php

namespace PhpGitHooks\Module\PhpUnit\Tests\Behaviour;

use PhpGitHooks\Module\Configuration\Service\HookQuestions;
use PhpGitHooks\Module\Git\Service\PreCommitOutputWriter;
use PhpGitHooks\Module\PhpUnit\Contract\Command\GuardCoverageCommand;
use PhpGitHooks\Module\PhpUnit\Contract\CommandHandler\GuardCoverageToolCommandHandler;
use PhpGitHooks\Module\PhpUnit\Service\GuardCoverageTool;
use PhpGitHooks\Module\PhpUnit\Tests\Infrastructure\PhpUnitUnitTestCase;

class GuardCoverageToolCommandHandlerTest extends PhpUnitUnitTestCase
{
    /**
     * @var GuardCoverageToolCommandHandler
     */
    private $guardCoverageToolCommandHandler;

    protected function setUp()
    {
        $this->guardCoverageToolCommandHandler = new GuardCoverageToolCommandHandler(
            new GuardCoverageTool(
                $this->getOutputInterface(),

It seems like $this->getOutputInterface() targeting PhpGitHooks\Module\Git\Tests\Infrastructure\OutputInterfaceTrait::getOutputInterface() can also be of type object<Mockery\MockInterface>; however, PhpGitHooks\Module\PhpUnit\Service\GuardCoverageTool::__construct() does only seem to accept object<Symfony\Component\Console\Output\OutputInterface>, maybe add an additional type check?

                $this->getStrictCoverageProcessor(),

It seems like $this->getStrictCoverageProcessor() targeting PhpGitHooks\Module\PhpUnit\Tests\Infrastructure\StrictCoverageProcessorTrait::getStrictCoverageProcessor() can also be of type object<Mockery\MockInterface>; however, PhpGitHooks\Module\PhpUnit\Service\GuardCoverageTool::__construct() does only seem to accept object<PhpGitHooks\Module\PhpUnit\Model\StrictCoverageProcessorInterface>, maybe add an additional type check?

                $this->getGuardCoverageFileReader(),

It seems like $this->getGuardCoverageFileReader() targeting PhpGitHooks\Module\PhpUnit\Tests\Infrastructure\GuardCoverageFileReaderTrait::getGuardCoverageFileReader() can also be of type object<Mockery\MockInterface>; however, PhpGitHooks\Module\PhpUnit\Service\GuardCoverageTool::__construct() does only seem to accept object<PhpGitHooks\Module\PhpUnit\Model\GuardCoverageFileReaderInterface>, maybe add an additional type check?

                $this->getGuardCoverageFileWriter()

It seems like $this->getGuardCoverageFileWriter() targeting PhpGitHooks\Module\PhpUnit\Tests\Infrastructure\GuardCoverageFileWriterTrait::getGuardCoverageFileWriter() can also be of type object<Mockery\MockInterface>; however, PhpGitHooks\Module\PhpUnit\Service\GuardCoverageTool::__construct() does only seem to accept object<PhpGitHooks\Module\PhpUnit\Model\GuardCoverageFileWriterInterface>, maybe add an additional type check?

            )
        );
    }

    /**
     * @test
     */
    public function itShouldShowWarningMessage()

This method seems to be duplicated in your project.

    {
        $outputMessage = new PreCommitOutputWriter(GuardCoverageTool::CHECKING_MESSAGE);
        $currentCoverage = 60.00;
        $previousCoverage = 70.00;

        $this->shouldProcessStrictCoverage($currentCoverage);
        $this->shouldWriteOutput($outputMessage->getMessage());
        $this->shouldReadGuardCoverage($previousCoverage);
        $this->shouldWriteLnOutput(
            sprintf(
                "\n<bg=yellow;options=bold>%s Previous coverage %s, current coverage %s.</>",
                HookQuestions::PHPUNIT_GUARD_COVERAGE_MESSAGE_DEFAULT,
                $previousCoverage,
                $currentCoverage
            )
        );
        $this->shouldWriteGuardCoverage($currentCoverage);

        $this->guardCoverageToolCommandHandler->handle(
            new GuardCoverageCommand(HookQuestions::PHPUNIT_GUARD_COVERAGE_MESSAGE_DEFAULT)
        );
    }

    /**
     * @test
     */
    public function itShouldWorksFine()

This method seems to be duplicated in your project.

    {
        $outputMessage = new PreCommitOutputWriter(GuardCoverageTool::CHECKING_MESSAGE);
        $currentCoverage = 70.00;
        $previousCoverage = 60.00;

        $this->shouldProcessStrictCoverage($currentCoverage);
        $this->shouldWriteOutput($outputMessage->getMessage());
        $this->shouldReadGuardCoverage($previousCoverage);
        $this->shouldWriteLnOutput($outputMessage->getSuccessfulMessage());
        $this->shouldWriteGuardCoverage($currentCoverage);

        $this->guardCoverageToolCommandHandler->handle(
            new GuardCoverageCommand(HookQuestions::PHPUNIT_GUARD_COVERAGE_MESSAGE_DEFAULT)
        );
    }
}