Do not use ReflectionProperty::setAccessible() with PHP >= 8.1

Author: sebastianbergmann

ChangeLog-8.5.md

@@ -2,6 +2,12 @@
 
 All notable changes of the PHPUnit 8.5 release series are documented in this file using the [Keep a CHANGELOG](https://keepachangelog.com/) principles.
 
+## [8.5.43] - 2025-MM-DD
+
+### Changed
+
+* Do not use `ReflectionProperty::setAccessible()` with PHP >= 8.1
+
 ## [8.5.42] - 2025-05-02
 
 ### Changed
@@ -327,6 +333,7 @@ All notable changes of the PHPUnit 8.5 release series are documented in this fil
 * [#3967](https://github.com/sebastianbergmann/phpunit/issues/3967): Cannot double interface that extends interface that extends `\Throwable`
 * [#3968](https://github.com/sebastianbergmann/phpunit/pull/3968): Test class run in a separate PHP process are passing when `exit` called inside
 
+[8.5.43]: https://github.com/sebastianbergmann/phpunit/compare/8.5.42...8.5
 [8.5.42]: https://github.com/sebastianbergmann/phpunit/compare/8.5.41...8.5.42
 [8.5.41]: https://github.com/sebastianbergmann/phpunit/compare/8.5.40...8.5.41
 [8.5.40]: https://github.com/sebastianbergmann/phpunit/compare/8.5.39...8.5.40

src/Framework/Assert.php

@@ -11,6 +11,7 @@
 
 use const DEBUG_BACKTRACE_IGNORE_ARGS;
 use const PHP_EOL;
+use const PHP_VERSION;
 use function array_key_exists;
 use function array_shift;
 use function array_unshift;
@@ -33,6 +34,7 @@
 use function preg_split;
 use function sprintf;
 use function strpos;
+use function version_compare;
 use ArrayAccess;
 use Countable;
 use DOMAttr;
@@ -3514,9 +3516,15 @@ public static function getObjectAttribute($object, string $attributeName)
                     return $object->{$attributeName};
                 }
 
-                $attribute->setAccessible(true);
+                if (version_compare(PHP_VERSION, '8.1.0', '<')) {
+                    $attribute->setAccessible(true);
+                }
+
                 $value = $attribute->getValue($object);
-                $attribute->setAccessible(false);
+
+                if (version_compare(PHP_VERSION, '8.1.0', '<')) {
+                    $attribute->setAccessible(false);
+                }
 
                 return $value;
             } catch (ReflectionException $e) {

tests/unit/Framework/MockObject/MockObjectTest.php

@@ -9,11 +9,13 @@
  */
 namespace PHPUnit\Framework\MockObject;
 
+use const PHP_VERSION;
 use function class_uses;
 use function func_get_args;
 use function get_class;
 use function get_parent_class;
 use function sprintf;
+use function version_compare;
 use Exception;
 use PHPUnit\Framework\ExpectationFailedException;
 use PHPUnit\Framework\TestCase;
@@ -1195,7 +1197,11 @@ private function resetMockObjects(): void
         $refl = new ReflectionObject($this);
         $refl = $refl->getParentClass();
         $prop = $refl->getProperty('mockObjects');
-        $prop->setAccessible(true);
+
+        if (version_compare(PHP_VERSION, '8.1.0', '<')) {
+            $prop->setAccessible(true);
+        }
+
         $prop->setValue($this, []);
     }
 }