From d7cb84d20fb08bcf833bbf4fcb0b9c88c15db0cf Mon Sep 17 00:00:00 2001 From: Jan Jakes Date: Fri, 27 Oct 2023 09:56:25 +0200 Subject: [PATCH] Fix and improve SchedulerTest::testItDeletesScheduledAutomationEmailWhenUserDoesNotExist() Having no segment and no task subscriber caused the test pass even without removing the subscriber. [MAILPOET-4375] --- mailpoet/tests/integration/Cron/Workers/SchedulerTest.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/mailpoet/tests/integration/Cron/Workers/SchedulerTest.php b/mailpoet/tests/integration/Cron/Workers/SchedulerTest.php index 2240fafa19..331b03afa0 100644 --- a/mailpoet/tests/integration/Cron/Workers/SchedulerTest.php +++ b/mailpoet/tests/integration/Cron/Workers/SchedulerTest.php @@ -866,7 +866,11 @@ class SchedulerTest extends \MailPoetTest { public function testItDeletesScheduledAutomationEmailWhenUserDoesNotExist() { $newsletter = $this->_createNewsletter(NewsletterEntity::TYPE_AUTOMATION, NewsletterEntity::STATUS_ACTIVE); $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 $this->entityManager->getConnection()->delete(