Merge branch '12.3'

Author: sebastianbergmann

src/Framework/TestCase.php

@@ -26,6 +26,7 @@
 use function getcwd;
 use function implode;
 use function in_array;
+use function ini_get;
 use function ini_set;
 use function is_array;
 use function is_callable;
@@ -83,6 +84,7 @@
 use PHPUnit\Runner\BackedUpEnvironmentVariable;
 use PHPUnit\Runner\DeprecationCollector\Facade as DeprecationCollector;
 use PHPUnit\Runner\HookMethodCollection;
+use PHPUnit\Runner\ShutdownHandler;
 use PHPUnit\TestRunner\TestResult\PassedTests;
 use PHPUnit\TextUI\Configuration\Registry as ConfigurationRegistry;
 use PHPUnit\Util\Exporter;
@@ -1297,6 +1299,10 @@ private function runTest(): mixed
 
         $capture = tmpfile();
         assert($capture !== false);
+
+        if (ini_get('display_errors') === '0') {
+            ShutdownHandler::setMessage('Fatal error: Premature end of PHP process. Use display_errors=On to see the error message.');
+        }
         $errorLogPrevious = ini_set('error_log', stream_get_meta_data($capture)['uri']);
 
         try {
@@ -1331,6 +1337,7 @@ private function runTest(): mixed
 
             return null;
         } finally {
+            ShutdownHandler::resetMessage();
             fclose($capture);
 
             ini_set('error_log', $errorLogPrevious);

src/Runner/ShutdownHandler.php

@@ -0,0 +1,47 @@
+<?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\Runner;
+
+use function register_shutdown_function;
+
+/**
+ * @no-named-arguments Parameter names are not covered by the backward compatibility promise for PHPUnit
+ *
+ * @internal This class is not covered by the backward compatibility promise for PHPUnit
+ */
+final class ShutdownHandler
+{
+    private static bool $registered = false;
+    private static string $message  = '';
+
+    public static function setMessage(string $message): void
+    {
+        self::register();
+
+        self::$message = $message;
+    }
+
+    public static function resetMessage(): void
+    {
+        self::$message = '';
+    }
+
+    private static function register(): void
+    {
+        if (self::$registered) {
+            return;
+        }
+
+        register_shutdown_function(static function (): void
+        {
+            print self::$message;
+        });
+    }
+}

tests/end-to-end/regression/6294-display-errors-off.phpt

@@ -0,0 +1,18 @@
+--TEST--
+https://github.com/sebastianbergmann/phpunit/issues/6294
+--INI--
+display_errors=0
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][] = '--do-not-cache-result';
+$_SERVER['argv'][] = '--no-configuration';
+$_SERVER['argv'][] = __DIR__ . '/6294/IssueTest6294.php';
+
+require_once __DIR__ . '/../../bootstrap.php';
+(new PHPUnit\TextUI\Application)->run($_SERVER['argv']);
+--EXPECTF--
+PHPUnit %s by Sebastian Bergmann and contributors.
+
+Runtime: %s
+
+Fatal error: Premature end of PHP process. Use display_errors=On to see the error message.

tests/end-to-end/regression/6294-display-errors-on.phpt

@@ -0,0 +1,19 @@
+--TEST--
+https://github.com/sebastianbergmann/phpunit/issues/6294
+--INI--
+display_errors=1
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][] = '--do-not-cache-result';
+$_SERVER['argv'][] = '--no-configuration';
+$_SERVER['argv'][] = __DIR__ . '/6294/IssueTest6294.php';
+
+require_once __DIR__ . '/../../bootstrap.php';
+(new PHPUnit\TextUI\Application)->run($_SERVER['argv']);
+--EXPECTF--
+PHPUnit %s by Sebastian Bergmann and contributors.
+
+Runtime: %s
+
+
+Fatal error: Access level to PHPUnit\TestFixture\B::someFunction() must be public (as in class PHPUnit\TestFixture\A) in %sB.php on line %i%A

tests/end-to-end/regression/6294/A.php

@@ -0,0 +1,20 @@
+<?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;
+
+use const PHP_EOL;
+
+class A
+{
+    public function someFunction(): void
+    {
+        print 'A::someFunction' . PHP_EOL;
+    }
+}

tests/end-to-end/regression/6294/B.php

@@ -0,0 +1,23 @@
+<?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;
+
+use const PHP_EOL;
+
+class B extends A
+{
+    // incorrect access level
+    protected function someFunction(): void
+    {
+        parent::someFunction();
+
+        print 'B::someFunction' . PHP_EOL;
+    }
+}

tests/end-to-end/regression/6294/IssueTest6294.php

@@ -0,0 +1,24 @@
+<?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;
+
+use PHPUnit\Framework\TestCase;
+
+final class IssueTest6294 extends TestCase
+{
+    public function testOne(): void
+    {
+        require_once 'A.php';
+
+        require_once 'B.php';
+
+        $this->assertSame(1, 1);
+    }
+}