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

Fix and improve SchedulerTest::testItDeletesScheduledAutomationEmailWhenUserDoesNotExist()

Browse Source 
Having no segment and no task subscriber caused the test pass even without removing the subscriber.

[MAILPOET-4375]
This commit is contained in:
Jan Jakes
2023-10-27 09:56:25 +02:00
committed by Aschepikov
parent dd5a11a34f
commit d7cb84d20f
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
mailpoet/tests/integration/Cron/Workers/SchedulerTest.php
View File

@@ -866,7 +866,11 @@ class SchedulerTest extends \MailPoetTest {
public function testItDeletesScheduledAutomationEmailWhenUserDoesNotExist() { public function testItDeletesScheduledAutomationEmailWhenUserDoesNotExist() {
$newsletter = $this->_createNewsletter(NewsletterEntity::TYPE_AUTOMATION, NewsletterEntity::STATUS_ACTIVE); $newsletter = $this->_createNewsletter(NewsletterEntity::TYPE_AUTOMATION, NewsletterEntity::STATUS_ACTIVE);
$subscriber = $this->_createSubscriber(); $subscriber = $this->_createSubscriber();
$this->createTaskWithQueue($newsletter); $task = $this->createTaskWithQueue($newsletter);
$this->createTaskSubscriber($task, $subscriber);
$task = $this->scheduledTasksRepository->findOneByNewsletter($newsletter);
verify($task)->notNull();
// remove subscriber, but not scheduled task subscriber // remove subscriber, but not scheduled task subscriber
$this->entityManager->getConnection()->delete( $this->entityManager->getConnection()->delete(