Code review changes

Author: Mohamed-Asar

app/code/Magento/CatalogInventory/Block/Plugin/ProductView.php

@@ -7,6 +7,7 @@
 
 use Magento\Catalog\Block\Product\View;
 use Magento\CatalogInventory\Model\Product\QuantityValidator;
+use Magento\Framework\App\ObjectManager;
 
 class ProductView
 {
@@ -16,12 +17,14 @@ class ProductView
     private $productQuantityValidator;
 
     /**
-     * @param QuantityValidator $productQuantityValidator
+     * @param QuantityValidator|null $productQuantityValidator
      */
     public function __construct(
-        QuantityValidator $productQuantityValidator
+        QuantityValidator $productQuantityValidator = null
     ) {
-        $this->productQuantityValidator = $productQuantityValidator;
+        $this->productQuantityValidator = $productQuantityValidator ?? ObjectManager::getInstance()->get(
+            QuantityValidator::class
+        );
     }
 
     /**

app/code/Magento/CatalogInventory/Model/Product/QuantityValidator.php

@@ -31,6 +31,10 @@ public function getData(int $productId, int|null $websiteId): array
     {
         $stockItem = $this->stockRegistry->getStockItem($productId, $websiteId);
 
+        if (!$stockItem) {
+            return [];
+        }
+
         $params = [];
         $validators = [];
         $params['minAllowed'] =  $stockItem->getMinSaleQty();

app/code/Magento/CatalogInventory/Test/Unit/Model/Product/QuantityValidatorTest.php

@@ -69,4 +69,64 @@ public function testGetDataWithMinMaxAndIncrements(): void
         $result = $this->quantityValidator->getData(self::PRODUCT_ID, self::WEBSITE_ID);
         $this->assertEquals($expected, $result);
     }
+
+    public function testReturnsEmptyArrayForNonExistentProductOrWebsite(): void
+    {
+        $this->stockRegistry->expects($this->once())
+            ->method('getStockItem')
+            ->with(self::PRODUCT_ID, self::WEBSITE_ID)
+            ->willReturn(null);
+
+        $result = $this->quantityValidator->getData(self::PRODUCT_ID, self::WEBSITE_ID);
+        $this->assertSame([], $result, 'Should return empty array when StockItem is not found');
+    }
+
+    public function testHandlesNullValuesFromStockItem(): void
+    {
+        $stockItem = $this->createMock(StockItemInterface::class);
+        $stockItem->method('getMinSaleQty')
+            ->willReturn(null);
+        $stockItem->method('getMaxSaleQty')
+            ->willReturn(null);
+        $stockItem->method('getQtyIncrements')
+            ->willReturn(null);
+
+        $this->stockRegistry->expects($this->once())
+            ->method('getStockItem')
+            ->with(self::PRODUCT_ID, self::WEBSITE_ID)
+            ->willReturn($stockItem);
+
+        $expected = [
+            'validate-item-quantity' => [
+                'minAllowed'    => null
+            ],
+        ];
+        $result = $this->quantityValidator->getData(self::PRODUCT_ID, self::WEBSITE_ID);
+        $this->assertEquals($expected, $result);
+    }
+
+    public function testHandlesInvalidValuesFromStockItem(): void
+    {
+        $stockItem = $this->createMock(StockItemInterface::class);
+        $stockItem->method('getMinSaleQty')
+            ->willReturn('not-a-number');
+        $stockItem->method('getMaxSaleQty')
+            ->willReturn(-5);
+        $stockItem->method('getQtyIncrements')
+            ->willReturn(false);
+
+        $this->stockRegistry->expects($this->once())
+            ->method('getStockItem')
+            ->with(self::PRODUCT_ID, self::WEBSITE_ID)
+            ->willReturn($stockItem);
+
+        $expected = [
+            'validate-item-quantity' => [
+                'minAllowed'    => 'not-a-number',
+                'maxAllowed'    => -5
+            ],
+        ];
+        $result = $this->quantityValidator->getData(self::PRODUCT_ID, self::WEBSITE_ID);
+        $this->assertEquals($expected, $result);
+    }
 }