Filter out files which has no coverable line inside.

kahlan · Oct 16, 2016 · c7411ec · c7411ec
1 parent d3664c0
commit c7411ec
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/spec/Suite/Reporter/Coverage/Metrics.spec.php b/spec/Suite/Reporter/Coverage/Metrics.spec.php
@@ -282,6 +282,12 @@
 
             expect($metrics->get('Kahlan\Spec\Fixture\Reporter\Coverage\ImplementsCoverageInterface'))->toBe(null);
 
+            expect($collector->export())->toBe([
+                str_replace('/', DS, 'spec/Fixture/Reporter/Coverage/ImplementsCoverage.php') => [
+                    7 => 1
+                ]
+            ]);
+
         });
 
         describe("->children()", function () {

diff --git a/src/Reporter/Coverage/Collector.php b/src/Reporter/Coverage/Collector.php
@@ -302,8 +302,10 @@ public function export($file = null)
         }
         $result = [];
         $base = preg_quote($this->base(), '~');
-        foreach ($this->_coverage as $file => $coverage) {
-            $result[preg_replace("~^{$base}~", '', $file)] = $this->_coverage($file, $coverage);
+        foreach ($this->_coverage as $file => $rawCoverage) {
+            if ($coverage = $this->_coverage($file, $rawCoverage)) {
+                $result[preg_replace("~^{$base}~", '', $file)] = $coverage;
+            }
         }
         return $result;
     }
@@ -316,7 +318,7 @@ public function export($file = null)
     public function metrics()
     {
         $this->_metrics = new Metrics();
-        foreach ($this->_coverage as $file => $xdebug) {
+        foreach ($this->_coverage as $file => $rawCoverage) {
             $root = $this->parse($file);
             $coverage = $this->export($file);
             $this->_processed = [