Fix PHP 8.3/8.5 deprecations in TestCaseTrait reflection helpers

[remyperona]

Fixes #29

Changes

setValue() on static properties. set_reflective_property() forwarded $instance straight to ReflectionProperty::setValue(), but callers pass a class name when targeting a static property. Static properties are now set with an explicit null target:

if ( $property->isStatic() ) {
    $property->setValue( null, $value );
} else {
    $property->setValue( $instance, $value );
}

setAccessible(). No-op since PHP 8.1, deprecated as of 8.5, but still required on 7.4/8.0. All four call sites now route through a private set_reflector_accessible() helper that returns early when PHP_VERSION_ID >= 80100.

getNonPublicPropertyValue() picked up an isStatic() check so it no longer relies on the object argument being silently ignored for static properties. Behaviour is unchanged.

The library keeps its >=7.4 compatibility — the version guard is the reason setAccessible() is kept rather than deleted.

Testing

• Both deprecations reproduced verbatim against the pre-fix code on 8.3 and 8.5, confirming the right code paths were exercised.
• New Tests/Unit/testTestCaseTrait.php covers instance/static property set+get (via both a class name and an instance), non-public method invocation, and asserts no E_DEPRECATED is emitted. That last test fails against the old implementation on both 8.3 and 8.5, so it is a real guard.

Notes for the reviewer

• Pre-existing and out of scope: get_reflective_property() uses ReflectionClass::getProperty(), which does not find a private property declared on a parent class.

🤖 Generated with Claude Code