Fix Xml Report and implement XmlTest

Author: ek9

src/Common/Report/Xml.php

@@ -10,7 +10,7 @@
 
 use LeanPHP\Behat\CodeCoverage\Common\ReportInterface;
 use SebastianBergmann\CodeCoverage\CodeCoverage;
-use SebastianBergmann\CodeCoverage\Report\XML as XMLCC;
+use SebastianBergmann\CodeCoverage\Report\Xml\Facade;;
 
 /**
  * XML report
@@ -34,15 +34,15 @@ class Xml implements ReportInterface
      */
     public function __construct(array $options)
     {
-        if ( ! class_exists('XMLCC')) {
+        if ( ! class_exists('SebastianBergmann\CodeCoverage\Report\Xml\Facade')) {
             throw new \Exception('XML requires CodeCoverage 4.0+');
         }
 
         if ( ! isset($options['target'])) {
             $options['target'] = null;
         }
 
-        $this->report = new XMLCC();
+        $this->report = new Facade(array());
         $this->options = $options;
     }
 

tests/Common/Report/PhpTest.php

@@ -12,6 +12,8 @@
 use LeanPHP\Behat\CodeCoverage\Common\Report\Factory;
 use SebastianBergmann\CodeCoverage\Filter;
 use SebastianBergmann\CodeCoverage\CodeCoverage;
+use SebastianBergmann\CodeCoverage\Report\Php;
+
 /**
  * PHP report test
  *
@@ -32,7 +34,7 @@ public function testProcess()
                   ->will($this->returnValue($filter));
 
 
-        $report = new PHP(array());
+        $report = new Php(array());
         $result = $report->process($coverage);
 
         $this->assertTrue(strncmp($result, '<?php', 2) === 0);

tests/Common/Report/XmlTest.php

@@ -10,7 +10,8 @@
 
 use VIPSoft\TestCase;
 use LeanPHP\Behat\CodeCoverage\Common\Report\Factory;
-use SebastianBergmann\CodeCoverage\Report\XML;
+use SebastianBergmann\CodeCoverage\Report\Xml\Facade;
+use org\bovigo\vfs\vfsStream;
 
 /**
  * XML report test
@@ -21,12 +22,33 @@ class XmlTest extends TestCase
 {
     public function testProcess()
     {
-        if ( ! class_exists('XML')) {
-            $this->markTestSkipped();
+        vfsStream::setup('tmp');
+        $target = vfsStream::url('tmp');
+
+        file_put_contents($target . '/file', "test\n");
+
+        $report = new \SebastianBergmann\CodeCoverage\Node\Directory($target);
+        $report->addFile('file', array('class' => array(1 => 1)), array(), false);
+
+        $coverage = $this->createMock('SebastianBergmann\CodeCoverage\CodeCoverage');
+        $coverage->expects($this->atLeast(1))
+                 ->method('getReport')
+                 ->will($this->returnValue($report));
+        $coverage->expects($this->once())
+                 ->method('getTests')
+                 ->will($this->returnValue(array()));
 
-            return;
-        }
 
-        $this->markTestIncomplete();
+
+        $report = new Xml(array(
+            'target' => $target,
+        ));
+
+        try {
+            $result = $report->process($coverage);
+        } catch (\Exception $e) {
+            print_r($e->getMessage());
+            $this->fail();
+        }
     }
 }