Merge pull request #77 from clue-labs/php8.5

Run tests on PHP 8.5 and update test environment

Author: clue

.github/workflows/ci.yml

@@ -11,7 +11,7 @@ jobs:
     strategy:
       matrix:
         os:
-          - ubuntu-22.04
+          - ubuntu-24.04
         env:
           - client
           - server
@@ -22,8 +22,9 @@ jobs:
           - 8.2
           - 8.3
           - 8.4
+          - 8.5
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:

tests/unit/Messaging/FrameTest.php

@@ -162,14 +162,18 @@ public function testFirstPayloadDesignationValue(int $bits, string $bin): void {
         $this->_frame->addBuffer(static::encode($bin));
         $ref = new \ReflectionClass($this->_frame);
         $cb  = $ref->getMethod('getFirstPayloadVal');
-        $cb->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $cb->setAccessible(true);
+        }
         $this->assertEquals(bindec($bin), $cb->invoke($this->_frame));
     }
 
     public function testFirstPayloadValUnderflow(): void {
         $ref = new \ReflectionClass($this->_frame);
         $cb  = $ref->getMethod('getFirstPayloadVal');
-        $cb->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $cb->setAccessible(true);
+        }
         $this->expectException(\UnderflowException::class);
         $cb->invoke($this->_frame);
     }
@@ -182,14 +186,18 @@ public function testDetermineHowManyBitsAreUsedToDescribePayload(int $expected_b
         $this->_frame->addBuffer(static::encode($bin));
         $ref = new \ReflectionClass($this->_frame);
         $cb  = $ref->getMethod('getNumPayloadBits');
-        $cb->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $cb->setAccessible(true);
+        }
         $this->assertEquals($expected_bits, $cb->invoke($this->_frame));
     }
 
     public function testgetNumPayloadBitsUnderflow(): void {
         $ref = new \ReflectionClass($this->_frame);
         $cb  = $ref->getMethod('getNumPayloadBits');
-        $cb->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $cb->setAccessible(true);
+        }
         $this->expectException(\UnderflowException::class);
         $cb->invoke($this->_frame);
     }

tests/unit/Messaging/MessageBufferTest.php

@@ -263,7 +263,9 @@ function (Frame $frame) use (&$controlFrame): void {
      */
     public function testMemoryLimits(string $phpConfigurationValue, int $expectedLimit): void {
         $method = new \ReflectionMethod(MessageBuffer::class, 'getMemoryLimit');
-        $method->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $method->setAccessible(true);
+        }
         $actualLimit = $method->invoke(null, $phpConfigurationValue);
 
         $this->assertSame($expectedLimit, $actualLimit);
@@ -339,11 +341,15 @@ static function (Frame $frame): void {},
         }
 
         $prop = new \ReflectionProperty($messageBuffer, 'maxMessagePayloadSize');
-        $prop->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $prop->setAccessible(true);
+        }
         $this->assertEquals($expectedLimit / 4, $prop->getValue($messageBuffer));
 
         $prop = new \ReflectionProperty($messageBuffer, 'maxFramePayloadSize');
-        $prop->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $prop->setAccessible(true);
+        }
         $this->assertEquals($expectedLimit / 4, $prop->getValue($messageBuffer));
     }
 
@@ -365,11 +371,15 @@ function (Frame $frame) {},
         );
 
         $prop = new \ReflectionProperty($messageBuffer, 'maxMessagePayloadSize');
-        $prop->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $prop->setAccessible(true);
+        }
         $this->assertEquals(0, $prop->getValue($messageBuffer));
 
         $prop = new \ReflectionProperty($messageBuffer, 'maxFramePayloadSize');
-        $prop->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $prop->setAccessible(true);
+        }
         $this->assertEquals(0, $prop->getValue($messageBuffer));
     }
 }