Skip to content

Commit 3184a3b

Browse files
committed
AC-1479: Incorrect qty to ship after few creditmemos
Fix static and Integration test
1 parent 71d5db0 commit 3184a3b

File tree

2 files changed

+6
-6
lines changed
  • app/code/Magento/Sales/Test/Unit/Model/Order
  • dev/tests/integration/testsuite/Magento/Sales/Model/Order

2 files changed

+6
-6
lines changed

app/code/Magento/Sales/Test/Unit/Model/Order/ItemTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@
1818

1919
/**
2020
* Unit test for order item class.
21-
*
22-
* @suppressWarnings(PHPMD.TooManyPublicMethods)
21+
*
2322
* @suppressWarnings(PHPMD.CouplingBetweenObjects)
23+
* @suppressWarnings(PHPMD.ExcessivePublicCount)
2424
*/
2525
class ItemTest extends TestCase
2626
{

dev/tests/integration/testsuite/Magento/Sales/Model/Order/ItemTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -89,19 +89,19 @@ public static function getProductOptionsDataProvider()
8989
DataFixture(InvoiceFixture::class, ['order_id' => '$order.id$'], 'invoice'),
9090
DataFixture(
9191
ShipmentFixture::class,
92-
['order_id' => '$order.id$', 'items' => [['product_id' => '$product.id$', 'qty' => 1]]]
92+
['order_id' => '$order.id$', 'items' => [['sku' => '$product.sku$', 'qty' => 1]]]
9393
),
9494
DataFixture(
9595
CreditmemoFixture::class,
96-
['order_id' => '$order.id$', 'items' => [['product_id' => '$product.id$', 'qty' => 2]]]
96+
['order_id' => '$order.id$', 'items' => [['sku' => '$product.sku$', 'qty' => 2]]]
9797
),
9898
DataFixture(
9999
ShipmentFixture::class,
100-
['order_id' => '$order.id$', 'items' => [['product_id' => '$product.id$', 'qty' => 1]]]
100+
['order_id' => '$order.id$', 'items' => [['sku' => '$product.sku$', 'qty' => 1]]]
101101
),
102102
DataFixture(
103103
CreditmemoFixture::class,
104-
['order_id' => '$order.id$', 'items' => [['product_id' => '$product.id$', 'qty' => 2]]]
104+
['order_id' => '$order.id$', 'items' => [['sku' => '$product.sku$', 'qty' => 2]]]
105105
)
106106
]
107107
public function testGetSimpleQtyToShip()

0 commit comments

Comments
 (0)