Skip to content

Commit 89061b2

Browse files
committed
MC-15295: Fix unit tests
1 parent 837d0db commit 89061b2

File tree

1 file changed

+39
-2
lines changed

1 file changed

+39
-2
lines changed

lib/internal/Magento/Framework/Mail/Test/Unit/Template/TransportBuilderTest.php

Lines changed: 39 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@
88
namespace Magento\Framework\Mail\Test\Unit\Template;
99

1010
use Magento\Framework\App\TemplateTypesInterface;
11+
use Magento\Framework\Mail\EmailMessageInterface;
12+
use Magento\Framework\Mail\EmailMessageInterfaceFactory;
13+
use Magento\Framework\Mail\MimePartInterface;
14+
use Magento\Framework\Mail\MimePartInterfaceFactory;
15+
use PHPUnit\Framework\MockObject\MockObject;
1116

1217
/**
1318
* Class TransportBuilderTest
@@ -61,6 +66,16 @@ class TransportBuilderTest extends \PHPUnit\Framework\TestCase
6166
*/
6267
private $emailMessageMock;
6368

69+
/**
70+
* @var MimePartInterfaceFactory|MockObject
71+
*/
72+
private $mimePartFactoryMock;
73+
74+
/**
75+
* @var EmailMessageInterfaceFactory|MockObject
76+
*/
77+
private $emailMessageInterfaceFactoryMock;
78+
6479
/**
6580
* @return void
6681
*/
@@ -81,6 +96,10 @@ protected function setUp()
8196
->disableOriginalConstructor()
8297
->setMethods(['create'])
8398
->getMockForAbstractClass();
99+
100+
$this->emailMessageInterfaceFactoryMock = $this->createMock(EmailMessageInterfaceFactory::class);
101+
$this->mimePartFactoryMock = $this->createMock(MimePartInterfaceFactory::class);
102+
84103
$this->builder = $objectManagerHelper->getObject(
85104
$this->builderClassName,
86105
[
@@ -89,7 +108,9 @@ protected function setUp()
89108
'objectManager' => $this->objectManagerMock,
90109
'senderResolver' => $this->senderResolverMock,
91110
'mailTransportFactory' => $this->mailTransportFactoryMock,
92-
'messageFactory' => $this->messageFactoryMock
111+
'messageFactory' => $this->messageFactoryMock,
112+
'emailMessageInterfaceFactory' => $this->emailMessageInterfaceFactoryMock,
113+
'mimePartInterfaceFactory' => $this->mimePartFactoryMock,
93114
]
94115
);
95116
}
@@ -108,6 +129,20 @@ public function testGetTransport($templateType, $bodyText, $templateNamespace)
108129
$vars = ['reason' => 'Reason', 'customer' => 'Customer'];
109130
$options = ['area' => 'frontend', 'store' => 1];
110131

132+
/** @var MimePartInterface|MockObject $mimePartMock */
133+
$mimePartMock = $this->createMock(MimePartInterface::class);
134+
135+
$this->mimePartFactoryMock->expects($this->any())
136+
->method('create')
137+
->willReturn($mimePartMock);
138+
139+
/** @var EmailMessageInterface|MockObject $emailMessage */
140+
$emailMessage = $this->createMock(EmailMessageInterface::class);
141+
142+
$this->emailMessageInterfaceFactoryMock->expects($this->any())
143+
->method('create')
144+
->willReturn($emailMessage);
145+
111146
$template = $this->createMock(\Magento\Framework\Mail\TemplateInterface::class);
112147
$template->expects($this->once())->method('setVars')->with($this->equalTo($vars))->willReturnSelf();
113148
$template->expects($this->once())->method('setOptions')->with($this->equalTo($options))->willReturnSelf();
@@ -127,7 +162,9 @@ public function testGetTransport($templateType, $bodyText, $templateNamespace)
127162
->willReturn($transport);
128163

129164
$this->builder->setTemplateIdentifier('identifier')->setTemplateVars($vars)->setTemplateOptions($options);
130-
$this->assertInstanceOf(\Magento\Framework\Mail\TransportInterface::class, $this->builder->getTransport());
165+
166+
$result = $this->builder->getTransport();
167+
$this->assertInstanceOf(\Magento\Framework\Mail\TransportInterface::class, $result);
131168
}
132169

133170
/**

0 commit comments

Comments
 (0)