Added test for the CoverageListener

symfony · lyrixx · Jun 12, 2017 · Sep 21, 2017 · Sep 22, 2017 · Sep 22, 2017
commit 30381f4e8530f595ba4be06c19721ca9f7c5e596
diff --git a/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/phpunit-with-listener.xml.dist b/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/phpunit-with-listener.xml.dist
@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/4.1/phpunit.xsd"
+         backupGlobals="false"
+         colors="true"
+         bootstrap="tests/bootstrap.php"
+         failOnRisky="true"
+         failOnWarning="true"
+>
+
+    <testsuites>
+        <testsuite name="Fixtures/coverage Test Suite">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+
+    <filter>
+        <whitelist>
+            <directory>src</directory>
+        </whitelist>
+    </filter>
+
+    <listeners>
+        <listener class="Symfony\Bridge\PhpUnit\CoverageListener" />
+    </listeners>
+</phpunit>
diff --git a/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/phpunit-without-listener.xml.dist b/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/phpunit-without-listener.xml.dist
@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/4.1/phpunit.xsd"
+         backupGlobals="false"
+         colors="true"
+         bootstrap="tests/bootstrap.php"
+         failOnRisky="true"
+         failOnWarning="true"
+>
+
+    <testsuites>
+        <testsuite name="Fixtures/coverage Test Suite">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+
+    <filter>
+        <whitelist>
+            <directory>src</directory>
+        </whitelist>
+    </filter>
+</phpunit>
diff --git a/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/src/Bar.php b/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/src/Bar.php
@@ -0,0 +1,27 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+class Bar
+{
+    private $foo;
+
+    public function __construct(Foo $foo)
+    {
+        $this->foo = $foo;
+    }
+
+    public function barZ()
+    {
+        $this->foo->fooZ();
+
+        return 'bar';
+    }
+}
diff --git a/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/src/Foo.php b/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/src/Foo.php
@@ -0,0 +1,18 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+class Foo
+{
+    public function fooZ()
+    {
+        return 'foo';
+    }
+}
diff --git a/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/tests/BarTest.php b/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/tests/BarTest.php
@@ -0,0 +1,23 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+use PHPUnit\Framework\TestCase;
+
+class BarTest extends TestCase
+{
+    public function testBar()
+    {
+        $foo = new Foo();
+        $bar = new Bar($foo);
+
+        $this->assertSame('bar', $bar->barZ());
+    }
+}
diff --git a/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/tests/bootstrap.php b/src/Symfony/Bridge/PhpUnit/Tests-Fixtures/coverage/tests/bootstrap.php
@@ -0,0 +1,19 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+require __DIR__.'/../src/Bar.php';
+require __DIR__.'/../src/Foo.php';
+
+require __DIR__.'/../../../Legacy/CoverageListenerTrait.php';
+if (class_exists('PHPUnit_Runner_Version') && version_compare(\PHPUnit_Runner_Version::id(), '6.0.0', '<')) {
+    require __DIR__.'/../../../Legacy/CoverageListener.php';
+}
+require __DIR__.'/../../../CoverageListener.php';
diff --git a/src/Symfony/Bridge/PhpUnit/Tests/CoverageListenerTest.php b/src/Symfony/Bridge/PhpUnit/Tests/CoverageListenerTest.php
@@ -0,0 +1,27 @@
+<?php
+
+namespace Symfony\Bridge\PhpUnit\Tests;
+
+use PHPUnit\Framework\TestCase;
+
+class CoverageListenerTest extends TestCase
+{
+    public function test()
+    {
+        if ("\n" !== PHP_EOL) {
+            $this->markTestSkipped('This test cannot be run on Windows.');
+        }
+
+        $dir = __DIR__.'/../Tests-Fixtures/coverage';
+            $php = PHP_BINARY;
+        $phpunit = $_SERVER['argv'][0];
+
+        exec("$php -d zend_extension=xdebug.so $phpunit -c $dir/phpunit-with-listener.xml.dist $dir/tests/ --coverage-text", $output);
+        $output = implode("\n", $output);
+        $this->assertNotContains('Foo', $output);
+
+        exec("$php -d zend_extension=xdebug.so $phpunit -c $dir/phpunit-without-listener.xml.dist $dir/tests/ --coverage-text", $output);
+        $output = implode("\n", $output);
+        $this->assertContains('Foo', $output);
+    }
+}