Skip to content

Commit fe7611f

Browse files
mrtuvnmrtuvn
committed
Fix static tests for b2b
1 parent b4debc7 commit fe7611f

File tree

1 file changed

+79
-34
lines changed

1 file changed

+79
-34
lines changed

‎app/code/Magento/Customer/Test/Unit/Model/EmailNotificationTest.php

Lines changed: 79 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6+
67
declare(strict_types=1);
78

89
namespace Magento\Customer\Test\Unit\Model;
@@ -17,9 +18,10 @@
1718
use Magento\Store\Model\Store;
1819
use Magento\Customer\Model\Data\CustomerSecure;
1920
use Magento\Store\Model\Website;
21+
use Magento\Framework\TestFramework\Unit\Helper\ObjectManager as ObjectManagerHelper;
2022

2123
/**
22-
* Class EmailNotificationTest
24+
* Unit test for \Magento\Customer\Model\EmailNotification
2325
*
2426
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
2527
*/
@@ -80,7 +82,7 @@ class EmailNotificationTest extends TestCase
8082
private $storeMock;
8183

8284
/**
83-
* @var \Magento\Customer\Model\EmailNotification
85+
* @var EmailNotification
8486
*/
8587
private $model;
8688

@@ -124,7 +126,7 @@ public function setUp(): void
124126
->disableOriginalConstructor()
125127
->getMockForAbstractClass();
126128

127-
$objectManager = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
129+
$objectManager = new ObjectManagerHelper($this);
128130

129131
$this->model = $objectManager->getObject(
130132
EmailNotification::class,
@@ -142,7 +144,7 @@ public function setUp(): void
142144

143145
/**
144146
* Test email notify when credentials changed
145-
*
147+
*
146148
* @param int $testNumber
147149
* @param string $oldEmail
148150
* @param string $newEmail
@@ -164,7 +166,7 @@ public function testEmailNotifyWhenCredentialsChanged($testNumber, $oldEmail, $n
164166
$expects = $this->once();
165167
break;
166168
case 2:
167-
$xmlPathTemplate = \Magento\Customer\Model\EmailNotification::XML_PATH_CHANGE_EMAIL_TEMPLATE;
169+
$xmlPathTemplate = EmailNotification::XML_PATH_CHANGE_EMAIL_TEMPLATE;
168170
$expects = $this->exactly(2);
169171
break;
170172
case 3:
@@ -249,20 +251,32 @@ public function testEmailNotifyWhenCredentialsChanged($testNumber, $oldEmail, $n
249251
$this->scopeConfigMock->expects($this->any())
250252
->method('getValue')
251253
->withConsecutive(
252-
[$xmlPathTemplate, ScopeInterface::SCOPE_STORE, self::STUB_CUSTOMER_STORE_ID],
253254
[
254-
\Magento\Customer\Model\EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY,
255+
$xmlPathTemplate,
256+
ScopeInterface::SCOPE_STORE,
257+
self::STUB_CUSTOMER_STORE_ID
258+
],
259+
[
260+
EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY,
261+
ScopeInterface::SCOPE_STORE,
262+
self::STUB_CUSTOMER_STORE_ID
263+
],
264+
[
265+
$xmlPathTemplate,
255266
ScopeInterface::SCOPE_STORE,
256267
self::STUB_CUSTOMER_STORE_ID
257268
],
258-
[$xmlPathTemplate, ScopeInterface::SCOPE_STORE, self::STUB_CUSTOMER_STORE_ID],
259269
[
260-
\Magento\Customer\Model\EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY,
270+
EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY,
261271
ScopeInterface::SCOPE_STORE,
262272
self::STUB_CUSTOMER_STORE_ID
263273
]
264-
)
265-
->willReturnOnConsecutiveCalls(self::STUB_EMAIL_IDENTIFIER, self::STUB_SENDER, self::STUB_EMAIL_IDENTIFIER, self::STUB_SENDER);
274+
)->willReturnOnConsecutiveCalls(
275+
self::STUB_EMAIL_IDENTIFIER,
276+
self::STUB_SENDER,
277+
self::STUB_EMAIL_IDENTIFIER,
278+
self::STUB_SENDER
279+
);
266280

267281
$this->transportBuilderMock->expects(clone $expects)
268282
->method('setTemplateIdentifier')
@@ -300,7 +314,7 @@ public function testEmailNotifyWhenCredentialsChanged($testNumber, $oldEmail, $n
300314

301315
/**
302316
* Provides Emails Data Provider
303-
*
317+
*
304318
* @param void
305319
* @return array
306320
*/
@@ -329,6 +343,8 @@ public function sendNotificationEmailsDataProvider(): array
329343
}
330344

331345
/**
346+
* Test Password Reminder Email Notify
347+
*
332348
* @SuppressWarnings(PHPMD.ExcessiveMethodLength)
333349
*/
334350
public function testPasswordReminder(): void
@@ -366,7 +382,7 @@ public function testPasswordReminder(): void
366382
->method('getStore')
367383
->willReturn($this->storeMock);
368384

369-
$websiteMock = $this->createPartialMock(\Magento\Store\Model\Website::class, ['getStoreIds']);
385+
$websiteMock = $this->createPartialMock(Website::class, ['getStoreIds']);
370386
$websiteMock->expects($this->any())
371387
->method('getStoreIds')
372388
->willReturn($storeIds);
@@ -402,12 +418,18 @@ public function testPasswordReminder(): void
402418

403419
$this->scopeConfigMock->expects($this->at(0))
404420
->method('getValue')
405-
->with(EmailNotification::XML_PATH_REMIND_EMAIL_TEMPLATE, ScopeInterface::SCOPE_STORE, self::STUB_CUSTOMER_STORE_ID)
406-
->willReturn(self::STUB_EMAIL_IDENTIFIER);
421+
->with(
422+
EmailNotification::XML_PATH_REMIND_EMAIL_TEMPLATE,
423+
ScopeInterface::SCOPE_STORE,
424+
self::STUB_CUSTOMER_STORE_ID
425+
)->willReturn(self::STUB_EMAIL_IDENTIFIER);
407426
$this->scopeConfigMock->expects($this->at(1))
408427
->method('getValue')
409-
->with(EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY, ScopeInterface::SCOPE_STORE, self::STUB_CUSTOMER_STORE_ID)
410-
->willReturn(self::STUB_SENDER);
428+
->with(
429+
EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY,
430+
ScopeInterface::SCOPE_STORE,
431+
self::STUB_CUSTOMER_STORE_ID
432+
)->willReturn(self::STUB_SENDER);
411433

412434
$this->mockDefaultTransportBuilder(
413435
self::STUB_EMAIL_IDENTIFIER,
@@ -423,7 +445,7 @@ public function testPasswordReminder(): void
423445

424446
/**
425447
* Test password reminder customer withouer store id info
426-
*
448+
*
427449
* @SuppressWarnings(PHPMD.ExcessiveMethodLength)
428450
*/
429451
public function testPasswordReminderCustomerWithoutStoreId(): void
@@ -462,7 +484,7 @@ public function testPasswordReminderCustomerWithoutStoreId(): void
462484
->method('getStore')
463485
->with($defaultStoreId)
464486
->willReturn($this->storeMock);
465-
$websiteMock = $this->createPartialMock(\Magento\Store\Model\Website::class, ['getStoreIds']);
487+
$websiteMock = $this->createPartialMock(Website::class, ['getStoreIds']);
466488
$websiteMock->expects($this->any())
467489
->method('getStoreIds')
468490
->willReturn($storeIds);
@@ -493,12 +515,18 @@ public function testPasswordReminderCustomerWithoutStoreId(): void
493515
->willReturnSelf();
494516
$this->scopeConfigMock->expects($this->at(0))
495517
->method('getValue')
496-
->with(EmailNotification::XML_PATH_REMIND_EMAIL_TEMPLATE, ScopeInterface::SCOPE_STORE, $defaultStoreId)
497-
->willReturn(self::STUB_EMAIL_IDENTIFIER);
518+
->with(
519+
EmailNotification::XML_PATH_REMIND_EMAIL_TEMPLATE,
520+
ScopeInterface::SCOPE_STORE,
521+
$defaultStoreId
522+
)->willReturn(self::STUB_EMAIL_IDENTIFIER);
498523
$this->scopeConfigMock->expects($this->at(1))
499524
->method('getValue')
500-
->with(EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY, ScopeInterface::SCOPE_STORE, $defaultStoreId)
501-
->willReturn(self::STUB_SENDER);
525+
->with(
526+
EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY,
527+
ScopeInterface::SCOPE_STORE,
528+
$defaultStoreId
529+
)->willReturn(self::STUB_SENDER);
502530
$this->mockDefaultTransportBuilder(
503531
self::STUB_EMAIL_IDENTIFIER,
504532
$defaultStoreId,
@@ -512,7 +540,7 @@ public function testPasswordReminderCustomerWithoutStoreId(): void
512540

513541
/**
514542
* Test email notify for password reset confirm
515-
*
543+
*
516544
* @SuppressWarnings(PHPMD.ExcessiveMethodLength)
517545
*/
518546
public function testPasswordResetConfirmation(): void
@@ -572,12 +600,18 @@ public function testPasswordResetConfirmation(): void
572600

573601
$this->scopeConfigMock->expects($this->at(0))
574602
->method('getValue')
575-
->with(EmailNotification::XML_PATH_FORGOT_EMAIL_TEMPLATE, ScopeInterface::SCOPE_STORE, self::STUB_CUSTOMER_STORE_ID)
576-
->willReturn(self::STUB_EMAIL_IDENTIFIER);
603+
->with(
604+
EmailNotification::XML_PATH_FORGOT_EMAIL_TEMPLATE,
605+
ScopeInterface::SCOPE_STORE,
606+
self::STUB_CUSTOMER_STORE_ID
607+
)->willReturn(self::STUB_EMAIL_IDENTIFIER);
577608
$this->scopeConfigMock->expects($this->at(1))
578609
->method('getValue')
579-
->with(EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY, ScopeInterface::SCOPE_STORE, self::STUB_CUSTOMER_STORE_ID)
580-
->willReturn(self::STUB_SENDER);
610+
->with(
611+
EmailNotification::XML_PATH_FORGOT_EMAIL_IDENTITY,
612+
ScopeInterface::SCOPE_STORE,
613+
self::STUB_CUSTOMER_STORE_ID
614+
)->willReturn(self::STUB_SENDER);
581615

582616
$this->mockDefaultTransportBuilder(
583617
self::STUB_EMAIL_IDENTIFIER,
@@ -593,7 +627,7 @@ public function testPasswordResetConfirmation(): void
593627

594628
/**
595629
* Test email notify with new account
596-
*
630+
*
597631
* @param void
598632
* @SuppressWarnings(PHPMD.ExcessiveMethodLength)
599633
*/
@@ -655,12 +689,18 @@ public function testNewAccount(): void
655689

656690
$this->scopeConfigMock->expects($this->at(0))
657691
->method('getValue')
658-
->with(EmailNotification::XML_PATH_REGISTER_EMAIL_TEMPLATE, ScopeInterface::SCOPE_STORE, self::STUB_CUSTOMER_STORE_ID)
659-
->willReturn(self::STUB_EMAIL_IDENTIFIER);
692+
->with(
693+
EmailNotification::XML_PATH_REGISTER_EMAIL_TEMPLATE,
694+
ScopeInterface::SCOPE_STORE,
695+
self::STUB_CUSTOMER_STORE_ID
696+
)->willReturn(self::STUB_EMAIL_IDENTIFIER);
660697
$this->scopeConfigMock->expects($this->at(1))
661698
->method('getValue')
662-
->with(EmailNotification::XML_PATH_REGISTER_EMAIL_IDENTITY, ScopeInterface::SCOPE_STORE, self::STUB_CUSTOMER_STORE_ID)
663-
->willReturn(self::STUB_SENDER);
699+
->with(
700+
EmailNotification::XML_PATH_REGISTER_EMAIL_IDENTITY,
701+
ScopeInterface::SCOPE_STORE,
702+
self::STUB_CUSTOMER_STORE_ID
703+
)->willReturn(self::STUB_SENDER);
664704

665705
$this->mockDefaultTransportBuilder(
666706
self::STUB_EMAIL_IDENTIFIER,
@@ -671,7 +711,12 @@ public function testNewAccount(): void
671711
['customer' => $this->customerSecureMock, 'back_url' => '', 'store' => $this->storeMock]
672712
);
673713

674-
$this->model->newAccount($customer, EmailNotification::NEW_ACCOUNT_EMAIL_REGISTERED, '', self::STUB_CUSTOMER_STORE_ID);
714+
$this->model->newAccount(
715+
$customer,
716+
EmailNotification::NEW_ACCOUNT_EMAIL_REGISTERED,
717+
'',
718+
self::STUB_CUSTOMER_STORE_ID
719+
);
675720
}
676721

677722
/**

0 commit comments

Comments
 (0)