Rmiller\ErrorExtension\Tester\ErrorTester

Complexity

Total Complexity

9

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
A	test()	0	4	1
A	tearDown()	0	4	1
A	setUp()	0	7	1
A	turnOffErrorDisplayingIfNotVerbose()	0	6	2
A	registerShutdownFunction()	0	23	3

<?php

namespace Rmiller\ErrorExtension\Tester;

use Behat\Testwork\Environment\Environment;
use Behat\Testwork\Output\Printer\OutputPrinter;
use Behat\Testwork\Specification\SpecificationIterator;
use Behat\Testwork\Tester\Result\TestResult;
use Behat\Testwork\Tester\Setup\Setup;
use Behat\Testwork\Tester\Setup\Teardown;
use Behat\Testwork\Tester\SuiteTester;
use RMiller\ErrorExtension\Observer\ErrorObservers;
use Symfony\Component\Console\Helper\FormatterHelper;
use Symfony\Component\Console\Output\OutputInterface;

class ErrorTester implements SuiteTester
{
    private $baseTester;
    private $observers;
    private $output;

    public function __construct(
        SuiteTester $baseTester,
        OutputInterface $output,
        array $observers = null
    ) {
        $this->baseTester = $baseTester;
        $this->output = $output;
        $this->observers = new ErrorObservers($observers);

It seems like $observers defined by parameter $observers on line 25 can also be of type null; however, RMiller\ErrorExtension\Observer\ErrorObservers::__construct() does only seem to accept array, maybe add an additional type check?

    }

    /**
     * Tests provided suite specifications.
     *
     * @param Environment $env
     * @param SpecificationIterator $iterator
     * @param Boolean $skip
     *
     * @return TestResult
     */
    public function test(Environment $env, SpecificationIterator $iterator, $skip)
    {
        return $this->baseTester->test($env, $iterator, $skip);
    }

    /**
     * Tears down suite after a test.
     *
     * @param Environment $env
     * @param SpecificationIterator $iterator
     * @param Boolean $skip
     * @param TestResult $result
     *
     * @return Teardown
     */
    public function tearDown(Environment $env, SpecificationIterator $iterator, $skip, TestResult $result)
    {
        return $this->baseTester->tearDown($env, $iterator, $skip, $result);
    }

    /**
     * Sets up suite for a test.
     *
     * @param Environment $env
     * @param SpecificationIterator $iterator
     * @param Boolean $skip
     *
     * @return Setup
     */
    public function setUp(Environment $env, SpecificationIterator $iterator, $skip)
    {
        $this->turnOffErrorDisplayingIfNotVerbose();
        $this->registerShutdownFunction();

        return $this->baseTester->setUp($env, $iterator, $skip);
    }

    private function turnOffErrorDisplayingIfNotVerbose()
    {
        if ($this->output->getVerbosity() == OutputPrinter::VERBOSITY_NORMAL) {

The constant Behat\Testwork\Output\Printer\OutputPrinter::VERBOSITY_NORMAL has been deprecated with message: since 3.1, to be removed in 4.0

            error_reporting(E_ALL & ~E_ERROR);
        }
    }

    private function registerShutdownFunction()
    {
        register_shutdown_function(function () {
            if ($error = error_get_last()) {

                $errorMessages = [
                    sprintf('The error "%s"', $error['message']),
                    sprintf('occurred in file %s', $error['file']),
                    sprintf('at line %s', $error['line']),
                ];

                $formatter = new FormatterHelper();
                $formattedBlock = $formatter->formatBlock($errorMessages, 'error', true);
                $this->output->writeln('');
                $this->output->writeln($formattedBlock);
                $this->output->writeln('');

                foreach ($this->observers as $observer) {
                    $observer->notify($error);
                }
            }
        });
    }
}