Run tests on PHPUnit 9

Author: SimonFrings

composer.json

@@ -18,7 +18,7 @@
     },
     "require-dev": {
         "clue/block-react": "^1.1",
-        "phpunit/phpunit": "^7.0 || ^5.0 || ^4.8"
+        "phpunit/phpunit": "^9.3 || ^5.7 || ^4.8"
     },
     "autoload": {
         "psr-4": { "Clue\\React\\Zenity\\": "src/" }

tests/FunctionalLauncherTest.php

@@ -13,7 +13,10 @@ class FunctionalLauncherTest extends TestCase
     private $dialog;
     private $launcher;
 
-    public function setUp()
+    /**
+     * @before
+     */
+    public function setUpLauncher()
     {
         $this->loop = Factory::create();
 

tests/TestCase.php

@@ -44,7 +44,13 @@ protected function expectCallableNever()
      */
     protected function createCallableMock()
     {
-        return $this->getMockBuilder('stdClass')->setMethods(array('__invoke'))->getMock();
+        if (method_exists('PHPUnit\Framework\MockObject\MockBuilder', 'addMethods')) {
+            // PHPUnit 9+
+            return $this->getMockBuilder('stdClass')->addMethods(array('__invoke'))->getMock();
+        } else {
+            // legacy PHPUnit 4 - PHPUnit 8
+            return $this->getMockBuilder('stdClass')->setMethods(array('__invoke'))->getMock();
+        }
     }
 
     protected function expectPromiseResolve($promise)

tests/Zen/BaseZenTest.php

@@ -11,7 +11,10 @@ abstract class BaseZenTest extends TestCase
     protected $process;
     protected $stdin = '';
 
-    public function setUp()
+    /**
+     * @before
+     */
+    public function setUpMocks()
     {
         $inbuffer =& $this->stdin;
         $this->instream = $this->getMockBuilder('React\Stream\WritableStreamInterface')->getMock();

tests/Zen/FunctionalBaseZenTest.php

@@ -8,7 +8,10 @@
 
 class FunctionalBaseZenTest extends TestCase
 {
-    public function setUp()
+    /**
+     * @before
+     */
+    public function setUpLoop()
     {
         $this->loop = Factory::create();
     }