AC-15345: Undefined array key in ProductRepository getById

Unit test coverage for returning int value of productId

Author: sumesh-GL

app/code/Magento/Review/Test/Unit/Block/FormTest.php

@@ -1,7 +1,7 @@
 <?php
 /**
- * Copyright © Magento, Inc. All rights reserved.
- * See COPYING.txt for license details.
+ * Copyright 2015 Adobe
+ * All Rights Reserved.
  */
 declare(strict_types=1);
 
@@ -129,6 +129,34 @@ public function testGetProductInfo()
         $this->assertSame($productMock, $this->object->getProductInfo());
     }
 
+    public function testGetProductInfoNonIntParam()
+    {
+        $productId = 3;
+        $productIdNonInt = "3abc";
+        $storeId = 1;
+
+        $this->storeManager->expects(
+            $this->any()
+        )->method(
+            'getStore'
+        )->willReturn(
+            new DataObject(['id' => $storeId])
+        );
+
+        $this->requestMock->expects($this->once())
+            ->method('getParam')
+            ->with('id', false)
+            ->willReturn($productIdNonInt);
+
+        $productMock = $this->getMockForAbstractClass(ProductInterface::class);
+        $this->productRepository->expects($this->once())
+            ->method('getById')
+            ->with($productId, false, $storeId)
+            ->willReturn($productMock);
+
+        $this->assertSame($productMock, $this->object->getProductInfo());
+    }
+
     /**
      * @param bool $isSecure
      * @param string $actionUrl