Paraunit\Printer\FinalPrinter

Complexity

Total Complexity

19

Size/Duplication

Total Lines	120
Duplicated Lines	16.67 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

98.61%

Importance

Changes	14
Bugs	2	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	onEngineEnd()	0	29	5
A	printAllFailuresOutput()	10	10	3
A	printFailuresOutput()	0	20	4
A	printAllFilesRecap()	10	10	3
A	printFileRecap()	0	20	3

<?php

namespace Paraunit\Printer;

use Paraunit\Lifecycle\EngineEvent;
use Paraunit\Parser\JSONLogParser;
use Paraunit\Parser\OutputContainerBearerInterface;
use Paraunit\Process\ParaunitProcessAbstract;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class FinalPrinter.
 */
class FinalPrinter
{
    /** @var  JSONLogParser */
    private $logParser;

    /** @var  OutputInterface */
    private $output;

    /**
     * FinalPrinter constructor.
     * @param JSONLogParser $logParser
     */
    public function __construct(JSONLogParser $logParser)
    {
        $this->logParser = $logParser;
    }

    /**
     * @param EngineEvent $engineEvent
     */
    public function onEngineEnd(EngineEvent $engineEvent)
    {
        if ( ! $engineEvent->has('start') || ! $engineEvent->has('end') || ! $engineEvent->has('process_completed')) {
            throw new \BadMethodCallException('missing argument/s');
        }

        $this->output = $engineEvent->getOutputInterface();
        /** @var \DateInterval $elapsedTime */
        $elapsedTime = $engineEvent->get('start')->diff($engineEvent->get('end'));
        $completedProcesses = $engineEvent->get('process_completed');

        $this->output->writeln('');
        $this->output->writeln('');
        $this->output->writeln($elapsedTime->format('Execution time -- %H:%I:%S '));

        $testsCount = 0;
        /** @var ParaunitProcessAbstract $process */
        foreach ($completedProcesses as $process) {
            $testsCount += count($process->getTestResults());
        }

        $this->output->writeln('');
        $this->output->writeln(sprintf('Executed: %d test classes, %d tests', count($completedProcesses), $testsCount));

        $this->printAllFailuresOutput();
        $this->printAllFilesRecap();

        $this->output->writeln('');
    }

    private function printAllFailuresOutput()

This method seems to be duplicated in your project.

    {
        $this->printFailuresOutput($this->logParser->getAbnormalTerminatedOutputContainer());

        foreach ($this->logParser->getParsersForPrinting() as $parser) {
            if ($parser instanceof OutputContainerBearerInterface) {
                $this->printFailuresOutput($parser->getOutputContainer());
            }
        }
    }

    /**
     * @param OutputContainerInterface $outputContainer
     */
    private function printFailuresOutput(OutputContainerInterface $outputContainer)
    {
        $buffer = $outputContainer->getOutputBuffer();
        if (count($buffer)) {
            $tag = $outputContainer->getTag();
            $this->output->writeln('');
            $this->output->writeln(sprintf('<%s>%s output:</%s>', $tag, ucwords($outputContainer->getTitle()), $tag));

            $i = 1;

            foreach ($buffer as $filename => $messages) {
                foreach ($messages as $message) {

The expression $messages of type string is not traversable.

                    $this->output->writeln('');
                    $this->output->writeln(
                        sprintf('<%s>%d)</%s> %s', $tag, $i++, $tag, $message)
                    );
                }
            }
        }
    }

    private function printAllFilesRecap()

This method seems to be duplicated in your project.

    {
        $this->printFileRecap($this->logParser->getAbnormalTerminatedOutputContainer());

        foreach ($this->logParser->getParsersForPrinting() as $parser) {
            if ($parser instanceof OutputContainerBearerInterface) {
                $this->printFileRecap($parser->getOutputContainer());
            }
        }
    }

    /**
     * @param OutputContainerInterface $outputContainer
     */
    private function printFileRecap(OutputContainerInterface $outputContainer)
    {
        if ($outputContainer->countFiles()) {
            $tag = $outputContainer->getTag();
            $this->output->writeln('');
            $this->output->writeln(
                sprintf(
                    '<%s>%d files with %s:</%s>',
                    $tag,
                    $outputContainer->countFiles(),
                    strtoupper($outputContainer->getTitle()),
                    $tag
                )
            );

            foreach ($outputContainer->getFileNames() as $fileName) {
                $this->output->writeln(sprintf(' <%s>%s</%s>', $tag, $fileName, $tag));
            }
        }
    }
}