From 0118e9ccabc635e9a7c0dd92b3b4588d315b9b82 Mon Sep 17 00:00:00 2001
From: marie <15118505+marie@users.noreply.github.com>
Date: Tue, 1 Oct 2019 16:57:44 +0500
Subject: [PATCH] remove dispatch mode in SignalRegistry

---
 .../SignalRegistry/SignalRegistry.php         | 11 ++-----
 .../SignalRegistryInterface.php               |  3 +-
 .../Tests/SignalRegistryTest.php              | 32 ++-----------------
 3 files changed, 6 insertions(+), 40 deletions(-)

diff --git a/src/Symfony/Component/SignalRegistry/SignalRegistry.php b/src/Symfony/Component/SignalRegistry/SignalRegistry.php
index c90523f3d8422..b2815e7d90626 100644
--- a/src/Symfony/Component/SignalRegistry/SignalRegistry.php
+++ b/src/Symfony/Component/SignalRegistry/SignalRegistry.php
@@ -6,11 +6,9 @@ final class SignalRegistry implements SignalRegistryInterface
 {
     private $signals = [];
 
-    public function __construct(bool $asynchronousMode = true)
+    public function __construct()
     {
-        if ($asynchronousMode) {
-            pcntl_async_signals(true);
-        }
+        pcntl_async_signals(true);
     }
 
     public function register(int $signal, callable $callback): void
@@ -19,11 +17,6 @@ public function register(int $signal, callable $callback): void
         pcntl_signal($signal, [$this, 'handler']);
     }
 
-    public function dispatch(): bool
-    {
-        return pcntl_signal_dispatch();
-    }
-
     public function handler(int $signal): void
     {
         foreach($this->signals[$signal] as $callback) {
diff --git a/src/Symfony/Component/SignalRegistry/SignalRegistryInterface.php b/src/Symfony/Component/SignalRegistry/SignalRegistryInterface.php
index 1fa189da4975f..795720b3131df 100644
--- a/src/Symfony/Component/SignalRegistry/SignalRegistryInterface.php
+++ b/src/Symfony/Component/SignalRegistry/SignalRegistryInterface.php
@@ -4,8 +4,7 @@
 
 interface SignalRegistryInterface
 {
-    public function __construct(bool $asynchronousMode = true);
+    public function __construct();
     public function register(int $signal, callable $callback): void;
-    public function dispatch(): bool;
     public function handler(int $signal): void;
 }
diff --git a/src/Symfony/Component/SignalRegistry/Tests/SignalRegistryTest.php b/src/Symfony/Component/SignalRegistry/Tests/SignalRegistryTest.php
index 57a57078efcf0..fea22140e7b5a 100644
--- a/src/Symfony/Component/SignalRegistry/Tests/SignalRegistryTest.php
+++ b/src/Symfony/Component/SignalRegistry/Tests/SignalRegistryTest.php
@@ -3,6 +3,9 @@
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\SignalRegistry\SignalRegistry;
 
+/**
+ * @requires extension pcntl
+ */
 class SignalRegistryTest  extends TestCase
 {
     public function tearDown(): void
@@ -70,33 +73,4 @@ public function testTwoSignals_AsyncMode_signalsAreHandled()
 
         $this->assertTrue($isHandled2);
     }
-
-    public function testOneCallbackForASignal_DispatchMode_signalIsNotHandled()
-    {
-        $signalRegistry = new SignalRegistry(false);
-
-        $isHandled = false;
-        $signalRegistry->register(SIGUSR1, function() use (&$isHandled) {
-            $isHandled = true;
-        });
-
-        posix_kill(posix_getpid(), SIGUSR1);
-
-        $this->assertFalse($isHandled);
-    }
-
-    public function testOneCallbackForASignal_DispatchMode_signalIsHandled()
-    {
-        $signalRegistry = new SignalRegistry(false);
-
-        $isHandled = false;
-        $signalRegistry->register(SIGUSR1, function() use (&$isHandled) {
-            $isHandled = true;
-        });
-
-        posix_kill(posix_getpid(), SIGUSR1);
-        $signalRegistry->dispatch();
-
-        $this->assertTrue($isHandled);
-    }
 }