From 9489d150446457c28c7d39f01f675b6cb89e880f Mon Sep 17 00:00:00 2001
From: jrfnl <jrfnl@users.noreply.github.com>
Date: Sun, 10 Aug 2025 05:23:23 +0200
Subject: [PATCH] PHP 8.5 | Tests: prevent deprecation notice for
 Reflection*::setAccessible()

Since PHP 8.1, calling the `Reflection*::setAccessible()` methods is no longer necessary as reflected properties/methods/etc will always be accessible.
However, the method calls are still needed for PHP < 8.1.

As of PHP 8.5, calling the `Reflection*::setAccessible()` methods is now formally deprecated and will yield a deprecation notice, which will fail test runs.
As of PHP 9.0, the `setAccessible()` method(s) will be removed.

With the latter in mind, this commit prevents the deprecation notice by making the calls to `setAccessible()` conditional.

Silencing the deprecation would mean, this would need to be "fixed" again come PHP 9.0, while the current solution should be stable, including for PHP 9.0.

Ref: https://wiki.php.net/rfc/deprecations_php_8_5#extreflection_deprecations
---
 tests/ColorLinesTest.php | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/ColorLinesTest.php b/tests/ColorLinesTest.php
index 32fefcc..55b0170 100644
--- a/tests/ColorLinesTest.php
+++ b/tests/ColorLinesTest.php
@@ -56,11 +56,11 @@ class ColorLinesTest extends HighlighterTestCase
     public function testColorLines($input, $expected, $withTheme = true)
     {
         $method = new ReflectionMethod('PHP_Parallel_Lint\PhpConsoleHighlighter\Highlighter', 'colorLines');
-        $method->setAccessible(true);
+        (\PHP_VERSION_ID < 80100) && $method->setAccessible(true);
 
         $highlighter = new Highlighter($this->getConsoleColorMock($withTheme));
         $output = $method->invoke($highlighter, $input);
-        $method->setAccessible(false);
+        (\PHP_VERSION_ID < 80100) && $method->setAccessible(false);
 
         $this->assertSame($expected, $output);
     }
@@ -107,11 +107,11 @@ public function testColorLinesWithRealColors()
         $color->setForceStyle(true);
 
         $method = new ReflectionMethod('PHP_Parallel_Lint\PhpConsoleHighlighter\Highlighter', 'colorLines');
-        $method->setAccessible(true);
+        (\PHP_VERSION_ID < 80100) && $method->setAccessible(true);
 
         $highlighter = new Highlighter($color);
         $output = $method->invoke($highlighter, self::$input);
-        $method->setAccessible(false);
+        (\PHP_VERSION_ID < 80100) && $method->setAccessible(false);
 
         $this->assertSame($expected, $output);
     }