Resume exit channel by coordinator after running phpunit. (#4227)

Author: limingxinleo

co-phpunit

@@ -9,10 +9,21 @@ declare (strict_types=1);
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
+if (!version_compare(PHP_VERSION, PHP_VERSION, '=')) {
+    fwrite(STDERR, sprintf('%s declares an invalid value for PHP_VERSION.' . PHP_EOL . 'This breaks fundamental functionality such as version_compare().' . PHP_EOL . 'Please use a different PHP interpreter.' . PHP_EOL, PHP_BINARY));
+    die(1);
+}
 if (version_compare('7.3.0', PHP_VERSION, '>')) {
     fwrite(STDERR, sprintf('This version of PHPUnit requires PHP >= 7.3.' . PHP_EOL . 'You are using PHP %s (%s).' . PHP_EOL, PHP_VERSION, PHP_BINARY));
     die(1);
 }
+foreach (['dom', 'json', 'libxml', 'mbstring', 'tokenizer', 'xml', 'xmlwriter'] as $extension) {
+    if (extension_loaded($extension)) {
+        continue;
+    }
+    fwrite(STDERR, sprintf('PHPUnit requires the "%s" extension.' . PHP_EOL, $extension));
+    die(1);
+}
 if (!ini_get('date.timezone')) {
     ini_set('date.timezone', 'UTC');
 }
@@ -40,5 +51,6 @@ Swoole\Coroutine::set(['hook_flags' => SWOOLE_HOOK_ALL, 'exit_condition' => func
 Swoole\Coroutine\run(function () use(&$code) {
     $code = PHPUnit\TextUI\Command::main(false);
     Swoole\Timer::clearAll();
+    Hyperf\Coordinator\CoordinatorManager::until(Hyperf\Coordinator\Constants::WORKER_EXIT)->resume();
 });
 die($code);

src/Client.php

@@ -34,24 +34,21 @@
 
 class Client extends Server
 {
-    /**
-     * @var PackerInterface
-     */
-    protected $packer;
+    protected PackerInterface $packer;
 
-    /**
-     * @var float
-     */
-    protected $waitTimeout = 10.0;
+    protected float $waitTimeout = 10.0;
 
-    /**
-     * @var string
-     */
-    protected $baseUri = 'http://127.0.0.1/';
+    protected string $baseUri = 'http://127.0.0.1/';
 
     public function __construct(ContainerInterface $container, PackerInterface $packer = null, $server = 'http')
     {
-        parent::__construct($container, $container->get(HttpDispatcher::class), $container->get(ExceptionHandlerDispatcher::class), $container->get(ResponseEmitter::class));
+        parent::__construct(
+            $container,
+            $container->get(HttpDispatcher::class),
+            $container->get(ExceptionHandlerDispatcher::class),
+            $container->get(ResponseEmitter::class)
+        );
+
         $this->packer = $packer ?? new JsonPacker();
 
         $this->initCoreMiddleware($server);

src/HttpClient.php

@@ -21,24 +21,12 @@
 
 class HttpClient
 {
-    /**
-     * @var ContainerInterface
-     */
-    protected $container;
-
-    /**
-     * @var Client
-     */
-    protected $client;
-
-    /**
-     * @var PackerInterface
-     */
-    protected $packer;
-
-    public function __construct(ContainerInterface $container, PackerInterface $packer = null, $baseUri = 'http://127.0.0.1:9501')
+    protected Client $client;
+
+    protected PackerInterface $packer;
+
+    public function __construct(protected ContainerInterface $container, PackerInterface $packer = null, $baseUri = 'http://127.0.0.1:9501')
     {
-        $this->container = $container;
         $this->packer = $packer ?? new JsonPacker();
         $handler = null;
         if (Coroutine::inCoroutine()) {