Skip to content

Commit 663c21b

Browse files
MC-33110: Paypal Express Checkout - payment loading infinitely in checkout
1 parent 29e876c commit 663c21b

File tree

1 file changed

+19
-5
lines changed

1 file changed

+19
-5
lines changed

app/code/Magento/Paypal/Test/Unit/Model/SmartButtonConfigTest.php

Lines changed: 19 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,37 +12,45 @@
1212
use Magento\Paypal\Model\SmartButtonConfig;
1313
use Magento\Framework\Locale\ResolverInterface;
1414
use Magento\Paypal\Model\ConfigFactory;
15+
use PHPUnit\Framework\MockObject\MockObject;
16+
use PHPUnit\Framework\TestCase;
1517

16-
class SmartButtonConfigTest extends \PHPUnit\Framework\TestCase
18+
/**
19+
* Test for smart button config
20+
*/
21+
class SmartButtonConfigTest extends TestCase
1722
{
1823
/**
1924
* @var \Magento\Paypal\Model\SmartButtonConfig
2025
*/
2126
private $model;
2227

2328
/**
24-
* @var \PHPUnit_Framework_MockObject_MockObject
29+
* @var MockObject
2530
*/
2631
private $localeResolverMock;
2732

2833
/**
29-
* @var \PHPUnit_Framework_MockObject_MockObject
34+
* @var MockObject
3035
*/
3136
private $configMock;
3237

38+
/**
39+
* @inheritdoc
40+
*/
3341
protected function setUp()
3442
{
3543
$this->localeResolverMock = $this->getMockForAbstractClass(ResolverInterface::class);
3644
$this->configMock = $this->getMockBuilder(Config::class)
3745
->disableOriginalConstructor()
3846
->getMock();
3947

40-
/** @var ScopeConfigInterface|\PHPUnit_Framework_MockObject_MockObject $scopeConfigMock */
48+
/** @var ScopeConfigInterface|MockObject $scopeConfigMock */
4149
$scopeConfigMock = $this->getMockForAbstractClass(ScopeConfigInterface::class);
4250
$scopeConfigMock->method('isSetFlag')
4351
->willReturn(true);
4452

45-
/** @var \PHPUnit_Framework_MockObject_MockObject $configFactoryMock */
53+
/** @var MockObject $configFactoryMock */
4654
$configFactoryMock = $this->getMockBuilder(ConfigFactory::class)
4755
->disableOriginalConstructor()
4856
->setMethods(['create'])
@@ -122,6 +130,8 @@ public function testGetConfig(
122130
}
123131

124132
/**
133+
* Get config data provider
134+
*
125135
* @return array
126136
*/
127137
public function getConfigDataProvider()
@@ -130,6 +140,8 @@ public function getConfigDataProvider()
130140
}
131141

132142
/**
143+
* Get default styles
144+
*
133145
* @return array
134146
*/
135147
private function getDefaultStyles()
@@ -138,6 +150,8 @@ private function getDefaultStyles()
138150
}
139151

140152
/**
153+
* Get allowed fundings
154+
*
141155
* @return array
142156
*/
143157
private function getAllowedFundings()

0 commit comments

Comments
 (0)