exussum12\CoverageChecker\CloverLoader

Complexity

Total Complexity

16

Size/Duplication

Total Lines	104
Duplicated Lines	10.58 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	parseLines()	0	14	2
A	getErrorsOnLine()	0	10	3
A	handleNotFoundFile()	0	4	1
A	getDescription()	0	4	1
A	checkForNewFiles()	11	11	3
A	addLine()	0	9	2
A	handleStatement()	0	9	3

<?php
namespace exussum12\CoverageChecker;

use XMLReader;

/**
 * Class XMLReport
 * Used for reading in a phpunit clover XML file
 * @package exussum12\CoverageChecker
 */
class CloverLoader implements FileChecker
{
    /**
     * @var string
     */
    protected $file;
    /**
     * @var array
     */
    protected $coveredLines;

    /**
     * XMLReport constructor.
     * @param string $file the path the to phpunit clover file
     */
    public function __construct($file)
    {
        $this->file = $file;
    }

    /**
     * {@inheritdoc}
     */
    public function parseLines()
    {
        $this->coveredLines = [];
        $reader = new XMLReader;
        $reader->open($this->file);
        $currentFile = '';
        while ($reader->read()) {
            $currentFile = $this->checkForNewFiles($reader, $currentFile);

            $this->handleStatement($reader, $currentFile);
        }

        return array_keys($this->coveredLines);
    }

    /**
     * {@inheritdoc}
     */
    public function getErrorsOnLine($file, $line)
    {
        if (!isset($this->coveredLines[$file][$line])) {
            return null;
        }
        return $this->coveredLines[$file][$line] > 0 ?
            []:
            ['No unit test covering this line']
        ;
    }

    /**
     * {@inheritdoc}
     */
    public function handleNotFoundFile()
    {
        return null;
    }

    /**
     * {@inheritdoc}
     */
    public static function getDescription()
    {
        return 'Parses text output in clover (xml) format';
    }

    protected function checkForNewFiles($reader, $currentFile)

This method seems to be duplicated in your project.

    {
        if ((
            $reader->name === "file" &&
            $reader->nodeType == XMLReader::ELEMENT
        )) {
            $currentFile = $reader->getAttribute('name');
            $this->coveredLines[$currentFile] = [];
        }
        return $currentFile;
    }

    /**
     * @param $reader
     * @param $currentFile
     */
    protected function addLine($reader, $currentFile)
    {
        $covered = $reader->getAttribute('count') > 0;

        $this->coveredLines
        [$currentFile]
        [$reader->getAttribute('num')]
            = $covered ?: "No test coverage";
    }

    protected function handleStatement($reader, $currentFile)
    {
        if ((
            $reader->name === "line" &&
            $reader->getAttribute("type") == "stmt"
        )) {
            $this->addLine($reader, $currentFile);
        }
    }
}