Merge pull request #269 from clue-labs/php8.3

Test on PHP 8.3 and update test environment

Author: WyriHaximus

.github/workflows/ci.yml

@@ -11,6 +11,7 @@ jobs:
     strategy:
       matrix:
         php:
+          - 8.3
           - 8.2
           - 8.1
           - 8.0
@@ -24,7 +25,7 @@ jobs:
           - 5.4
           - 5.3
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php }}
@@ -57,7 +58,7 @@ jobs:
           - 5.4
           - 5.3
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php }}
@@ -106,6 +107,7 @@ jobs:
     strategy:
       matrix:
         php:
+          - 8.3
           - 8.2
           - 8.1
           - 8.0
@@ -114,7 +116,7 @@ jobs:
           - 7.2
           - 7.1
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php }}
@@ -132,7 +134,7 @@ jobs:
     runs-on: ubuntu-22.04
     continue-on-error: true
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - run: cp "$(which composer)" composer.phar && ./composer.phar self-update --2.2 # downgrade Composer for HHVM
       - name: Run hhvm composer.phar install
         uses: docker://hhvm/hhvm:3.30-lts-latest

tests/LoopTest.php

@@ -4,7 +4,6 @@
 
 use React\EventLoop\Factory;
 use React\EventLoop\Loop;
-use ReflectionClass;
 
 final class LoopTest extends TestCase
 {
@@ -66,7 +65,7 @@ public function testStaticAddReadStreamWithNoDefaultLoopCallsAddReadStreamOnNewL
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $stream = stream_socket_server('127.0.0.1:0');
         $listener = function () { };
@@ -92,7 +91,7 @@ public function testStaticAddWriteStreamWithNoDefaultLoopCallsAddWriteStreamOnNe
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $stream = stream_socket_server('127.0.0.1:0');
         $listener = function () { };
@@ -117,7 +116,7 @@ public function testStaticRemoveReadStreamWithNoDefaultLoopIsNoOp()
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $stream = tmpfile();
         Loop::removeReadStream($stream);
@@ -141,7 +140,7 @@ public function testStaticRemoveWriteStreamWithNoDefaultLoopIsNoOp()
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $stream = tmpfile();
         Loop::removeWriteStream($stream);
@@ -169,7 +168,7 @@ public function testStaticAddTimerWithNoDefaultLoopCallsAddTimerOnNewLoopInstanc
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $interval = 1.0;
         $callback = function () { };
@@ -199,7 +198,7 @@ public function testStaticAddPeriodicTimerWithNoDefaultLoopCallsAddPeriodicTimer
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $interval = 1.0;
         $callback = function () { };
@@ -226,7 +225,7 @@ public function testStaticCancelTimerWithNoDefaultLoopIsNoOp()
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $timer = $this->getMockBuilder('React\EventLoop\TimerInterface')->getMock();
         Loop::cancelTimer($timer);
@@ -250,7 +249,7 @@ public function testStaticFutureTickWithNoDefaultLoopCallsFutureTickOnNewLoopIns
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $listener = function () { };
         Loop::futureTick($listener);
@@ -279,7 +278,7 @@ public function testStaticAddSignalWithNoDefaultLoopCallsAddSignalOnNewLoopInsta
 
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $signal = 1;
         $listener = function () { };
@@ -309,7 +308,7 @@ public function testStaticRemoveSignalWithNoDefaultLoopIsNoOp()
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         $signal = 1;
         $listener = function () { };
@@ -332,7 +331,7 @@ public function testStaticRunWithNoDefaultLoopCallsRunsOnNewLoopInstance()
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         Loop::run();
 
@@ -353,7 +352,7 @@ public function testStaticStopCallWithNoDefaultLoopIsNoOp()
     {
         $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
         $ref->setAccessible(true);
-        $ref->setValue(null);
+        $ref->setValue(null, null);
 
         Loop::stop();
 
@@ -366,10 +365,8 @@ public function testStaticStopCallWithNoDefaultLoopIsNoOp()
      */
     public function unsetLoopFromLoopAccessor()
     {
-        $ref = new ReflectionClass('\React\EventLoop\Loop');
-        $prop = $ref->getProperty('instance');
-        $prop->setAccessible(true);
-        $prop->setValue(null);
-        $prop->setAccessible(false);
+        $ref = new \ReflectionProperty('React\EventLoop\Loop', 'instance');
+        $ref->setAccessible(true);
+        $ref->setValue(null, null);
     }
 }