Skip to content

Commit 4c41ad2

Browse files
author
Olexandr Lysenko
committed
MAGETWO-30913: Error when trying to apply coupon code in backend reorder after unchecking the discount apply checkbox
1 parent e5e1d3e commit 4c41ad2

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

dev/tests/unit/testsuite/Magento/SalesRule/Model/Quote/DiscountTest.php

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,11 +89,14 @@ public function testCollectItemNoDiscount()
8989
{
9090
$itemNoDiscount = $this->getMockBuilder('Magento\Sales\Model\Quote\Item')
9191
->disableOriginalConstructor()
92-
->setMethods(['getNoDiscount', '__wakeup'])
92+
->setMethods(['getNoDiscount', 'setIsApplyDiscount', '__wakeup'])
9393
->getMock();
9494
$itemNoDiscount->expects($this->once())
9595
->method('getNoDiscount')
9696
->willReturn(true);
97+
$itemNoDiscount->expects($this->once())
98+
->method('setIsApplyDiscount')
99+
->willReturn(false);
97100

98101
$this->validatorMock->expects($this->any())
99102
->method('sortItemsByPriority')
@@ -201,6 +204,7 @@ public function testCollectItemHasChildren($childItemData, $parentData, $expecte
201204
'getHasChildren',
202205
'isChildrenCalculated',
203206
'getChildren',
207+
'setIsApplyDiscount',
204208
'__wakeup',
205209
]
206210
)
@@ -214,6 +218,9 @@ public function testCollectItemHasChildren($childItemData, $parentData, $expecte
214218
$itemWithChildren->expects($this->once())
215219
->method('getHasChildren')
216220
->willReturn(true);
221+
$itemWithChildren->expects($this->once())
222+
->method('setIsApplyDiscount')
223+
->willReturn(true);
217224
$itemWithChildren->expects($this->once())
218225
->method('isChildrenCalculated')
219226
->willReturn(true);
@@ -332,13 +339,17 @@ public function testCollectItemHasNoChildren()
332339
'getHasChildren',
333340
'isChildrenCalculated',
334341
'getChildren',
342+
'setIsApplyDiscount',
335343
'__wakeup',
336344
]
337345
)
338346
->getMock();
339347
$itemWithChildren->expects($this->once())
340348
->method('getNoDiscount')
341349
->willReturn(false);
350+
$itemWithChildren->expects($this->once())
351+
->method('setIsApplyDiscount')
352+
->willReturn(true);
342353
$itemWithChildren->expects($this->once())
343354
->method('getParentItem')
344355
->willReturn(false);

0 commit comments

Comments
 (0)