Depends attribute and annotation only skip tests, but don't execute them on success about phpunit

[sebastianbergmann]

I cannot reproduce this.

Issue5826Test.php

<?php declare(strict_types=1);
namespace PHPUnit\TestFixture\Issue5826;

use PHPUnit\Framework\Attributes\Depends;
use PHPUnit\Framework\TestCase;

final class Issue5826Test extends TestCase
{
    public function testSucceeds(): void
    {
        self::assertTrue(true);
    }

    public function testFails(): void
    {
        self::assertTrue(false);
    }

    #[Depends('testSucceeds')]
    public function testDependsOnSucceedingTest(): void
    {
        self::assertTrue(true);
    }

    #[Depends('testFails')]
    public function testDependsOnFailingTest(): void
    {
        self::assertTrue(true);
    }
}

$ phpunit Issue5826Test.php --no-configuration --display-skipped
PHPUnit 10.5.19-20-g6da42eddf9 by Sebastian Bergmann and contributors.

Runtime:       PHP 8.3.6

.F.S                                                                4 / 4 (100%)

Time: 00:00.010, Memory: 8.00 MB

There was 1 failure:

1) PHPUnit\TestFixture\Issue5826\Issue5826Test::testFails
Failed asserting that false is true.

/home/sb/Issue5826Test.php:16

--

There was 1 skipped test:

1) PHPUnit\TestFixture\Issue5826\Issue5826Test::testDependsOnFailingTest
This test depends on "PHPUnit\TestFixture\Issue5826\Issue5826Test::testFails" to pass

FAILURES!
Tests: 4, Assertions: 3, Failures: 1, Skipped: 1.

[annadamm-check24]

Okay, sorry for this. I could not reproduce it in an empty project but in ours and digged a bit deeper.
We use dg/bypass-finals to not have to add interfaces to every class we use internally. Apparently this is a bug in that extension..