Skip to content

Commit 90c3540

Browse files
#26622 - Update test to check combinations of parentItemId and parentItem
1 parent 199afbf commit 90c3540

File tree

1 file changed

+15
-2
lines changed

1 file changed

+15
-2
lines changed

app/code/Magento/SalesRule/Test/Unit/Model/ValidatorTest.php

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -346,11 +346,14 @@ public function testInitTotalsCanApplyDiscount()
346346
'getDiscountCalculationPrice',
347347
'getBaseDiscountCalculationPrice',
348348
'getCalculationPrice',
349-
'getParentItemId'
349+
'getParentItemId',
350+
'getParentItem'
350351
]
351352
);
352353
$item2 = clone $item1;
353-
$items = [$item1, $item2];
354+
$item3 = clone $item1;
355+
$item4 = clone $item1;
356+
$items = [$item1, $item2, $item3, $item4];
354357

355358
$rule->expects($this->any())
356359
->method('getSimpleAction')
@@ -368,11 +371,21 @@ public function testInitTotalsCanApplyDiscount()
368371
$validator->expects($this->at(1))->method('isValid')->with($item2)->willReturn(true);
369372

370373
$item1->expects($this->any())->method('getParentItemId')->willReturn(false);
374+
$item1->expects($this->any())->method('getParentItem')->willReturn(false);
371375
$item1->expects($this->never())->method('getDiscountCalculationPrice');
372376
$item1->expects($this->never())->method('getBaseDiscountCalculationPrice');
373377
$item2->expects($this->any())->method('getParentItemId')->willReturn(false);
378+
$item2->expects($this->any())->method('getParentItem')->willReturn(false);
374379
$item2->expects($this->any())->method('getDiscountCalculationPrice')->willReturn(50);
375380
$item2->expects($this->once())->method('getBaseDiscountCalculationPrice')->willReturn(50);
381+
$item3->expects($this->any())->method('getParentItemId')->willReturn(false);
382+
$item3->expects($this->any())->method('getParentItem')->willReturn(true);
383+
$item3->expects($this->never())->method('getDiscountCalculationPrice');
384+
$item3->expects($this->never())->method('getBaseDiscountCalculationPrice');
385+
$item4->expects($this->any())->method('getParentItemId')->willReturn(true);
386+
$item4->expects($this->any())->method('getParentItem')->willReturn(false);
387+
$item4->expects($this->never())->method('getDiscountCalculationPrice');
388+
$item4->expects($this->never())->method('getBaseDiscountCalculationPrice');
376389
$this->utility->expects($this->once())->method('getItemQty')->willReturn(1);
377390
$this->utility->expects($this->any())->method('canProcessRule')->willReturn(true);
378391

0 commit comments

Comments
 (0)