DVDoug\Behat\CodeCoverage\Common\Report\Html

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	20	4
A	process()	0	5	1

<?php

declare(strict_types=1);
/**
 * Code Coverage HTML Report.
 *
 * @copyright 2013 Anthon Pang
 *
 * @license BSD-3-Clause
 */

namespace DVDoug\Behat\CodeCoverage\Common\Report;

use DVDoug\Behat\CodeCoverage\Common\ReportInterface;
use SebastianBergmann\CodeCoverage\CodeCoverage;
use SebastianBergmann\CodeCoverage\Report\Html\Facade;

/**
 * HTML report.
 *
 * @author Anthon Pang <apang@softwaredevelopment.ca>
 */
class Html implements ReportInterface
{
    /**
     * @var Facade
     */
    private $report;

    /**
     * @var array
     */
    private $options;

    /**
     * {@inheritdoc}
     */
    public function __construct(array $options)
    {
        if (!isset($options['target'])) {
            $options['target'] = null;
        }

        if (!isset($options['lowUpperBound'])) {
            $options['lowUpperBound'] = 50;
        }

        if (!isset($options['highLowerBound'])) {
            $options['highLowerBound'] = 90;
        }

        $this->report = new Facade(
            $options['lowUpperBound'],
            $options['highLowerBound']
        );

        $this->options = $options;
    }

    /**
     * {@inheritdoc}
     */
    public function process(CodeCoverage $coverage)
    {
        return $this->report->process(

Are you sure the usage of $this->report->process($coverage, $this->options['target']) targeting SebastianBergmann\CodeCoverage\Report\Html\Facade::process() seems to always return null.

The expression return $this->report->process($coverage, $this->options['target']) returns the type void which is incompatible with the return type mandated by DVDoug\Behat\CodeCoverage\Common\ReportInterface::process() of null|string.

            $coverage,
            $this->options['target']
        );
    }
}