Run tests on PHPUnit 9

Author: SimonFrings

composer.json

@@ -32,9 +32,9 @@
         "react/stream": "^1.0 || ^0.7.6"
     },
     "require-dev": {
-        "phpunit/phpunit": "^7.0 || ^6.4 || ^5.7 || ^4.8.35",
+        "phpunit/phpunit": "^9.3 || ^5.7 || ^4.8.35",
         "react/socket": "^1.0",
-        "sebastian/environment": "^3.0 || ^2.0 || ^1.0"
+        "sebastian/environment": "^5.0 || ^3.0 || ^2.0 || ^1.0"
     },
     "autoload": {
         "psr-4": { "React\\ChildProcess\\": "src" }

tests/AbstractProcessTest.php

@@ -84,17 +84,15 @@ public function testStartWithCustomPipesWillAssignPipes()
         $this->assertInstanceOf('React\Stream\WritableStreamInterface', $process->pipes[3]);
     }
 
-    /**
-     * @expectedException RuntimeException
-     * @expectedExceptionMessage No such file or directory
-     */
     public function testStartWithInvalidFileDescriptorPathWillThrow()
     {
         $fds = array(
             4 => array('file', '/dev/does-not-exist', 'r')
         );
 
         $process = new Process('exit 0', null, null, $fds);
+
+        $this->setExpectedException('RuntimeException', 'No such file or directory');
         $process->start($this->createLoop());
     }
 
@@ -130,7 +128,7 @@ public function testStartWithExcessiveNumberOfFileDescriptorsWillThrow()
             // clear dummy files handles to make some room again (avoid fatal errors for autoloader)
             $fds = array();
 
-            $this->assertContains('Too many open files', $e->getMessage());
+            $this->assertContainsString('Too many open files', $e->getMessage());
         }
     }
 
@@ -631,9 +629,6 @@ public function testStartInvalidProcess()
         $this->assertNotEmpty($output);
     }
 
-    /**
-     * @expectedException RuntimeException
-     */
     public function testStartAlreadyRunningProcess()
     {
         if (DIRECTORY_SEPARATOR === '\\') {
@@ -645,6 +640,8 @@ public function testStartAlreadyRunningProcess()
         //var_dump($process);
 
         $process->start($this->createLoop());
+
+        $this->setExpectedException('RuntimeException');
         $process->start($this->createLoop());
     }
 
@@ -873,4 +870,32 @@ private function getPhpBinary()
 
         return $runtime->getBinary();
     }
+
+    public function setExpectedException($exception, $exceptionMessage = '', $exceptionCode = null)
+    {
+        if (method_exists($this, 'expectException')) {
+            // PHPUnit 5.2+
+            $this->expectException($exception);
+            if ($exceptionMessage !== '') {
+                $this->expectExceptionMessage($exceptionMessage);
+            }
+            if ($exceptionCode !== null) {
+                $this->expectExceptionCode($exceptionCode);
+            }
+        } else {
+            // legacy PHPUnit 4 - PHPUnit 5.1
+            parent::setExpectedException($exception, $exceptionMessage, $exceptionCode);
+        }
+    }
+
+    public function assertContainsString($needle, $haystack)
+    {
+        if (method_exists($this, 'assertStringContainsString')) {
+            // PHPUnit 7.5+
+            $this->assertStringContainsString($needle, $haystack);
+        } else {
+            // legacy PHPUnit 4 - PHPUnit 7.5
+            $this->assertContains($needle, $haystack);
+        }
+    }
 }