MC-35618: Unexpected results of editing a Grouped Product in the Wish List by a Customer

Author: StasKozar

app/code/Magento/GroupedProduct/Model/Wishlist/Product/Item.php

@@ -83,8 +83,7 @@ public function beforeCompareOptions(
 
         if (!$diff) {
             foreach (array_keys($options1) as $key) {
-                if (
-                    preg_match('/associated_product_\d+/', $key)
+                if (preg_match('/associated_product_\d+/', $key)
                     && $this->isAddAction($productOptions['info_buyRequest'])
                 ) {
                     unset($options1[$key]);

app/code/Magento/Wishlist/Model/Wishlist.php

@@ -756,9 +756,9 @@ public function updateItem($itemId, $buyRequest, $params = null)
             $isForceSetQuantity = true;
             foreach ($items as $_item) {
                 /* @var $_item Item */
-                if ($_item->getProductId() == $product->getId() && $_item->representProduct(
-                    $product
-                ) && $_item->getId() != $item->getId()
+                if ($_item->getProductId() == $product->getId()
+                    && $_item->getId() != $item->getId()
+                    && $_item->representProduct($product)
                 ) {
                     // We do not add new wishlist item, but updating the existing one
                     $isForceSetQuantity = false;

app/code/Magento/Wishlist/Test/Unit/Controller/Index/UpdateItemOptionsTest.php

@@ -380,7 +380,7 @@ public function testExecuteAddSuccessException()
         $wishlist
             ->expects($this->once())
             ->method('updateItem')
-            ->with(3, new DataObject([]))
+            ->with(3, new DataObject(['action' => 'updateItem']))
             ->willReturnSelf();
         $wishlist
             ->expects($this->once())
@@ -509,7 +509,7 @@ public function testExecuteAddSuccessCriticalException()
         $wishlist
             ->expects($this->once())
             ->method('updateItem')
-            ->with(3, new DataObject([]))
+            ->with(3, new DataObject(['action' => 'updateItem']))
             ->willReturnSelf();
         $wishlist
             ->expects($this->once())