PhpGitHooks\Module\PhpUnit\Tests\Behaviour\GuardCoverageToolHandlerTest

Complexity

Total Complexity

4

Size/Duplication

Total Lines	80
Duplicated Lines	27.5 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	9	1
A	itShouldShowWarningMessage()	0	23	1
A	itShouldWorksFine()	22	22	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\GuardCoverage;
use PhpGitHooks\Module\PhpUnit\Contract\Command\GuardCoverageToolHandler;
use PhpGitHooks\Module\PhpUnit\Tests\Infrastructure\PhpUnitUnitTestCase;

class GuardCoverageToolHandlerTest extends PhpUnitUnitTestCase
{
    /**
     * @var GuardCoverageToolHandler
     */
    private $guardCoverageToolCommandHandler;

    protected function setUp(): void
    {
        $this->guardCoverageToolCommandHandler = new GuardCoverageToolHandler(
            $this->getOutputInterface(),
            $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\Contract\Command\GuardCoverageToolHandler::__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\Contract\Command\GuardCoverageToolHandler::__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\Contract\Command\GuardCoverageToolHandler::__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(GuardCoverageToolHandler::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 GuardCoverage(HookQuestions::PHPUNIT_GUARD_COVERAGE_MESSAGE_DEFAULT)
        );
    }

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

This method seems to be duplicated in your project.

    {
        $outputMessage = new PreCommitOutputWriter(GuardCoverageToolHandler::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 GuardCoverage(HookQuestions::PHPUNIT_GUARD_COVERAGE_MESSAGE_DEFAULT)
        );
    }

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