PhpGitHooks\Module\PhpUnit\Tests\Behaviour\PhpUnitToolHandlerTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	8	1
A	itShouldThrowsException()	0	20	1
A	itShouldExecuteAndWorksFine()	0	17	1

<?php

namespace PhpGitHooks\Module\PhpUnit\Tests\Behaviour;

use PhpGitHooks\Module\Configuration\Service\HookQuestions;
use PhpGitHooks\Module\Git\Contract\Response\BadJobLogoResponse;
use PhpGitHooks\Module\Git\Service\PreCommitOutputWriter;
use PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitTool;
use PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitToolHandler;
use PhpGitHooks\Module\PhpUnit\Contract\Exception\PhpUnitViolationException;
use PhpGitHooks\Module\PhpUnit\Tests\Infrastructure\PhpUnitUnitTestCase;

class PhpUnitToolHandlerTest extends PhpUnitUnitTestCase
{
    /**
     * @var PhpUnitToolHandler
     */
    private $phpUnitToolCommandHandler;

    protected function setUp()
    {
        $this->phpUnitToolCommandHandler = new PhpUnitToolHandler(
            $this->getOutputInterface(),
            $this->getPhpUnitProcessor(),

It seems like $this->getPhpUnitProcessor() targeting PhpGitHooks\Module\PhpUnit\Tests\Infrastructure\PhpUnitProcessorTrait::getPhpUnitProcessor() can also be of type object<Mockery\MockInterface>; however, PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitToolHandler::__construct() does only seem to accept object<PhpGitHooks\Module\PhpUnit\Model\PhpUnitProcessorInterface>, maybe add an additional type check?

            $this->getPhpUnitProcessor()

It seems like $this->getPhpUnitProcessor() targeting PhpGitHooks\Module\PhpUnit\Tests\Infrastructure\PhpUnitProcessorTrait::getPhpUnitProcessor() can also be of type object<Mockery\MockInterface>; however, PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitToolHandler::__construct() does only seem to accept object<PhpGitHooks\Module\PhpUnit\Model\PhpUnitProcessorInterface>, maybe add an additional type check?

        );
    }

    /**
     * @test
     */
    public function itShouldThrowsException()
    {
        $this->expectException(PhpUnitViolationException::class);

        $options = '--testsuite default';
        $errorMessage = HookQuestions::PRE_COMMIT_ERROR_MESSAGE_DEFAULT;
        $outputMessage = new PreCommitOutputWriter(PhpUnitToolHandler::EXECUTING_MESSAGE);

        $this->shouldWriteLnOutput($outputMessage->getMessage());
        $this->shouldProcessPhpUnit($options, false);
        $this->shouldWriteLnOutput(BadJobLogoResponse::paint($errorMessage));

        $this->phpUnitToolCommandHandler->handle(
            new PhpUnitTool(
                true,
                $options,
                $errorMessage
            )
        );
    }

    /**
     * @test
     */
    public function itShouldExecuteAndWorksFine()
    {
        $options = '--testsuite default';
        $errorMessage = HookQuestions::PRE_COMMIT_ERROR_MESSAGE_DEFAULT;
        $outputMessage = new PreCommitOutputWriter(PhpUnitToolHandler::EXECUTING_MESSAGE);

        $this->shouldWriteLnOutput($outputMessage->getMessage());
        $this->shouldProcessPhpUnit($options, true);

        $this->phpUnitToolCommandHandler->handle(
            new PhpUnitTool(
                true,
                $options,
                $errorMessage
            )
        );
    }
}