[3.x] PHP8.5

Author: WyriHaximus

.github/workflows/ci.yml

@@ -9,8 +9,10 @@ jobs:
     name: PHPUnit (PHP ${{ matrix.php }})
     runs-on: ubuntu-24.04
     strategy:
+      fail-fast: false
       matrix:
         php:
+          - 8.5
           - 8.4
           - 8.3
           - 8.2
@@ -42,7 +44,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: shivammathur/setup-php@v2
         with:
-          php-version: 8.2
+          php-version: 8.5
           coverage: xdebug
           ini-file: development
       - run: composer install

tests/DuplexResourceStreamTest.php

@@ -33,7 +33,9 @@ public function testConstructWithoutLoopAssignsLoopAutomatically(): void
         $stream = new DuplexResourceStream($resource);
 
         $ref = new \ReflectionProperty($stream, 'loop');
-        $ref->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $ref->setAccessible(true);
+        }
         $loop = $ref->getValue($stream);
 
         $this->assertInstanceOf('React\EventLoop\LoopInterface', $loop);

tests/ReadableResourceStreamTest.php

@@ -32,7 +32,9 @@ public function testConstructWithoutLoopAssignsLoopAutomatically(): void
         $stream = new ReadableResourceStream($resource);
 
         $ref = new \ReflectionProperty($stream, 'loop');
-        $ref->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $ref->setAccessible(true);
+        }
         $loop = $ref->getValue($stream);
 
         $this->assertInstanceOf('React\EventLoop\LoopInterface', $loop);

tests/WritableResourceStreamTest.php

@@ -31,7 +31,9 @@ public function testConstructWithoutLoopAssignsLoopAutomatically(): void
         $stream = new WritableResourceStream($resource);
 
         $ref = new \ReflectionProperty($stream, 'loop');
-        $ref->setAccessible(true);
+        if (PHP_VERSION_ID < 80100) {
+            $ref->setAccessible(true);
+        }
         $loop = $ref->getValue($stream);
 
         $this->assertInstanceOf('React\EventLoop\LoopInterface', $loop);
@@ -509,6 +511,10 @@ public function testDoubleCloseWillEmitOnlyOnce(): void
      */
     public function testWritingToClosedWritableResourceStreamShouldNotWriteToStream(): void
     {
+        if (PHP_VERSION_ID > 80499) {
+            $this->markTestSkipped('Since PHP8.5 attempting to write to a closed stream will result in an error');
+        }
+
         $stream = fopen('php://temp', 'r+');
         assert(is_resource($stream));