Skip to content

Commit 6c60062

Browse files
author
Serhiy Shkolyarenko
committed
MAGETWO-51791: [Github] Sales Order extension_attributes #3967
fixed static tests
1 parent 492af16 commit 6c60062

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

app/code/Magento/GiftMessage/Test/Unit/Model/Plugin/OrderGetTest.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,10 @@
66

77
namespace Magento\GiftMessage\Test\Unit\Model\Plugin;
88

9-
use Magento\GiftMessage\Model\Plugin\OrderGet;
10-
119
class OrderGetTest extends \PHPUnit_Framework_TestCase
1210
{
1311
/**
14-
* @var OrderGet
12+
* @var \Magento\GiftMessage\Model\Plugin\OrderGet
1513
*/
1614
private $plugin;
1715

@@ -97,7 +95,7 @@ public function setUp()
9795
$this->orderRepositoryMock = $this->getMock(
9896
\Magento\Sales\Api\OrderRepositoryInterface::class
9997
);
100-
$this->plugin = new OrderGet(
98+
$this->plugin = new \Magento\GiftMessage\Model\Plugin\OrderGet(
10199
$this->giftMessageOrderRepositoryMock,
102100
$this->giftMessageOrderItemRepositoryMock,
103101
$this->orderExtensionFactoryMock,

app/code/Magento/GiftMessage/Test/Unit/Model/Plugin/OrderSaveTest.php

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@ class OrderSaveTest extends \PHPUnit_Framework_TestCase
2525
*/
2626
private $giftMessageOrderItemRepositoryMock;
2727

28-
/**
29-
* @var \PHPUnit_Framework_MockObject_MockObject
30-
*/
31-
private $orderItemExtensionFactoryMock;
32-
3328
/**
3429
* @var \PHPUnit_Framework_MockObject_MockObject
3530
*/
@@ -72,7 +67,11 @@ public function setUp()
7267
\Magento\Sales\Api\Data\OrderInterface::class
7368
);
7469
$this->orderExtensionMock = $this->getMock(
75-
\Magento\Sales\Api\Data\OrderExtension::class, ['getGiftMessage', 'setGiftMessage'], [], '', false
70+
\Magento\Sales\Api\Data\OrderExtension::class,
71+
['getGiftMessage', 'setGiftMessage'],
72+
[],
73+
'',
74+
false
7675
);
7776
$this->giftMessageMock = $this->getMock(
7877
\Magento\GiftMessage\Api\Data\MessageInterface::class
@@ -81,7 +80,11 @@ public function setUp()
8180
\Magento\Sales\Api\Data\OrderItemInterface::class
8281
);
8382
$this->orderItemExtensionMock = $this->getMock(
84-
\Magento\Sales\Api\Data\OrderItemExtension::class, ['setGiftMessage', 'getGiftMessage'], [], '', false
83+
\Magento\Sales\Api\Data\OrderItemExtension::class,
84+
['setGiftMessage', 'getGiftMessage'],
85+
[],
86+
'',
87+
false
8588
);
8689
$this->orderRepositoryMock = $this->getMock(
8790
\Magento\Sales\Api\OrderRepositoryInterface::class
@@ -137,7 +140,6 @@ public function testAfterSaveIfGiftMessagesNotExist()
137140
{
138141
// save Gift Message on order level
139142
$orderId = 1;
140-
$orderItemId = 2;
141143
$this->orderMock->expects($this->once())->method('getEntityId')->willReturn($orderId);
142144
$this->orderMock
143145
->expects($this->once())

0 commit comments

Comments
 (0)