From 2292ab09a8328dfcb6268fff982c9b4a628aa327 Mon Sep 17 00:00:00 2001 From: John Oleksowicz Date: Thu, 30 Mar 2023 11:03:44 -0500 Subject: [PATCH] Don't assume auto increment of 1 MAILPOET-5145 --- .../ViewInBrowser/ViewInBrowserControllerTest.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/mailpoet/tests/integration/Newsletter/ViewInBrowser/ViewInBrowserControllerTest.php b/mailpoet/tests/integration/Newsletter/ViewInBrowser/ViewInBrowserControllerTest.php index dc3eec1384..443ec98721 100644 --- a/mailpoet/tests/integration/Newsletter/ViewInBrowser/ViewInBrowserControllerTest.php +++ b/mailpoet/tests/integration/Newsletter/ViewInBrowser/ViewInBrowserControllerTest.php @@ -68,6 +68,7 @@ class ViewInBrowserControllerTest extends \MailPoetTest { $subscriber->setLastName('Last'); $this->subscribersRepository->persist($subscriber); $this->subscribersRepository->flush(); + $this->subscriber = $subscriber; // create task & queue $sendingTask = SendingTask::create(); @@ -148,11 +149,12 @@ class ViewInBrowserControllerTest extends \MailPoetTest { } public function testItSetsSubscriberToLoggedInWPUserWhenPreviewIsEnabled() { + $subscriberId = $this->subscriber->getId(); $viewInBrowserRenderer = $this->make(ViewInBrowserRenderer::class, [ - 'render' => Expected::once(function (bool $isPreview, NewsletterEntity $newsletter, SubscriberEntity $subscriber = null, SendingQueueEntity $queue = null) { + 'render' => Expected::once(function (bool $isPreview, NewsletterEntity $newsletter, SubscriberEntity $subscriber = null, SendingQueueEntity $queue = null) use ($subscriberId) { $this->assertNotNull($subscriber); // PHPStan expect($subscriber)->notNull(); - expect($subscriber->getId())->equals(1); + expect($subscriber->getId())->equals($subscriberId); }), ]);