Closes #5976

Author: sebastianbergmann

ChangeLog-11.5.md

@@ -2,6 +2,12 @@
 
 All notable changes of the PHPUnit 11.5 release series are documented in this file using the [Keep a CHANGELOG](https://keepachangelog.com/) principles.
 
+## [11.5.12] - 2025-MM-DD
+
+### Fixed
+
+* [#5976](https://github.com/sebastianbergmann/phpunit/issues/5976): TestDox result printer does not display details about errors triggered in before-first-test and after-last-test methods
+
 ## [11.5.11] - 2025-03-05
 
 ### Fixed
@@ -123,6 +129,7 @@ All notable changes of the PHPUnit 11.5 release series are documented in this fi
 * [#6055](https://github.com/sebastianbergmann/phpunit/issues/6055): `assertNotContainsOnly()` (use `assertContainsNotOnlyArray()`, `assertContainsNotOnlyBool()`, `assertContainsNotOnlyCallable()`, `assertContainsNotOnlyFloat()`, `assertContainsNotOnlyInt()`, `assertContainsNotOnlyIterable()`, `assertContainsNotOnlyNumeric()`, `assertContainsNotOnlyObject()`, `assertContainsNotOnlyResource()`, `assertContainsNotOnlyClosedResource()`, `assertContainsNotOnlyScalar()`, or `assertContainsNotOnlyString()` instead)
 * [#6059](https://github.com/sebastianbergmann/phpunit/issues/6059): `containsOnly()` (use `containsOnlyArray()`, `containsOnlyBool()`, `containsOnlyCallable()`, `containsOnlyFloat()`, `containsOnlyInt()`, `containsOnlyIterable()`, `containsOnlyNumeric()`, `containsOnlyObject()`, `containsOnlyResource()`, `containsOnlyClosedResource()`, `containsOnlyScalar()`, or `containsOnlyString()` instead)
 
+[11.5.12]: https://github.com/sebastianbergmann/phpunit/compare/11.5.10...11.5
 [11.5.11]: https://github.com/sebastianbergmann/phpunit/compare/11.5.10...11.5.11
 [11.5.10]: https://github.com/sebastianbergmann/phpunit/compare/11.5.9...11.5.10
 [11.5.9]: https://github.com/sebastianbergmann/phpunit/compare/11.5.8...11.5.9

src/TextUI/Output/Facade.php

@@ -92,7 +92,7 @@ public static function printResult(TestResult $result, ?array $testDoxResult, Du
         }
 
         if (self::$testDoxResultPrinter !== null && $testDoxResult !== null) {
-            self::$testDoxResultPrinter->print($testDoxResult);
+            self::$testDoxResultPrinter->print($result, $testDoxResult);
         }
 
         if (self::$defaultResultPrinter !== null) {

src/TextUI/Output/TestDox/ResultPrinter.php

@@ -17,12 +17,16 @@
 use function preg_match;
 use function preg_split;
 use function rtrim;
+use function sprintf;
 use function str_starts_with;
 use function trim;
 use PHPUnit\Event\Code\Throwable;
+use PHPUnit\Event\Test\AfterLastTestMethodErrored;
+use PHPUnit\Event\Test\BeforeFirstTestMethodErrored;
 use PHPUnit\Framework\TestStatus\TestStatus;
 use PHPUnit\Logging\TestDox\TestResult as TestDoxTestResult;
 use PHPUnit\Logging\TestDox\TestResultCollection;
+use PHPUnit\TestRunner\TestResult\TestResult;
 use PHPUnit\TextUI\Output\Printer;
 use PHPUnit\Util\Color;
 
@@ -49,7 +53,7 @@ public function __construct(Printer $printer, bool $colors, int $columns, bool $
     /**
      * @param array<string, TestResultCollection> $tests
      */
-    public function print(array $tests): void
+    public function print(TestResult $result, array $tests): void
     {
         $this->doPrint($tests, false);
 
@@ -58,6 +62,29 @@ public function print(array $tests): void
 
             $this->doPrint($tests, true);
         }
+
+        $beforeFirstTestMethodErrored = [];
+        $afterLastTestMethodErrored   = [];
+
+        foreach ($result->testErroredEvents() as $error) {
+            if ($error instanceof BeforeFirstTestMethodErrored) {
+                $beforeFirstTestMethodErrored[$error->calledMethod()->className() . '::' . $error->calledMethod()->methodName()] = $error;
+            }
+
+            if ($error instanceof AfterLastTestMethodErrored) {
+                $afterLastTestMethodErrored[$error->calledMethod()->className() . '::' . $error->calledMethod()->methodName()] = $error;
+            }
+        }
+
+        $this->printBeforeClassOrAfterClassErrors(
+            'These before-first-test methods errored:',
+            $beforeFirstTestMethodErrored,
+        );
+
+        $this->printBeforeClassOrAfterClassErrors(
+            'These after-last-test methods errored:',
+            $afterLastTestMethodErrored,
+        );
     }
 
     /**
@@ -407,4 +434,34 @@ private function symbolFor(TestStatus $status): string
 
         return '?';
     }
+
+    /**
+     * @param non-empty-string                                                                 $header
+     * @param array<non-empty-string, AfterLastTestMethodErrored|BeforeFirstTestMethodErrored> $errors
+     */
+    private function printBeforeClassOrAfterClassErrors(string $header, array $errors): void
+    {
+        if (empty($errors)) {
+            return;
+        }
+
+        $index = 0;
+
+        $this->printer->print($header . PHP_EOL . PHP_EOL);
+
+        foreach ($errors as $method => $error) {
+            $this->printer->print(
+                sprintf(
+                    '%d) %s' . PHP_EOL,
+                    ++$index,
+                    $method,
+                ),
+            );
+
+            $this->printer->print(trim($error->throwable()->description()) . PHP_EOL . PHP_EOL);
+            $this->printer->print($this->formatStackTrace($error->throwable()->stackTrace()) . PHP_EOL);
+        }
+
+        $this->printer->print(PHP_EOL);
+    }
 }

tests/end-to-end/regression/5976.phpt

@@ -0,0 +1,30 @@
+--TEST--
+https://github.com/sebastianbergmann/phpunit/issues/5976
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][] = '--do-not-cache-result';
+$_SERVER['argv'][] = '--no-configuration';
+$_SERVER['argv'][] = '--testdox';
+$_SERVER['argv'][] = __DIR__ . '/5976/Issue5976Test.php';
+
+require_once __DIR__ . '/../../bootstrap.php';
+
+(new PHPUnit\TextUI\Application)->run($_SERVER['argv']);
+--EXPECTF--
+PHPUnit %s by Sebastian Bergmann and contributors.
+
+Runtime: %s
+
+E                                                                   1 / 1 (100%)
+
+Time: %s, Memory: %s
+
+These before-first-test methods errored:
+
+1) PHPUnit\TestFixture\Issue5967\Issue5976Test::setUpBeforeClass
+Exception: message
+
+%sIssue5976Test.php:%d
+
+ERRORS!
+Tests: 1, Assertions: 0, Errors: 1.

tests/end-to-end/regression/5976/Issue5976Test.php

@@ -0,0 +1,28 @@
+<?php declare(strict_types=1);
+/*
+ * This file is part of PHPUnit.
+ *
+ * (c) Sebastian Bergmann <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+namespace PHPUnit\TestFixture\Issue5967;
+
+use Exception;
+use PHPUnit\Framework\TestCase;
+
+final class Issue5976Test extends TestCase
+{
+    /**
+     * @throws Exception
+     */
+    public static function setUpBeforeClass(): void
+    {
+        throw new Exception('message');
+    }
+
+    public function testOne(): void
+    {
+    }
+}