Adds phpunit/[email protected] support

nunomaduro · Aug 4, 2023 · c67a90d · c67a90d
1 parent 69a0719
commit c67a90d
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 6 deletions.
diff --git a/composer.json b/composer.json
@@ -29,13 +29,10 @@
         "nunomaduro/larastan": "^2.6.3",
         "orchestra/testbench-core": "^8.5.8",
         "pestphp/pest": "^2.8.1",
-        "phpunit/phpunit": "^10.2.2",
+        "phpunit/phpunit": "^10.3.1",
         "sebastian/environment": "^6.0.1",
         "spatie/laravel-ignition": "^2.2.0"
     },
-    "conflict": {
-        "phpunit/phpunit": "<10.1.2"
-    },
     "autoload-dev": {
         "psr-4": {
             "Tests\\Printer\\": "tests/Printer",

diff --git a/src/Adapters/Phpunit/Printers/DefaultPrinter.php b/src/Adapters/Phpunit/Printers/DefaultPrinter.php
@@ -7,6 +7,7 @@
 use NunoMaduro\Collision\Adapters\Phpunit\ConfigureIO;
 use NunoMaduro\Collision\Adapters\Phpunit\State;
 use NunoMaduro\Collision\Adapters\Phpunit\Style;
+use NunoMaduro\Collision\Adapters\Phpunit\Support\ResultReflection;
 use NunoMaduro\Collision\Adapters\Phpunit\TestResult;
 use NunoMaduro\Collision\Exceptions\ShouldNotHappen;
 use NunoMaduro\Collision\Exceptions\TestOutcome;
@@ -368,7 +369,9 @@ public function testRunnerExecutionFinished(ExecutionFinished $event): void
     {
         $result = Facade::result();
 
-        if (Facade::result()->numberOfTests() === 0) {
+
+
+        if (ResultReflection::numberOfTests(Facade::result()) === 0) {
             $this->output->writeln([
                 '',
                 '  <fg=white;options=bold;bg=blue> INFO </> No tests found.',

diff --git a/src/Adapters/Phpunit/Style.php b/src/Adapters/Phpunit/Style.php
@@ -6,6 +6,7 @@
 
 use Closure;
 use NunoMaduro\Collision\Adapters\Phpunit\Printers\DefaultPrinter;
+use NunoMaduro\Collision\Adapters\Phpunit\Support\ResultReflection;
 use NunoMaduro\Collision\Exceptions\ShouldNotHappen;
 use NunoMaduro\Collision\Exceptions\TestException;
 use NunoMaduro\Collision\Exceptions\TestOutcome;
@@ -240,7 +241,7 @@ public function writeRecap(State $state, Info $telemetry, PHPUnitTestResult $res
             }
         }
 
-        $pending = $result->numberOfTests() - $result->numberOfTestsRun();
+        $pending = ResultReflection::numberOfTests($result) - $result->numberOfTestsRun();
         if ($pending > 0) {
             $tests[] = "\e[2m$pending pending\e[22m";
         }

diff --git a/src/Adapters/Phpunit/Support/ResultReflection.php b/src/Adapters/Phpunit/Support/ResultReflection.php
@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace NunoMaduro\Collision\Adapters\Phpunit\Support;
+
+use PHPUnit\TestRunner\TestResult\TestResult;
+
+/**
+ * @internal
+ */
+final class ResultReflection
+{
+    /**
+     * The number of processed tests.
+     */
+    public static function numberOfTests(TestResult $testResult): int
+    {
+        return (fn () => $this->numberOfTests)->call($testResult);
+    }
+}