App\Console\CoverageCommand::run()

Complexity

Conditions	7
Paths	24

Size

Total Lines	33
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function run()
    {
        $inputFile = isset($this->args[0]) ? $this->args[0] : "clover.xml";
        $targetCoverage = (int)(isset($argv[1]) ? $argv[1] : 1);

The variable $argv seems to never exist, and therefore isset should always return false. Did you maybe rename this variable?

        $percentage = min(100, max(0, $targetCoverage));

        if (!file_exists($inputFile)) {
            throw new InvalidArgumentException('Invalid input file provided');
        }

        if (!$percentage) {
            throw new InvalidArgumentException('An integer checked percentage must be given as second parameter');
        }

        $xml = new SimpleXMLElement(file_get_contents($inputFile));
        $metrics = $xml->xpath('//metrics');
        $totalElements = 0;
        $checkedElements = 0;

        foreach ($metrics as $metric) {
            $totalElements += (int)$metric['elements'];
            $checkedElements += (int)$metric['coveredelements'];
        }

        $coverage = ($checkedElements / $totalElements) * 100;

        if ($coverage < $percentage) {
            echo 'Code coverage is ' . $coverage . '%, which is below the accepted ' . $percentage . '%' . PHP_EOL;
            exit(1);
        }

        echo 'Code coverage is ' . round($coverage, 2) . '% - OK!' . PHP_EOL;
    }
}