PhpGitHooks\Application\PhpUnit\PhpUnitHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Test Coverage

Coverage

100%

Importance

Changes	6
Bugs	0	Features	2

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	run()	0	15	2
A	setSuite()	0	4	1
A	setTitle()	0	6	1

<?php

namespace PhpGitHooks\Application\PhpUnit;

use PhpGitHooks\Application\Message\MessageConfigData;
use PhpGitHooks\Command\BadJobLogo;
use PhpGitHooks\Command\OutputHandlerInterface;
use PhpGitHooks\Infrastructure\Common\ProcessBuilderInterface;
use PhpGitHooks\Infrastructure\Common\ToolHandler;
use PhpGitHooks\Infrastructure\PhpUnit\PhpUnitProcessBuilder;

/**
 * Class PhpUnitHandler.
 */
class PhpUnitHandler extends ToolHandler
{
    /** @var PhpUnitProcessBuilder  */
    private $phpUnitProcessBuilder;

    /**
     * @param OutputHandlerInterface  $outputHandler
     * @param ProcessBuilderInterface $processBuilderInterface
     */
    public function __construct(OutputHandlerInterface $outputHandler, ProcessBuilderInterface $processBuilderInterface)
    {
        parent::__construct($outputHandler);

        $this->phpUnitProcessBuilder = $processBuilderInterface;

$processBuilderInterface is of type object<PhpGitHooks\Infrastructure\Common\ProcessBuilderInterface>, but the property $phpUnitProcessBuilder was declared to be of type object<PhpGitHooks\Infrastructure\PhpUnit\PhpUnitProcessBuilder>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * @param array $messages
     *
     * @throws UnitTestsException
     */
    public function run(array $messages)
    {
        $this->setTitle();

        $processBuilder = $this->phpUnitProcessBuilder->getProcessBuilder();
        $processBuilder->setTimeout(3600);
        $phpunit = $processBuilder->getProcess();
        $this->phpUnitProcessBuilder
            ->executeProcess($phpunit, $this->output);

        if (!$phpunit->isSuccessful()) {
            $this->output->writeln(BadJobLogo::paint($messages[MessageConfigData::KEY_ERROR_MESSAGE]));
            throw new UnitTestsException();
        }
    }

    /**
     * @param $suite
     */
    public function setSuite($suite)
    {
        $this->phpUnitProcessBuilder->setSuite($suite);
    }

    private function setTitle()
    {
        $this->outputHandler->setTitle('Running unit tests');
        $this->output->write($this->outputHandler->getTitle());
        $this->output->writeln($this->outputHandler->getSuccessfulStepMessage('Executing'));
    }
}