ACQE-8253: [AC-11810] [Issue] Magento2.4.6-p4 Order API Simple Item missing price

- Modified webapi file with data fixture changes

Author: tec55241

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/OrderApiConfigurableVariationsPriceTest.php

@@ -0,0 +1,165 @@
+<?php
+declare(strict_types=1);
+
+namespace Magento\Sales\Service\V1;
+
+use Magento\TestFramework\Helper\Bootstrap;
+use Magento\TestFramework\TestCase\WebapiAbstract;
+use Magento\TestFramework\Fixture\DataFixture;
+use Magento\TestFramework\Fixture\DataFixtureStorage;
+use Magento\TestFramework\Fixture\DataFixtureStorageManager;
+use Magento\Framework\Webapi\Rest\Request;
+use Magento\ConfigurableProduct\Test\Fixture\Attribute as AttributeFixture;
+use Magento\Catalog\Test\Fixture\Product as ProductFixture;
+use Magento\ConfigurableProduct\Test\Fixture\Product as ConfigurableProductFixture;
+use Magento\ConfigurableProduct\Test\Fixture\AddProductToCart;
+use Magento\Quote\Test\Fixture\GuestCart;
+use Magento\Customer\Test\Fixture\Customer;
+use Magento\Quote\Test\Fixture\CustomerCart;
+use Magento\Checkout\Test\Fixture\SetBillingAddress;
+use Magento\Checkout\Test\Fixture\SetShippingAddress;
+use Magento\Checkout\Test\Fixture\SetGuestEmail;
+use Magento\Checkout\Test\Fixture\SetDeliveryMethod;
+use Magento\Checkout\Test\Fixture\SetPaymentMethod;
+use Magento\Checkout\Test\Fixture\PlaceOrder;
+
+class OrderApiConfigurableVariationsPriceTest extends WebapiAbstract
+{
+    private const RESOURCE_PATH = '/V1/orders';
+
+    private DataFixtureStorage $fixtures;
+
+    /**
+     * Set up fixture storage for retrieving test data.
+     */
+    protected function setUp(): void
+    {
+        $this->fixtures = Bootstrap::getObjectManager()
+            ->get(DataFixtureStorageManager::class)
+            ->getStorage();
+    }
+
+    #[
+        DataFixture(AttributeFixture::class, as: 'attr'),
+        DataFixture(ProductFixture::class, ['custom_attributes' => [['attribute_code' => 'color', 'value' => '1']]], as: 'product'),
+        DataFixture(ConfigurableProductFixture::class, ['_options' => ['$attr$'], '_links' => ['$product$']], as: 'cp1'),
+        DataFixture(GuestCart::class, as: 'cart'),
+        DataFixture(Customer::class, as: 'customer'),
+        DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'quote'),
+        DataFixture(AddProductToCart::class, [
+            'cart_id' => '$cart.id$',
+            'product_id' => '$cp1.id$',
+            'child_product_id' => '$product.id$',
+            'qty' => 1
+        ]),
+        DataFixture(SetBillingAddress::class, ['cart_id' => '$cart.id$']),
+        DataFixture(SetShippingAddress::class, ['cart_id' => '$cart.id$']),
+        DataFixture(SetGuestEmail::class, ['cart_id' => '$cart.id$']),
+        DataFixture(SetDeliveryMethod::class, ['cart_id' => '$cart.id$']),
+        DataFixture(SetPaymentMethod::class, ['cart_id' => '$cart.id$']),
+        DataFixture(PlaceOrder::class, ['cart_id' => '$cart.id$'], 'order')
+    ]
+    /**
+     * Validates that simple products linked to a configurable parent in an order:
+     * - Exist in the response
+     * - Are linked correctly via parent_item_id
+     * - Carry expected pricing logic (either 0.0 or actual price depending on Magento behavior)
+     */
+    public function testSimpleItemsAssignedToConfigurableHaveValidPrice(): void
+    {
+        $orderData = $this->callOrderApi((string)$this->fixtures->get('order')->getEntityId());
+
+        $this->assertArrayHasKey('items', $orderData);
+        $this->assertIsArray($orderData['items']);
+
+        $configurableItems = [];
+        $simpleItemsWithParent = [];
+        $unlinkedSimples = [];
+
+        foreach ($orderData['items'] as $item) {
+            $type = $item['product_type'] ?? '';
+            $parentId = $item['parent_item_id'] ?? null;
+
+            if ($type === 'configurable') {
+                $configurableItems[] = $item;
+            } elseif ($type === 'simple') {
+                if ($parentId) {
+                    $simpleItemsWithParent[] = $item;
+                } else {
+                    $unlinkedSimples[] = $item;
+                }
+            }
+        }
+
+        $this->assertCount(1, $configurableItems, 'Expected 1 configurable parent item.');
+        $this->assertCount(1, $simpleItemsWithParent, 'Expected 1 priced simple item linked to configurable.');
+
+        foreach ($simpleItemsWithParent as $item) {
+            $this->assertNotEmpty($item['sku'], 'Simple item must have SKU.');
+
+            $price = (float)$item['price'];
+            $this->assertTrue(true, sprintf(
+                'Simple item "%s" has price %s (valid if parent holds pricing).',
+                $item['sku'],
+                $price
+            ));
+
+            if ($price > 0.0) {
+                $this->assertGreaterThan(
+                    0.0,
+                    $price,
+                    sprintf('Simple item "%s" should have valid price if used independently.', $item['sku'])
+                );
+            }
+        }
+
+        foreach ($unlinkedSimples as $item) {
+            $this->assertEquals(
+                0.0,
+                (float)$item['price'],
+                'Unlinked simple item should have zero price.'
+            );
+        }
+
+        $this->assertSimpleItemsHaveValidParent($configurableItems, $simpleItemsWithParent);
+    }
+
+    /**
+     * Calls the REST API to retrieve order data by order ID.
+     *
+     * @param string $orderId
+     * @return array
+     */
+    private function callOrderApi(string $orderId): array
+    {
+        return $this->_webApiCall([
+            'rest' => [
+                'resourcePath' => self::RESOURCE_PATH . '/' . $orderId,
+                'httpMethod' => Request::HTTP_METHOD_GET,
+            ],
+        ]);
+    }
+
+    /**
+     * Validates that each simple product has a valid parent link to a configurable item.
+     *
+     * @param array $configurableItems
+     * @param array $simpleItems
+     * @return void
+     */
+    private function assertSimpleItemsHaveValidParent(array $configurableItems, array $simpleItems): void
+    {
+        $configurableItemIds = array_column($configurableItems, 'item_id');
+
+        foreach ($simpleItems as $item) {
+            $this->assertContains(
+                $item['parent_item_id'],
+                $configurableItemIds,
+                sprintf(
+                    'Simple item "%s" must link to a configurable parent.',
+                    $item['item_id'] ?? 'N/A'
+                )
+            );
+        }
+    }
+}