Cavemanon/piratepoet
7
Fork 0
Code Pull Requests Actions Releases 16 Activity

Update tests [MAILPOET-2392]

Browse Source
This commit is contained in:
wxa
2019-10-09 13:56:42 +03:00
committed by Jack Kitterhing
parent 33c9b51ae6
commit cb3a7ed8bc
2 changed files with 22 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
tests/integration/Cron/Workers/InactiveSubscribersTest.php
View File

@ -82,12 +82,31 @@ class InactiveSubscribersTest extends \MailPoetTest {
], $this); ], $this);
$worker = new InactiveSubscribers($controller_mock, $this->settings); $worker = new InactiveSubscribers($controller_mock, $this->settings);
$worker->processTaskStrategy(ScheduledTask::createOrUpdate([])); $worker->processTaskStrategy(ScheduledTask::createOrUpdate(
['meta' => ['max_subscriber_id' => 2001 /* 3 iterations of BATCH_SIZE in markInactiveSubscribers */]]
));
expect($controller_mock->markInactiveSubscribers(5, 1000))->equals('ok'); expect($controller_mock->markInactiveSubscribers(5, 1000))->equals('ok');
expect($controller_mock->markActiveSubscribers(5, 1000))->equals('ok'); expect($controller_mock->markActiveSubscribers(5, 1000))->equals('ok');
} }
function testItCanStopDeactivationIfMarkInactiveSubscribersReturnsFalse() {
$this->settings->set('deactivate_subscriber_after_inactive_days', 5);
$controller_mock = Stub::make(InactiveSubscribersController::class, [
'markInactiveSubscribers' => Stub\Expected::once(false),
'markActiveSubscribers' => Stub\Expected::once(1),
'reactivateInactiveSubscribers' => Stub\Expected::never(),
], $this);
$task = ScheduledTask::createOrUpdate([]);
$worker = new InactiveSubscribers($controller_mock, $this->settings);
$worker->processTaskStrategy($task);
$meta = $task->getMeta();
expect(isset($meta['last_subscriber_id']))->equals(false);
}
function testThrowsAnExceptionWhenTimeIsOut() { function testThrowsAnExceptionWhenTimeIsOut() {
$this->settings->set('deactivate_subscriber_after_inactive_days', 5); $this->settings->set('deactivate_subscriber_after_inactive_days', 5);
$controller_mock = Stub::make(InactiveSubscribersController::class, [ $controller_mock = Stub::make(InactiveSubscribersController::class, [

4
tests/integration/Subscribers/InactiveSubscribersControllerTest.php
View File

@ -67,14 +67,14 @@ class InactiveSubscribersControllerTest extends \MailPoetTest {
$this->addSubcriberToTask($subscriber2, $task); $this->addSubcriberToTask($subscriber2, $task);
$batch_size = 1; $batch_size = 1;
$result = $this->controller->markInactiveSubscribers(self::INACTIVITY_DAYS_THRESHOLD, $batch_size); $result = $this->controller->markInactiveSubscribers(self::INACTIVITY_DAYS_THRESHOLD, $batch_size, $subscriber1->id);
expect($result)->equals(1); expect($result)->equals(1);
$subscriber1 = Subscriber::findOne($subscriber1->id); $subscriber1 = Subscriber::findOne($subscriber1->id);
$subscriber2 = Subscriber::findOne($subscriber2->id); $subscriber2 = Subscriber::findOne($subscriber2->id);
expect($subscriber1->status === Subscriber::STATUS_INACTIVE || $subscriber2->status === Subscriber::STATUS_INACTIVE)->true(); expect($subscriber1->status === Subscriber::STATUS_INACTIVE || $subscriber2->status === Subscriber::STATUS_INACTIVE)->true();
expect($subscriber1->status === Subscriber::STATUS_SUBSCRIBED || $subscriber2->status === Subscriber::STATUS_SUBSCRIBED)->true(); expect($subscriber1->status === Subscriber::STATUS_SUBSCRIBED || $subscriber2->status === Subscriber::STATUS_SUBSCRIBED)->true();
$result = $this->controller->markInactiveSubscribers(self::INACTIVITY_DAYS_THRESHOLD, $batch_size); $result = $this->controller->markInactiveSubscribers(self::INACTIVITY_DAYS_THRESHOLD, $batch_size, $subscriber2->id);
expect($result)->equals(1); expect($result)->equals(1);
$subscriber1 = Subscriber::findOne($subscriber1->id); $subscriber1 = Subscriber::findOne($subscriber1->id);
$subscriber2 = Subscriber::findOne($subscriber2->id); $subscriber2 = Subscriber::findOne($subscriber2->id);