diff --git a/lib/Cron/Workers/Bounce.php b/lib/Cron/Workers/Bounce.php index f1c4b4c428..56145e6d92 100644 --- a/lib/Cron/Workers/Bounce.php +++ b/lib/Cron/Workers/Bounce.php @@ -27,8 +27,15 @@ class Bounce extends SimpleWorker { /** @var SettingsController */ private $settings; - public function __construct(SettingsController $settings) { + /** @var Bridge */ + private $bridge; + + public function __construct( + SettingsController $settings, + Bridge $bridge + ) { $this->settings = $settings; + $this->bridge = $bridge; parent::__construct(); } @@ -39,7 +46,7 @@ class Bounce extends SimpleWorker { } public function checkProcessingRequirements() { - return Bridge::isMPSendingServiceEnabled(); + return $this->bridge->isMailpoetSendingServiceEnabled(); } public function prepareTaskStrategy(ScheduledTask $task, $timer) { diff --git a/tests/integration/Cron/Workers/BounceTest.php b/tests/integration/Cron/Workers/BounceTest.php index 03b0af30d0..be47336309 100644 --- a/tests/integration/Cron/Workers/BounceTest.php +++ b/tests/integration/Cron/Workers/BounceTest.php @@ -8,6 +8,7 @@ use MailPoet\Mailer\Mailer; use MailPoet\Models\ScheduledTask; use MailPoet\Models\ScheduledTaskSubscriber; use MailPoet\Models\Subscriber; +use MailPoet\Services\Bridge; use MailPoet\Services\Bridge\API; use MailPoet\Settings\SettingsController; use MailPoet\Settings\SettingsRepository; @@ -36,7 +37,10 @@ class BounceTest extends \MailPoetTest { ]); } - $this->worker = new Bounce($this->diContainer->get(SettingsController::class)); + $this->worker = new Bounce( + $this->diContainer->get(SettingsController::class), + $this->diContainer->get(Bridge::class) + ); $this->worker->api = new MockAPI(); } @@ -47,7 +51,7 @@ class BounceTest extends \MailPoetTest { public function testItCanInitializeBridgeAPI() { $this->setMailPoetSendingMethod(); - $worker = new Bounce($this->diContainer->get(SettingsController::class)); + $worker = new Bounce($this->diContainer->get(SettingsController::class), $this->diContainer->get(Bridge::class)); $worker->init(); expect($worker->api instanceof API)->true(); }