Make HttpClientTestCase compatible with PHPUnit8

symfony · Aug 8, 2019 · c8c717c · c8c717c
1 parent fda49e6
commit c8c717c
Showing 1 changed file with 27 additions and 6 deletions.
diff --git a/src/Symfony/Contracts/HttpClient/Test/HttpClientTestCase.php b/src/Symfony/Contracts/HttpClient/Test/HttpClientTestCase.php
@@ -21,16 +21,12 @@
  * A reference test suite for HttpClientInterface implementations.
  *
  * @experimental in 1.1
+ * @internal
  */
-abstract class HttpClientTestCase extends TestCase
+trait HttpClientTestCaseTrait
 {
     private static $server;
 
-    public static function setUpBeforeClass()
-    {
-        TestHttpServer::start();
-    }
-
     abstract protected function getHttpClient(string $testCase): HttpClientInterface;
 
     public function testGetRequest()
@@ -788,3 +784,28 @@ public function testGzipBroken()
         $response->getContent();
     }
 }
+
+if (method_exists(\ReflectionMethod::class, 'hasReturnType') && (new \ReflectionMethod(TestCase::class, 'setUpBeforeClass'))->hasReturnType()) {
+    abstract class HttpClientTestCase extends TestCase
+    {
+        use HttpClientTestCaseTrait;
+
+        public static function setUpBeforeClass(): void
+        {
+            TestHttpServer::start();
+        }
+    }
+} else {
+    abstract class HttpClientTestCase extends TestCase
+    {
+        use HttpClientTestCaseTrait;
+
+        /**
+         * @return void
+         */
+        public static function setUpBeforeClass()
+        {
+            TestHttpServer::start();
+        }
+    }
+}