From 60f1234b71d88da3dfd79a8dc30b1fb16ce89ef8 Mon Sep 17 00:00:00 2001 From: Jan Jakes Date: Fri, 9 Feb 2024 16:39:23 +0100 Subject: [PATCH] Check newsletter stats "sentAt" agains correct date (from subsriber task) [MAILPOET-5887] --- .../App/Migration_20240207_105912_App_Test.php | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/mailpoet/tests/integration/Migrations/App/Migration_20240207_105912_App_Test.php b/mailpoet/tests/integration/Migrations/App/Migration_20240207_105912_App_Test.php index 57f0c89e9e..4426910241 100644 --- a/mailpoet/tests/integration/Migrations/App/Migration_20240207_105912_App_Test.php +++ b/mailpoet/tests/integration/Migrations/App/Migration_20240207_105912_App_Test.php @@ -249,11 +249,16 @@ class Migration_20240207_105912_App_Test extends \MailPoetTest { $processedSubscribers = $task->getSubscribersByProcessed(ScheduledTaskSubscriberEntity::STATUS_PROCESSED); $this->assertCount(4, $stats); // 3 ok + 1 failed $this->assertCount(4, $processedSubscribers); // 3 ok + 1 failed + for ($i = 0; $i < 4; $i++) { $this->assertSame($newsletter, $stats[$i]->getNewsletter()); $this->assertSame($task->getSendingQueue(), $stats[$i]->getQueue()); - $this->assertEquals($task->getUpdatedAt(), $stats[$i]->getSentAt()); - $this->assertContains($stats[$i]->getSubscriber(), $processedSubscribers); + + $subscriberIndex = array_search($stats[$i]->getSubscriber(), $processedSubscribers, true); + $subscriber = $processedSubscribers[$subscriberIndex] ?? null; + $this->assertNotNull($subscriber); + $this->assertSame($subscriber, $stats[$i]->getSubscriber()); + $this->assertEquals($subscriber->getUpdatedAt(), $stats[$i]->getSentAt()); } }