Skip to content

Commit 12f7e9c

Browse files
author
Stanislav Idolov
committed
Fixed static test failures
1 parent 87c8f5a commit 12f7e9c

File tree

9 files changed

+7
-9
lines changed

9 files changed

+7
-9
lines changed

app/code/Magento/Sales/Test/Unit/Model/Order/ItemTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ class ItemTest extends \PHPUnit_Framework_TestCase
2020
* @var \Magento\Sales\Model\Order\Item
2121
*/
2222
protected $model;
23+
2324
/**
2425
* @var \Magento\Framework\TestFramework\Unit\Helper\ObjectManager
2526
*/
@@ -42,14 +43,12 @@ protected function setUp()
4243
$this->model = $this->objectManager->getObject('Magento\Sales\Model\Order\Item', $arguments);
4344
}
4445

45-
4646
public function testSetParentItemNull()
4747
{
4848
$this->assertEquals($this->model, $this->model->setParentItem(null));
4949
$this->assertNull($this->model->getParentItem());
5050
}
5151

52-
5352
public function testSetParentItem()
5453
{
5554
$item = $this->objectManager->getObject('Magento\Sales\Model\Order\Item', []);

app/code/Magento/Sales/Test/Unit/Model/Order/Payment/Transaction/BuilderTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,6 @@ protected function expectsLinkWithParentTransaction(
194194
}
195195
}
196196

197-
198197
/**
199198
* @param int $orderId
200199
* @param int $paymentId

app/code/Magento/SalesRule/Test/Unit/Observer/SalesOrderAfterPlaceObserverTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ class SalesOrderAfterPlaceObserverTest extends \PHPUnit_Framework_TestCase
2727
*/
2828
protected $ruleCustomerFactory;
2929

30-
3130
/**
3231
* @var \Magento\SalesRule\Model\ResourceModel\Coupon\Usage|\PHPUnit_Framework_MockObject_MockObject
3332
*/

app/code/Magento/Tax/Test/Unit/Model/Quote/ToOrderConverterTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,7 @@ public function testAfterConvertNullExtensionAttribute(
181181
* Data provider for testAfterConvert and testAfterConvertNullExtensionAttribute
182182
*
183183
* @return array
184+
* @SuppressWarnings(PHPMD.ExcessiveMethodLength)
184185
*/
185186
public function afterConvertDataProvider()
186187
{

app/code/Magento/UrlRewrite/Test/Unit/Helper/UrlRewriteTest.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,9 @@ class UrlRewriteTest extends \PHPUnit_Framework_TestCase
1616

1717
protected function setUp()
1818
{
19-
$this->_helper = (new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this))->getObject('Magento\UrlRewrite\Helper\UrlRewrite');
19+
$this->_helper = (new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this))->getObject(
20+
'Magento\UrlRewrite\Helper\UrlRewrite'
21+
);
2022
}
2123

2224
/**

lib/internal/Magento/Framework/HTTP/Test/Unit/PhpEnvironment/RequestTest.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,6 @@ public function testGetAliasWhenAliasAreEmpty()
217217
$this->assertNull($this->model->getAlias(''));
218218
}
219219

220-
221220
public function testGetCookie()
222221
{
223222
$key = "cookieName";
@@ -231,7 +230,6 @@ public function testGetCookie()
231230
$this->getModel()->getCookie($key, $default);
232231
}
233232

234-
235233
public function testGetCookieDefault()
236234
{
237235
$key = "cookieName";

lib/internal/Magento/Framework/Image/Test/Unit/Adapter/AbstractTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ protected function setUp()
5454
)->will(
5555
$this->returnValue($this->directoryWriteMock)
5656
);
57-
$this->loggerMock = $this->getMockBuilder( 'Psr\Log\LoggerInterface')->getMock();
57+
$this->loggerMock = $this->getMockBuilder('Psr\Log\LoggerInterface')->getMock();
5858

5959
$this->_model = $this->getMockForAbstractClass(
6060
'Magento\Framework\Image\Adapter\AbstractAdapter',

lib/internal/Magento/Framework/Test/Unit/Message/PhraseFactoryTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,5 +55,4 @@ public function dataProvider()
5555
],
5656
];
5757
}
58-
5958
}

lib/internal/Magento/Framework/View/Test/Unit/Asset/MergeServiceTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
/**
2020
* Class MergeServiceTest
21+
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
2122
*/
2223
class MergeServiceTest extends \PHPUnit_Framework_TestCase
2324
{

0 commit comments

Comments
 (0)