Fix DaemonHttpRunnerTest
[MAILPOET-2538]
This commit is contained in:
committed by
Jack Kitterhing
parent
072aa54f37
commit
f4c2dda560
@@ -7,6 +7,7 @@ use MailPoet\Cron\CronHelper;
|
||||
use MailPoet\Cron\CronTrigger;
|
||||
use MailPoet\Cron\Daemon;
|
||||
use MailPoet\Cron\DaemonHttpRunner;
|
||||
use MailPoet\Cron\Triggers\WordPress;
|
||||
use MailPoet\Cron\Workers\SimpleWorker;
|
||||
use MailPoet\Cron\Workers\WorkersFactory;
|
||||
use MailPoet\DI\ContainerWrapper;
|
||||
@@ -86,7 +87,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
'callSelf' => null,
|
||||
]);
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$daemon_http_runner->__construct($daemon, $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon_http_runner->__construct($daemon, $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon_http_runner->run($data);
|
||||
$updated_daemon = $this->settings->get(CronHelper::DAEMON_SETTING);
|
||||
expect($updated_daemon['last_error'][0]['message'])->equals('Message');
|
||||
@@ -107,7 +108,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
'token' => 123,
|
||||
];
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$daemon_http_runner->__construct($daemon, $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon_http_runner->__construct($daemon, $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon_http_runner->run($data);
|
||||
}
|
||||
|
||||
@@ -129,7 +130,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
'token' => 123,
|
||||
];
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$daemon->__construct(new Daemon($workers_factory_mock, $this->cron_helper), $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon->__construct(new Daemon($workers_factory_mock, $this->cron_helper), $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon->run($data);
|
||||
}
|
||||
|
||||
@@ -153,7 +154,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
'token' => 123,
|
||||
];
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$daemon->__construct(new Daemon($workers_factory_mock, $this->cron_helper), $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon->__construct(new Daemon($workers_factory_mock, $this->cron_helper), $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon->run($data);
|
||||
$data_after_run = $this->settings->get(CronHelper::DAEMON_SETTING);
|
||||
expect($data_after_run['token'])->equals(567);
|
||||
@@ -170,7 +171,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
'status' => CronHelper::DAEMON_STATUS_INACTIVE,
|
||||
];
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$daemon->__construct($this->makeEmpty(Daemon::class), $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon->__construct($this->makeEmpty(Daemon::class), $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon->run($data);
|
||||
}
|
||||
|
||||
@@ -189,7 +190,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$this->settings->set(CronTrigger::SETTING_NAME . '.method', 'WordPress');
|
||||
WPFunctions::get()->addFilter('mailpoet_cron_enable_self_deactivation', '__return_true');
|
||||
$daemon_http_runner->__construct($daemon, $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon_http_runner->__construct($daemon, $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon_http_runner->run($data);
|
||||
WPFunctions::get()->removeAllFilters('mailpoet_cron_enable_self_deactivation');
|
||||
}
|
||||
@@ -204,7 +205,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
'token' => 123,
|
||||
];
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$daemon_http_runner->__construct(new Daemon($this->createWorkersFactoryMock(), $this->cron_helper), $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon_http_runner->__construct(new Daemon($this->createWorkersFactoryMock(), $this->cron_helper), $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon_http_runner->run($data);
|
||||
$updated_daemon = $this->settings->get(CronHelper::DAEMON_SETTING);
|
||||
expect($updated_daemon['token'])->equals($daemon_http_runner->token);
|
||||
@@ -234,7 +235,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
];
|
||||
$now = time();
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$daemon_http_runner->__construct($daemon, $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon_http_runner->__construct($daemon, $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon_http_runner->run($data);
|
||||
$updated_daemon = $this->settings->get(CronHelper::DAEMON_SETTING);
|
||||
expect($updated_daemon['run_started_at'])->greaterOrEquals($now);
|
||||
@@ -256,7 +257,7 @@ class DaemonHttpRunnerTest extends \MailPoetTest {
|
||||
'token' => 123,
|
||||
];
|
||||
$this->settings->set(CronHelper::DAEMON_SETTING, $data);
|
||||
$daemon->__construct(new Daemon($this->createWorkersFactoryMock(), $this->cron_helper), $this->cron_helper, SettingsController::getInstance());
|
||||
$daemon->__construct(new Daemon($this->createWorkersFactoryMock(), $this->cron_helper), $this->cron_helper, SettingsController::getInstance(), $this->di_container->get(WordPress::class));
|
||||
$daemon->run($data);
|
||||
expect(ignore_user_abort())->equals(1);
|
||||
}
|
||||
|
Reference in New Issue
Block a user