Merge pull request #24 from franmomu/fix/output_coverage_by_default

Fixed output html by default
henrikbjorn · Jul 4, 2014 · 987be21 · 987be21
2 parents 11afc5b + b5d7ca6
commit 987be21
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/spec/PhpSpec/Extension/Listener/CodeCoverageListenerSpec.php b/spec/PhpSpec/Extension/Listener/CodeCoverageListenerSpec.php
@@ -74,4 +74,30 @@ function it_should_output_text_report(
 
         $this->afterSuite($event);
     }
+
+    function it_should_output_html_report(
+        \PHP_CodeCoverage $coverage,
+        \PHP_CodeCoverage_Report_HTML $html,
+        SuiteEvent $event,
+        IO $io
+    ) {
+        $reports = array(
+            'html' => $html
+        );
+
+        $this->beConstructedWith($coverage, $reports);
+        $this->setOptions(array(
+            'format' => 'html',
+            'output' => array('html' => 'coverage'),
+        ));
+
+        $io->isVerbose()->willReturn(false);
+        $this->setIO($io);
+
+        $io->writeln('Generating code coverage report in html format ...')->shouldBeCalled();
+
+        $html->process($coverage, 'coverage')->willReturn('report');
+
+        $this->afterSuite($event);
+    }
 }
diff --git a/src/PhpSpec/Extension/Listener/CodeCoverageListener.php b/src/PhpSpec/Extension/Listener/CodeCoverageListener.php
@@ -22,7 +22,7 @@ public function __construct(\PHP_CodeCoverage $coverage, array $reports)
             'blacklist' => array('vendor', 'spec'),
             'whitelist_files' => array(),
             'blacklist_files' => array(),
-            'output'    => 'coverage',
+            'output'    => array('html' => 'coverage'),
             'format'    => array('html'),
         );
     }