PhpGitHooks\Module\PhpUnit\Tests\Behaviour\GuardCoverageToolCommandHandlerTest

Complexity

Total Complexity

4

Size/Duplication

Total Lines	77
Duplicated Lines	22.08 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	3
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	11	1
A	itShouldShowWarningMessage()	0	23	1
A	itShouldWorksFine()	17	17	1
A	buildStrictCoverageSuccessfulMessage()	0	9	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()
    {
        $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($this->buildStrictCoverageSuccessfulMessage($currentCoverage, $previousCoverage,
            $outputMessage->getSuccessfulMessage()));
        $this->shouldWriteGuardCoverage($currentCoverage);

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

    private function buildStrictCoverageSuccessfulMessage($currentCoverage, $previousCoverage, $getSuccessfulMessage)
    {
        return $getSuccessfulMessage .
        ' <comment>[' .
        round($currentCoverage, 0) .
        '% >= ' .
        round($previousCoverage, 0) .
        '%]</comment>';
    }
}