Skip to content

Commit 05c53dc

Browse files
committed
Merge remote-tracking branch 'origin/BUG#AC-2583' into Hammer_Quality_Backlog_GraphQl_13042022
2 parents c18de12 + f026c3a commit 05c53dc

File tree

3 files changed

+11
-6
lines changed

3 files changed

+11
-6
lines changed

app/code/Magento/Quote/Model/Quote/Address/Total/Shipping.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public function collect(
7373
$address->setFreeMethodWeight($data['freeMethodWeight']);
7474

7575
$isFreeShipping = $this->freeShipping->isFreeShipping($quote, $shippingAssignment->getItems());
76-
$address->setFreeShipping($isFreeShipping);
76+
$address->setFreeShipping((int)$isFreeShipping);
7777
// recalculate weights
7878
$data = $this->getAssignmentWeightData($address, $shippingAssignment->getItems());
7979
$address->setItemQty($data['addressQty']);

app/code/Magento/Quote/Test/Unit/Model/Quote/Address/Total/ShippingTest.php

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -209,11 +209,16 @@ public function testCollect(): void
209209
$this->shippingAssignment->expects($this->atLeastOnce())
210210
->method('getItems')
211211
->willReturn([$this->cartItem]);
212-
$this->freeShipping->method('isFreeShipping')
212+
$isFreeShipping = true;
213+
$this->freeShipping
214+
->expects($this->once())
215+
->method('isFreeShipping')
213216
->with($this->quote, [$this->cartItem])
214-
->willReturn(true);
215-
$this->address->method('setFreeShipping')
216-
->with(true);
217+
->willReturn($isFreeShipping);
218+
$this->address
219+
->expects($this->once())
220+
->method('setFreeShipping')
221+
->with((int)$isFreeShipping);
217222
$this->total->expects($this->atLeastOnce())
218223
->method('setTotalAmount');
219224
$this->total->expects($this->atLeastOnce())

dev/tests/integration/testsuite/Magento/SalesRule/_files/cart_rule_free_shipping_by_cart.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464
'is_rss' => 1,
6565
'use_auto_generation' => 0,
6666
'uses_per_coupon' => 0,
67-
'simple_free_shipping' => 2,
67+
'simple_free_shipping' => 1,
6868

6969
'website_ids' => [
7070
$objectManager->get(StoreManagerInterface::class)->getWebsite()->getId()

0 commit comments

Comments
 (0)