Don't assume auto increment of 1
MAILPOET-5145
This commit is contained in:
committed by
John Oleksowicz
parent
4c3e7e62d4
commit
2292ab09a8
@@ -68,6 +68,7 @@ class ViewInBrowserControllerTest extends \MailPoetTest {
|
|||||||
$subscriber->setLastName('Last');
|
$subscriber->setLastName('Last');
|
||||||
$this->subscribersRepository->persist($subscriber);
|
$this->subscribersRepository->persist($subscriber);
|
||||||
$this->subscribersRepository->flush();
|
$this->subscribersRepository->flush();
|
||||||
|
$this->subscriber = $subscriber;
|
||||||
|
|
||||||
// create task & queue
|
// create task & queue
|
||||||
$sendingTask = SendingTask::create();
|
$sendingTask = SendingTask::create();
|
||||||
@@ -148,11 +149,12 @@ class ViewInBrowserControllerTest extends \MailPoetTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
public function testItSetsSubscriberToLoggedInWPUserWhenPreviewIsEnabled() {
|
public function testItSetsSubscriberToLoggedInWPUserWhenPreviewIsEnabled() {
|
||||||
|
$subscriberId = $this->subscriber->getId();
|
||||||
$viewInBrowserRenderer = $this->make(ViewInBrowserRenderer::class, [
|
$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
|
$this->assertNotNull($subscriber); // PHPStan
|
||||||
expect($subscriber)->notNull();
|
expect($subscriber)->notNull();
|
||||||
expect($subscriber->getId())->equals(1);
|
expect($subscriber->getId())->equals($subscriberId);
|
||||||
}),
|
}),
|
||||||
]);
|
]);
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user