Run tests on PHPUnit 9

reactphp · jsor · Jul 19, 2020 · Jul 17, 2020 · Jul 17, 2020 · Jul 17, 2020
commit 02c6c530e9f5c93c0f25d8655de261f5ac44c854
diff --git a/composer.json b/composer.json
@@ -7,7 +7,7 @@
         "php": ">=5.3.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "^7.0 || ^6.4 || ^5.7 || ^4.8.35"
+        "phpunit/phpunit": "^9.0 || ^5.7 || ^4.8.35"
     },
     "suggest": {
         "ext-event": "~1.0 for ExtEventLoop",

diff --git a/tests/AbstractLoopTest.php b/tests/AbstractLoopTest.php
@@ -16,7 +16,10 @@ abstract class AbstractLoopTest extends TestCase
 
     const PHP_DEFAULT_CHUNK_SIZE = 8192;
 
-    public function setUp()
+    /**
+     * @before
+     */
+    public function setUpLoop()
     {
         // It's a timeout, don't set it too low. Travis and other CI systems are slow.
         $this->tickTimeout = 0.02;
@@ -111,6 +114,10 @@ public function testAddWriteStreamTriggersWhenSocketConnectionSucceeds()
 
     public function testAddWriteStreamTriggersWhenSocketConnectionRefused()
     {
+        if (defined('HHVM_VERSION')) {
+            $this->markTestSkipped('Not supported on HHVM');
+        }
+
         // first verify the operating system actually refuses the connection and no firewall is in place
         // use higher timeout because Windows retires multiple times and has a noticeable delay
         // @link https://stackoverflow.com/questions/19440364/why-do-failed-attempts-of-socket-connect-take-1-sec-on-windows

diff --git a/tests/CallableStub.php b/tests/CallableStub.php
diff --git a/tests/ExtLibeventLoopTest.php b/tests/ExtLibeventLoopTest.php
@@ -21,7 +21,10 @@ public function createLoop()
         return new ExtLibeventLoop();
     }
 
-    public function tearDown()
+    /**
+     * @after
+     */
+    public function tearDownFile()
     {
         if (file_exists($this->fifoPath)) {
             unlink($this->fifoPath);

diff --git a/tests/StreamSelectLoopTest.php b/tests/StreamSelectLoopTest.php
@@ -7,7 +7,10 @@
 
 class StreamSelectLoopTest extends AbstractLoopTest
 {
-    protected function tearDown()
+    /**
+     * @after
+     */
+    protected function tearDownSignalHandlers()
     {
         parent::tearDown();
         if (strncmp($this->getName(false), 'testSignal', 10) === 0 && extension_loaded('pcntl')) {

diff --git a/tests/TestCase.php b/tests/TestCase.php
@@ -39,7 +39,13 @@ protected function expectCallableNever()
 
     protected function createCallableMock()
     {
-        return $this->getMockBuilder('React\Tests\EventLoop\CallableStub')->getMock();
+        if (method_exists('PHPUnit\Framework\MockObject\MockBuilder', 'addMethods')) {
+            // PHPUnit 9+
+            return $this->getMockBuilder('stdClass')->addMethods(array('__invoke'))->getMock();
+        } else {
+            // legacy PHPUnit 4 - PHPUnit 9
+            return $this->getMockBuilder('stdClass')->setMethods(array('__invoke'))->getMock();
+        }
     }
 
     protected function tickLoop(LoopInterface $loop)