Skip to content

Commit e4861d3

Browse files
author
Stanislav Idolov
committed
Code style fixes
1 parent bb9a652 commit e4861d3

File tree

29 files changed

+192
-30
lines changed

29 files changed

+192
-30
lines changed

‎app/code/Magento/Braintree/Test/Unit/Model/InstantPurchase/PayPal/TokenFormatterTest.php

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
*/
66
declare(strict_types=1);
77

8-
namespace Magento\Braintree\Test\Unit\Model\InstantPurchase\Paypal;
8+
namespace Magento\Braintree\Test\Unit\Model\InstantPurchase\PayPal;
99

1010
use Magento\Braintree\Model\InstantPurchase\CreditCard\TokenFormatter as PaypalTokenFormatter;
1111
use Magento\Vault\Api\Data\PaymentTokenInterface;
@@ -31,6 +31,9 @@ class TokenFormatterTest extends \PHPUnit\Framework\TestCase
3131
'expirationDate' => '07-07-2025'
3232
];
3333

34+
/**
35+
* Test setup
36+
*/
3437
protected function setUp()
3538
{
3639
$this->paymentTokenMock = $this->getMockBuilder(PaymentTokenInterface::class)
@@ -39,6 +42,9 @@ protected function setUp()
3942
$this->paypalTokenFormatter = new PaypalTokenFormatter();
4043
}
4144

45+
/**
46+
* testFormatPaymentTokenWithKnownCardType
47+
*/
4248
public function testFormatPaymentTokenWithKnownCardType()
4349
{
4450
$this->tokenDetails['type'] = key(PaypalTokenFormatter::$baseCardTypes);
@@ -59,6 +65,9 @@ public function testFormatPaymentTokenWithKnownCardType()
5965
self::assertEquals($formattedString, $this->paypalTokenFormatter->formatPaymentToken($this->paymentTokenMock));
6066
}
6167

68+
/**
69+
* testFormatPaymentTokenWithUnknownCardType
70+
*/
6271
public function testFormatPaymentTokenWithUnknownCardType()
6372
{
6473
$this->paymentTokenMock->expects($this->once())
@@ -78,6 +87,9 @@ public function testFormatPaymentTokenWithUnknownCardType()
7887
self::assertEquals($formattedString, $this->paypalTokenFormatter->formatPaymentToken($this->paymentTokenMock));
7988
}
8089

90+
/**
91+
* testFormatPaymentTokenWithWrongData
92+
*/
8193
public function testFormatPaymentTokenWithWrongData()
8294
{
8395
unset($this->tokenDetails['type']);

‎app/code/Magento/ConfigurableProduct/Model/Quote/Item/CartItemProcessor.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public function __construct(
5959
}
6060

6161
/**
62-
* {@inheritdoc}
62+
* @inheritdoc
6363
*/
6464
public function convertToBuyRequest(CartItemInterface $cartItem)
6565
{
@@ -78,7 +78,7 @@ public function convertToBuyRequest(CartItemInterface $cartItem)
7878
}
7979

8080
/**
81-
* {@inheritdoc}
81+
* @inheritdoc
8282
*/
8383
public function processOptions(CartItemInterface $cartItem)
8484
{

‎app/code/Magento/Customer/Block/Adminhtml/Edit/Tab/Cart.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,7 @@ public function getGridParentHtml()
220220
}
221221

222222
/**
223-
* {@inheritdoc}
223+
* @inheritdoc
224224
*/
225225
public function getRowUrl($row)
226226
{

‎app/code/Magento/Customer/Block/Widget/Company.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,12 @@ class Company extends AbstractWidget
4040
protected $options;
4141

4242
/**
43-
* @param Context $context
44-
* @param AddressHelper $addressHelper
43+
* @param Context $context
44+
* @param AddressHelper $addressHelper
4545
* @param CustomerMetadataInterface $customerMetadata
46-
* @param Options $options
47-
* @param AddressMetadataInterface $addressMetadata
48-
* @param array $data
46+
* @param Options $options
47+
* @param AddressMetadataInterface $addressMetadata
48+
* @param array $data
4949
*/
5050
public function __construct(
5151
Context $context,
@@ -95,7 +95,7 @@ public function showCompany()
9595
}
9696

9797
/**
98-
* {@inheritdoc}
98+
* @inheritdoc
9999
*/
100100
protected function _getAttribute($attributeCode)
101101
{

‎app/code/Magento/Customer/Block/Widget/Fax.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,12 @@ class Fax extends AbstractWidget
4040
protected $options;
4141

4242
/**
43-
* @param Context $context
44-
* @param AddressHelper $addressHelper
43+
* @param Context $context
44+
* @param AddressHelper $addressHelper
4545
* @param CustomerMetadataInterface $customerMetadata
46-
* @param Options $options
47-
* @param AddressMetadataInterface $addressMetadata
48-
* @param array $data
46+
* @param Options $options
47+
* @param AddressMetadataInterface $addressMetadata
48+
* @param array $data
4949
*/
5050
public function __construct(
5151
Context $context,
@@ -95,7 +95,7 @@ public function showFax()
9595
}
9696

9797
/**
98-
* {@inheritdoc}
98+
* @inheritdoc
9999
*/
100100
protected function _getAttribute($attributeCode)
101101
{

‎app/code/Magento/Customer/Block/Widget/Name.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ public function getContainerClassName()
201201
}
202202

203203
/**
204-
* {@inheritdoc}
204+
* @inheritdoc
205205
*/
206206
protected function _getAttribute($attributeCode)
207207
{

‎app/code/Magento/Customer/Block/Widget/Telephone.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,12 @@ class Telephone extends AbstractWidget
4040
protected $options;
4141

4242
/**
43-
* @param Context $context
44-
* @param AddressHelper $addressHelper
43+
* @param Context $context
44+
* @param AddressHelper $addressHelper
4545
* @param CustomerMetadataInterface $customerMetadata
46-
* @param Options $options
47-
* @param AddressMetadataInterface $addressMetadata
48-
* @param array $data
46+
* @param Options $options
47+
* @param AddressMetadataInterface $addressMetadata
48+
* @param array $data
4949
*/
5050
public function __construct(
5151
Context $context,
@@ -95,7 +95,7 @@ public function showTelephone()
9595
}
9696

9797
/**
98-
* {@inheritdoc}
98+
* @inheritdoc
9999
*/
100100
protected function _getAttribute($attributeCode)
101101
{

‎app/code/Magento/Sales/Controller/Adminhtml/Order/Status/Save.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ public function execute()
6868

6969
/**
7070
* @param \Magento\Backend\Model\View\Result\Redirect $resultRedirect
71-
* @param $isNew
71+
* @param bool $isNew
7272
* @return \Magento\Backend\Model\View\Result\Redirect
7373
*/
7474
private function getRedirect(\Magento\Backend\Model\View\Result\Redirect $resultRedirect, $isNew)

‎app/code/Magento/Sales/Test/Unit/Controller/Adminhtml/Creditmemo/AbstractCreditmemo/EmailTest.php

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,9 @@ class EmailTest extends \PHPUnit\Framework\TestCase
7373
*/
7474
protected $resultRedirectMock;
7575

76+
/**
77+
* Test setup
78+
*/
7679
protected function setUp()
7780
{
7881
$objectManagerHelper = new ObjectManagerHelper($this);
@@ -131,6 +134,9 @@ protected function setUp()
131134
);
132135
}
133136

137+
/**
138+
* testEmail
139+
*/
134140
public function testEmail()
135141
{
136142
$cmId = 10000031;
@@ -160,6 +166,9 @@ public function testEmail()
160166
$this->assertEquals($this->response, $this->creditmemoEmail->getResponse());
161167
}
162168

169+
/**
170+
* testEmailNoCreditmemoId
171+
*/
163172
public function testEmailNoCreditmemoId()
164173
{
165174
$this->request->expects($this->once())

‎app/code/Magento/Sales/Test/Unit/Controller/Adminhtml/Invoice/AbstractInvoice/EmailTest.php

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,9 @@ class EmailTest extends \PHPUnit\Framework\TestCase
8989
*/
9090
protected $invoiceManagement;
9191

92+
/**
93+
* Test setup
94+
*/
9295
protected function setUp()
9396
{
9497
$objectManagerHelper = new ObjectManagerHelper($this);
@@ -153,6 +156,9 @@ protected function setUp()
153156
);
154157
}
155158

159+
/**
160+
* testEmail
161+
*/
156162
public function testEmail()
157163
{
158164
$invoiceId = 10000031;
@@ -209,6 +215,9 @@ public function testEmail()
209215
$this->assertInstanceOf(\Magento\Backend\Model\View\Result\Redirect::class, $this->invoiceEmail->execute());
210216
}
211217

218+
/**
219+
* testEmailNoInvoiceId
220+
*/
212221
public function testEmailNoInvoiceId()
213222
{
214223
$this->request->expects($this->once())
@@ -226,6 +235,9 @@ public function testEmailNoInvoiceId()
226235
$this->assertInstanceOf(\Magento\Backend\Model\View\Result\Forward::class, $this->invoiceEmail->execute());
227236
}
228237

238+
/**
239+
* testEmailNoInvoice
240+
*/
229241
public function testEmailNoInvoice()
230242
{
231243
$invoiceId = 10000031;

0 commit comments

Comments
 (0)