Skip to content

Commit a037edf

Browse files
#26622 - Fix return values of mock object functions
1 parent 75b03e0 commit a037edf

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

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

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -370,20 +370,20 @@ public function testInitTotalsCanApplyDiscount()
370370
$validator->expects($this->at(0))->method('isValid')->with($item1)->willReturn(false);
371371
$validator->expects($this->at(1))->method('isValid')->with($item2)->willReturn(true);
372372

373-
$item1->expects($this->any())->method('getParentItemId')->willReturn(false);
374-
$item1->expects($this->any())->method('getParentItem')->willReturn(false);
373+
$item1->expects($this->any())->method('getParentItemId')->willReturn(null);
374+
$item1->expects($this->any())->method('getParentItem')->willReturn(null);
375375
$item1->expects($this->never())->method('getDiscountCalculationPrice');
376376
$item1->expects($this->never())->method('getBaseDiscountCalculationPrice');
377-
$item2->expects($this->any())->method('getParentItemId')->willReturn(false);
378-
$item2->expects($this->any())->method('getParentItem')->willReturn(false);
377+
$item2->expects($this->any())->method('getParentItemId')->willReturn(null);
378+
$item2->expects($this->any())->method('getParentItem')->willReturn(null);
379379
$item2->expects($this->any())->method('getDiscountCalculationPrice')->willReturn(50);
380380
$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);
381+
$item3->expects($this->any())->method('getParentItemId')->willReturn(null);
382+
$item3->expects($this->any())->method('getParentItem')->willReturn($item1);
383383
$item3->expects($this->never())->method('getDiscountCalculationPrice');
384384
$item3->expects($this->never())->method('getBaseDiscountCalculationPrice');
385-
$item4->expects($this->any())->method('getParentItemId')->willReturn(true);
386-
$item4->expects($this->any())->method('getParentItem')->willReturn(false);
385+
$item4->expects($this->any())->method('getParentItemId')->willReturn(12345);
386+
$item4->expects($this->any())->method('getParentItem')->willReturn(null);
387387
$item4->expects($this->never())->method('getDiscountCalculationPrice');
388388
$item4->expects($this->never())->method('getBaseDiscountCalculationPrice');
389389
$this->utility->expects($this->once())->method('getItemQty')->willReturn(1);

0 commit comments

Comments
 (0)