add --metric/-m option flag

Author: patinthehat

src/Commands/CheckCoverageCommand.php

@@ -27,6 +27,7 @@ protected function configure(): void
     {
         $this->addArgument('filename')
             ->addOption('require', 'r', InputOption::VALUE_REQUIRED, 'Require a minimum code coverage percentage', null)
+            ->addOption('metric', 'm', InputOption::VALUE_REQUIRED, 'Use a specific metric field (element, statement, or method)')
             ->addOption('coverage-only', 'C', InputOption::VALUE_NONE, 'Display only the code coverage percentage')
             ->setDescription('Checks a clover-format coverage file for a minimum coverage percentage and optionally enforces it.');
     }
@@ -52,7 +53,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
             return $result ? Command::SUCCESS : Command::FAILURE;
         }
 
-        $checker = new CoverageChecker($this->config->filename);
+        $checker = new CoverageChecker($this->config->filename, $this->config);
         $coverage = $checker->getCoveragePercent();
 
         $this->displayCoverageResults($coverage);
@@ -62,7 +63,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
 
     protected function checkCoverage(string $filename, float $requiredPercentage): array
     {
-        $checker = new CoverageChecker($filename);
+        $checker = new CoverageChecker($filename, $this->config);
 
         return [$checker->check($requiredPercentage), $checker->getCoveragePercent()];
     }

src/Configuration/Configuration.php

@@ -16,18 +16,25 @@ class Configuration
     /** @var bool */
     public $displayCoverageOnly = false;
 
-    public function __construct(string $filename, bool $requireMode, $required, bool $displayCoverageOnly)
+    public $metricField = 'elements';
+
+    public function __construct(string $filename, bool $requireMode, $required, bool $displayCoverageOnly, string $metricField = 'element')
     {
         $this->filename = $filename;
         $this->required = $required;
         $this->requireMode = $requireMode;
         $this->displayCoverageOnly = $displayCoverageOnly;
+        $this->metricField = rtrim(strtolower($metricField), 's');
     }
 
     public function validate(): void
     {
         if (! file_exists($this->filename) || ! is_file($this->filename)) {
             throw new \InvalidArgumentException('Invalid input file provided.');
         }
+
+        if (! in_array($this->metricField, ['element', 'statement', 'method'])) {
+            throw new \InvalidArgumentException('Invalid metric field name provided. Valid options are "element", "statement", "method".');
+        }
     }
 }

src/Configuration/ConfigurationFactory.php

@@ -12,7 +12,12 @@ public static function create(InputInterface $input): Configuration
         $requireMode = $input->hasOption('require') && $input->getOption('require') !== null;
         $percentage = $requireMode ? (float)$input->getOption('require') : null;
         $displayCoverageOnly = $input->hasOption('coverage-only') && $input->getOption('coverage-only') === true;
+        $metricField = 'element';
 
-        return new Configuration($filename, $requireMode, $percentage, $displayCoverageOnly);
+        if ($input->hasOption('metric') && $input->getOption('metric') !== null) {
+            $metricField = $input->getOption('metric');
+        }
+
+        return new Configuration($filename, $requireMode, $percentage, $displayCoverageOnly, $metricField);
     }
 }

src/CoverageChecker.php

@@ -2,15 +2,20 @@
 
 namespace Permafrost\CoverageCheck;
 
+use Permafrost\CoverageCheck\Configuration\Configuration;
 use SimpleXMLElement;
 
 class CoverageChecker
 {
     /** @var SimpleXMLElement */
     public $xml;
 
-    public function __construct(string $filename)
+    /** @var Configuration */
+    public $config;
+
+    public function __construct(string $filename, Configuration $config)
     {
+        $this->config = $config;
         $this->xml = new SimpleXMLElement(file_get_contents($filename));
     }
 
@@ -29,8 +34,10 @@ public function getCoveragePercent(): float
     {
         $metrics = $this->xml->xpath('//metrics');
 
-        $totalElements = $this->getMetricFieldSum($metrics, 'elements');
-        $checkedElements = $this->getMetricFieldSum($metrics, 'coveredelements');
+        [$totalName, $coveredName] = $this->getMetricFieldNames();
+
+        $totalElements = $this->getMetricFieldSum($metrics, $totalName);
+        $checkedElements = $this->getMetricFieldSum($metrics, $coveredName);
 
         return round(($checkedElements / $totalElements) * 100, 4);
     }
@@ -39,4 +46,9 @@ public function check(float $minPercentage): bool
     {
         return $this->getCoveragePercent() >= $minPercentage;
     }
+
+    protected function getMetricFieldNames(): array
+    {
+        return ["{$this->config->metricField}s", "covered{$this->config->metricField}s"];
+    }
 }

tests/CoverageCheckerTest.php

@@ -2,6 +2,7 @@
 
 namespace Permafrost\CoverageCheck\Tests;
 
+use Permafrost\CoverageCheck\Configuration\Configuration;
 use Permafrost\CoverageCheck\CoverageChecker;
 use PHPUnit\Framework\TestCase;
 
@@ -10,15 +11,17 @@ class CoverageCheckerTest extends TestCase
     /** @test */
     public function it_gets_the_coverage_percentage()
     {
-        $checker = new CoverageChecker(__DIR__ . '/data/coverage-clover.xml');
+        $config = new Configuration(__DIR__ . '/data/coverage-clover.xml', false, false, 'element');
+        $checker = new CoverageChecker(__DIR__ . '/data/coverage-clover.xml', $config);
 
         $this->assertEquals(89.8765, round($checker->getCoveragePercent(), 4));
     }
 
     /** @test */
     public function it_checks_for_a_minimum_coverage_percentage()
     {
-        $checker = new CoverageChecker(__DIR__ . '/data/coverage-clover.xml');
+        $config = new Configuration(__DIR__ . '/data/coverage-clover.xml', false, false, 'element');
+        $checker = new CoverageChecker(__DIR__ . '/data/coverage-clover.xml', $config);
 
         $this->assertTrue($checker->check(75));
         $this->assertFalse($checker->check(99));