Merge pull request #6 from robiningelbrecht/add-runtime-info

Add runtime info to output
robiningelbrecht · Sep 8, 2023 · 5d42dbf · 5d42dbf
2 parents b8a4c72 + d6e10a3
commit 5d42dbf
Show file tree

Hide file tree

Showing 11 changed files with 304 additions and 385 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
 vendor
 .phpunit.cache
+.phpunit.result.cache
 .php-cs-fixer.cache
diff --git a/composer.json b/composer.json
@@ -16,8 +16,8 @@
   ],
   "require": {
     "php": "^8.1",
-    "nunomaduro/collision": "^7.5",
-    "phpunit/phpunit": "^10.1"
+    "nunomaduro/collision": "^7.8",
+    "phpunit/phpunit": "^10.3"
   },
   "autoload": {
     "psr-4": {
@@ -30,7 +30,6 @@
     }
   },
   "require-dev": {
-    "dg/bypass-finals": "^1.4",
     "friendsofphp/php-cs-fixer": "^3.15",
     "phpstan/phpstan": "^1.10",
     "spatie/phpunit-snapshot-assertions": "^5.0"
@@ -40,6 +39,6 @@
   },
   "scripts": {
     "lint:fix": " ./vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.dist.php",
-    "phpunit:test": "vendor/bin/phpunit --configuration=tests/phpunit.test.xml --no-output"
+    "phpunit:test": "vendor/bin/phpunit --configuration=tests/phpunit.test.xml"
   }
 }
diff --git a/composer.lock b/composer.lock
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -6,13 +6,7 @@
          colors="true"
          defaultTestSuite="unit"
          cacheDirectory=".phpunit.cache"
-         displayDetailsOnTestsThatTriggerWarnings="true"
-         bootstrap="tests/bootstrap.php">
-  <extensions>
-    <bootstrap class="RobinIngelbrecht\PHPUnitPrettyPrint\PhpUnitExtension">
-
-    </bootstrap>
-  </extensions>
+         displayDetailsOnTestsThatTriggerWarnings="true">
   <testsuites>
     <testsuite name="unit">
       <file>tests/OutputTest.php</file>

diff --git a/src/PhpUnitExtension.php b/src/PhpUnitExtension.php
@@ -8,6 +8,7 @@
 use PHPUnit\Runner\Extension\ParameterCollection;
 use PHPUnit\TextUI\Configuration\Configuration as PHPUnitConfiguration;
 use RobinIngelbrecht\PHPUnitPrettyPrint\Subscriber\Application\ApplicationFinishedSubscriber;
+use RobinIngelbrecht\PHPUnitPrettyPrint\Subscriber\Application\ApplicationStartedSubscriber;
 
 final class PhpUnitExtension implements Extension
 {
@@ -33,6 +34,7 @@ public function bootstrap(PHPUnitConfiguration $configuration, Facade $facade, P
 
         EnsurePrinterIsRegisteredSubscriber::register();
 
+        $facade->registerSubscriber(new ApplicationStartedSubscriber());
         if (!$configuration->displayQuote()) {
             return;
         }

diff --git a/src/Subscriber/Application/ApplicationStartedSubscriber.php b/src/Subscriber/Application/ApplicationStartedSubscriber.php
@@ -0,0 +1,20 @@
+<?php
+
+namespace RobinIngelbrecht\PHPUnitPrettyPrint\Subscriber\Application;
+
+use PHPUnit\Event\Application\Started;
+use PHPUnit\Event\Application\StartedSubscriber;
+
+use function Termwind\render;
+
+class ApplicationStartedSubscriber implements StartedSubscriber
+{
+    public function notify(Started $event): void
+    {
+        render('<br />');
+        render(sprintf(
+            '<div>&nbsp;&nbsp;Runtime: %s</div>',
+            $event->runtime()->asString()
+        ));
+    }
+}
diff --git a/tests/OutputTest.php b/tests/OutputTest.php
@@ -30,6 +30,8 @@ public function testWithProfiling(): void
 
         exec(implode(' ', $command), $out);
         $output = implode(PHP_EOL, $out);
+
+        $this->assertMatchesRegularExpression('/Runtime: PHPUnit [\s\S]+ using PHP [\s\S]+ on [\s\S]+/', $output);
         $this->assertStringContainsString('Tests:    2 passed (4 assertions)', $output);
         $this->assertStringContainsString('Top 10 slowest tests', $output);
     }
@@ -44,6 +46,8 @@ public function testWithProfilingAtRunTime(): void
         ];
         exec(implode(' ', $command), $out);
         $output = implode(PHP_EOL, $out);
+
+        $this->assertMatchesRegularExpression('/Runtime: PHPUnit [\s\S]+ using PHP [\s\S]+ on [\s\S]+/', $output);
         $this->assertStringContainsString('Tests:    2 passed (4 assertions)', $output);
         $this->assertStringContainsString('Top 10 slowest tests', $output);
     }
@@ -57,6 +61,8 @@ public function testPrintCompactMode(): void
 
         exec(implode(' ', $command), $out);
         $output = implode(PHP_EOL, $out);
+
+        $this->assertMatchesRegularExpression('/Runtime: PHPUnit [\s\S]+ using PHP [\s\S]+ on [\s\S]+/', $output);
         $this->assertStringContainsString('.⨯⨯⨯!si..', $output);
         $this->assertStringContainsString('Tests:    3 failed, 1 risky, 1 incomplete, 1 skipped, 3 passed (7 assertions)', $output);
     }
@@ -70,6 +76,8 @@ public function testPrintCompactModeAtRunTime(): void
         ];
         exec(implode(' ', $command), $out);
         $output = implode(PHP_EOL, $out);
+
+        $this->assertMatchesRegularExpression('/Runtime: PHPUnit [\s\S]+ using PHP [\s\S]+ on [\s\S]+/', $output);
         $this->assertStringContainsString('.⨯⨯⨯!si..', $output);
         $this->assertStringContainsString('Tests:    3 failed, 1 risky, 1 incomplete, 1 skipped, 3 passed (7 assertions)', $output);
     }
@@ -137,6 +145,8 @@ public function testItShouldBeEnabled(): void
         ];
         exec(implode(' ', $command), $out);
         $output = implode(PHP_EOL, $out);
+
+        $this->assertMatchesRegularExpression('/Runtime: PHPUnit [\s\S]+ using PHP [\s\S]+ on [\s\S]+/', $output);
         $this->assertStringContainsString('Tests:    2 passed (4 assertions)', $output);
     }
 
@@ -149,6 +159,8 @@ public function testItShouldBeDisabled(): void
         ];
         exec(implode(' ', $command), $out);
         $output = implode(PHP_EOL, $out);
+
+        $this->assertDoesNotMatchRegularExpression('/Runtime: PHPUnit [\s\S]+ using PHP [\s\S]+ on [\s\S]+/', $output);
         $this->assertStringContainsString('OK (2 tests, 4 assertions)', $output);
     }
 }