Merge branch '11.5'

Author: sebastianbergmann

src/Framework/MockObject/Generator/Generator.php

@@ -10,6 +10,7 @@
 namespace PHPUnit\Framework\MockObject\Generator;
 
 use const PHP_EOL;
+use const PHP_VERSION;
 use function array_merge;
 use function array_pop;
 use function array_unique;
@@ -22,14 +23,14 @@
 use function interface_exists;
 use function is_array;
 use function md5;
-use function method_exists;
 use function mt_rand;
 use function preg_match;
 use function serialize;
 use function sort;
 use function sprintf;
 use function substr;
 use function trait_exists;
+use function version_compare;
 use Exception;
 use Iterator;
 use IteratorAggregate;
@@ -48,7 +49,6 @@
 use ReflectionClass;
 use ReflectionMethod;
 use ReflectionObject;
-use ReflectionProperty;
 use SebastianBergmann\Type\ReflectionMapper;
 use SebastianBergmann\Type\Type;
 use Throwable;
@@ -815,7 +815,7 @@ private function configurableMethods(DoubledMethodSet $methods, array $propertie
      */
     private function properties(?ReflectionClass $class): array
     {
-        if (!method_exists(ReflectionProperty::class, 'isFinal')) {
+        if (version_compare('8.4.1', PHP_VERSION, '>')) {
             // @codeCoverageIgnoreStart
             return [];
             // @codeCoverageIgnoreEnd
@@ -829,12 +829,6 @@ private function properties(?ReflectionClass $class): array
         $properties = [];
 
         foreach ($class->getProperties() as $property) {
-            assert(method_exists($property, 'getHook'));
-            assert(method_exists($property, 'hasHooks'));
-            assert(method_exists($property, 'hasHook'));
-            assert(method_exists($property, 'isFinal'));
-            assert(class_exists(PropertyHookType::class));
-
             if (!$property->isPublic()) {
                 continue;
             }

tests/unit/Framework/MockObject/TestDoubleTestCase.php

@@ -11,7 +11,7 @@
 
 use Exception;
 use PHPUnit\Framework\Attributes\IgnorePhpunitDeprecations;
-use PHPUnit\Framework\Attributes\RequiresMethod;
+use PHPUnit\Framework\Attributes\RequiresPhp;
 use PHPUnit\Framework\Attributes\TestDox;
 use PHPUnit\Framework\Attributes\Ticket;
 use PHPUnit\Framework\MockObject\Runtime\PropertyHook;
@@ -25,7 +25,6 @@
 use PHPUnit\TestFixture\MockObject\InterfaceWithNeverReturningMethod;
 use PHPUnit\TestFixture\MockObject\InterfaceWithPropertyWithGetHook;
 use PHPUnit\TestFixture\MockObject\InterfaceWithReturnTypeDeclaration;
-use ReflectionProperty;
 use stdClass;
 
 abstract class TestDoubleTestCase extends TestCase
@@ -309,7 +308,7 @@ public function testDoubledMethodsCanBeCalledFromDestructorOnTestDoubleCreatedBy
         );
     }
 
-    #[RequiresMethod(ReflectionProperty::class, 'isFinal')]
+    #[RequiresPhp('^8.4')]
     public function testGetHookForPropertyOfInterfaceCanBeConfigured(): void
     {
         $double = $this->createTestDouble(InterfaceWithPropertyWithGetHook::class);
@@ -319,7 +318,7 @@ public function testGetHookForPropertyOfInterfaceCanBeConfigured(): void
         $this->assertSame('value', $double->property);
     }
 
-    #[RequiresMethod(ReflectionProperty::class, 'isFinal')]
+    #[RequiresPhp('^8.4')]
     public function testGetHookForPropertyOfExtendableClassCanBeConfigured(): void
     {
         $double = $this->createTestDouble(ExtendableClassWithPropertyWithGetHook::class);