Fatal with some error/exception handlers about phpunit HOT 3 OPEN

Thank you for your report.

Please provide a minimal, self-contained, reproducing test case that shows the problem you are reporting.

Without such a minimal, self-contained, reproducing test case I will not be able to investigate this issue.

from phpunit.

Here's a really basic one (of course not very realistic, but it's minimal and self-contained). Same thing also (more realistic) is possible with exception handler

<?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 function set_error_handler;
use function trigger_error;
use PHPUnit\Framework\TestCase;

class SomeExternalDependency {
    public function __construct()
    {
        set_error_handler([$this, 'logError']);
    }

    private function logError(): bool
    {
        return false;
    }
}

class Issue5844Test extends TestCase
{

    public function testSetErrorHandlerNonCallable(): void
    {
        new SomeExternalDependency();

        trigger_error('error', E_USER_WARNING);

        $this->assertTrue(true);
    }
}

from phpunit.

Here is an repro: https://3v4l.org/GFd18

The issue is the callback must be restored in rebound context. I belive reflection can be used to detect the target context.

Here is docs https://www.php.net/manual/en/language.types.callable.php for all supportted callable types. All possibly private/not-callable-from-non-this must be tested.

php/php-src#14494 should be linked as IMO it is a little flaw in php design/callable validation on set_error_handler.

from phpunit.