Fix for ignored Ignore attributes (#758)

src/WrapperRunner/ApplicationForWrapperWorker.php

@@ -11,6 +11,7 @@
 use PHPUnit\Logging\JUnit\JunitXmlLogger;
 use PHPUnit\Logging\TeamCity\TeamCityLogger;
 use PHPUnit\Logging\TestDox\TestResultCollector;
+use PHPUnit\Metadata\Api\CodeCoverage as CodeCoverageMetadataApi;
 use PHPUnit\Runner\CodeCoverage;
 use PHPUnit\Runner\Extension\ExtensionBootstrapper;
 use PHPUnit\Runner\Extension\Facade as ExtensionFacade;
@@ -65,6 +66,12 @@ public function runTest(string $testPath): int
 
         (new TestSuiteFilterProcessor())->process($this->configuration, $testSuite);
 
+        if (CodeCoverage::instance()->isActive()) {
+            CodeCoverage::instance()->ignoreLines(
+                (new CodeCoverageMetadataApi())->linesToBeIgnored($testSuite),
+            );
+        }
+
         EventFacade::emitter()->testRunnerExecutionStarted(
             TestSuiteBuilder::from($testSuite),
         );

test/Unit/WrapperRunner/WrapperRunnerTest.php

@@ -496,6 +496,25 @@ public function testHandleCollisionWithSymfonyOutput(): void
         self::assertStringContainsString('<bg=%s>', $runnerResult->output);
     }
 
+    /** @group github */
+    #[CoversNothing]
+    public function testIgnoreAttributes(): void
+    {
+        $this->bareOptions['--configuration'] = $this->fixture('github' . DIRECTORY_SEPARATOR . 'GH756' . DIRECTORY_SEPARATOR . 'phpunit.xml');
+        $this->bareOptions['--processes']     = '1';
+        $runnerResult                         = $this->runRunner();
+
+        $expectedContains = <<<'EOF'
+        ParaTest\Tests\fixtures\github\GH756\CoveredOneClass
+          Methods: 100.00% ( 1/ 1)   Lines: 100.00% (  1/  1)
+        ParaTest\Tests\fixtures\github\GH756\CoveredTwoClass
+          Methods: 100.00% ( 1/ 1)   Lines: 100.00% (  1/  1)
+        EOF;
+
+        self::assertEquals(0, $runnerResult->exitCode);
+        self::assertStringContainsString($expectedContains, $runnerResult->output);
+    }
+
     public function testProcessIsolation(): void
     {
         $this->bareOptions['path']                = $this->fixture('process_isolation' . DIRECTORY_SEPARATOR . 'FooTest.php');

test/fixtures/github/GH756/CoveredOneClass.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace ParaTest\Tests\fixtures\github\GH756;
+
+final class CoveredOneClass
+{
+    public function m(): bool
+    {
+        return $this->n();
+    }
+
+    private function n(): bool
+    {
+        return true;
+    }
+}

test/fixtures/github/GH756/CoveredOneTest.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace ParaTest\Tests\fixtures\github\GH756;
+
+use PHPUnit\Framework\Attributes\IgnoreMethodForCodeCoverage;
+use PHPUnit\Framework\TestCase;
+
+/** @internal */
+#[IgnoreMethodForCodeCoverage(CoveredOneClass::class, 'n')]
+final class CoveredOneTest extends TestCase
+{
+    public function testOne(): void
+    {
+        $this->assertTrue((new CoveredOneClass())->m());
+    }
+}

test/fixtures/github/GH756/CoveredTwoClass.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace ParaTest\Tests\fixtures\github\GH756;
+
+final class CoveredTwoClass
+{
+    public function m(): bool
+    {
+        return $this->n();
+    }
+
+    private function n(): bool
+    {
+        return true;
+    }
+}

test/fixtures/github/GH756/CoveredTwoTest.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace ParaTest\Tests\fixtures\github\GH756;
+
+use PHPUnit\Framework\Attributes\IgnoreMethodForCodeCoverage;
+use PHPUnit\Framework\TestCase;
+
+/** @internal */
+#[IgnoreMethodForCodeCoverage(CoveredTwoClass::class, 'n')]
+final class CoveredTwoTest extends TestCase
+{
+    public function testOne(): void
+    {
+        $this->assertTrue((new CoveredTwoClass())->m());
+    }
+}

test/fixtures/github/GH756/phpunit.xml

@@ -0,0 +1,19 @@
+<phpunit cacheDirectory="tmp">
+    <source>
+        <include>
+            <file>CoveredOneClass.php</file>
+            <file>CoveredTwoClass.php</file>
+        </include>
+    </source>
+    <coverage>
+        <report>
+            <text outputFile="php://stdout"/>
+        </report>
+    </coverage>
+    <testsuites>
+        <testsuite name="Github issue">
+            <file>CoveredOneTest.php</file>
+            <file>CoveredTwoTest.php</file>
+        </testsuite>
+    </testsuites>
+</phpunit>

test/fixtures/github/GH756/tmp/.gitignore

@@ -0,0 +1,2 @@
+*
+!.gitignore