ACP2E-1315: [GraphQL] Any child products of bundle product are not returned in GraphQL response

Author: dhorytskyi

app/code/Magento/CatalogGraphQl/Model/Resolver/Products/DataProvider/Product.php

@@ -7,6 +7,7 @@
 
 namespace Magento\CatalogGraphQl\Model\Resolver\Products\DataProvider;
 
+use Magento\Catalog\Api\Data\ProductInterface;
 use Magento\Catalog\Model\Product\Visibility;
 use Magento\CatalogGraphQl\Model\Resolver\Products\DataProvider\Product\CollectionPostProcessor;
 use Magento\Framework\Api\SearchCriteriaInterface;
@@ -93,7 +94,7 @@ public function getList(
             $visibilityIds = $isSearch
                 ? $this->visibility->getVisibleInSearchIds()
                 : $this->visibility->getVisibleInCatalogIds();
-            $collection->setVisibility($visibilityIds);
+            $collection->addAttributeToFilter(ProductInterface::VISIBILITY, $visibilityIds);
         }
 
         $collection->load();

dev/tests/api-functional/testsuite/Magento/GraphQl/Bundle/BundleProductViewTest.php

@@ -212,7 +212,7 @@ public function testBundleProductWithNotVisibleChildren()
         }
         $this->assertBundleBaseFields($bundleProduct, $response['products']['items'][0]);
 
-        $this->assertBundleProductOptions($bundleProduct, $response['products']['items'][0], false);
+        $this->assertBundleProductOptions($bundleProduct, $response['products']['items'][0]);
         $this->assertNotEmpty(
             $response['products']['items'][0]['items'],
             "Precondition failed: 'items' must not be empty"
@@ -242,9 +242,8 @@ private function assertBundleBaseFields($product, $actualResponse)
     /**
      * @param ProductInterface $product
      * @param  array $actualResponse
-     * @param bool $isChildVisible
      */
-    private function assertBundleProductOptions($product, $actualResponse, $isChildVisible = true)
+    private function assertBundleProductOptions($product, $actualResponse)
     {
         $this->assertNotEmpty(
             $actualResponse['items'],
@@ -284,22 +283,18 @@ private function assertBundleProductOptions($product, $actualResponse, $isChildV
             ]
         );
         $this->assertEquals(
-            $isChildVisible ? $childProduct->getName() : null,
+            $childProduct->getName(),
             $actualResponse['items'][0]['options'][0]['label']
         );
-        if ($isChildVisible) {
-            $this->assertResponseFields(
-                $actualResponse['items'][0]['options'][0]['product'],
-                [
-                    'id' => $childProduct->getId(),
-                    'name' => $childProduct->getName(),
-                    'type_id' => $childProduct->getTypeId(),
-                    'sku' => $childProduct->getSku()
-                ]
-            );
-        } else {
-            $this->assertNull($actualResponse['items'][0]['options'][0]['product']);
-        }
+        $this->assertResponseFields(
+            $actualResponse['items'][0]['options'][0]['product'],
+            [
+                'id' => $childProduct->getId(),
+                'name' => $childProduct->getName(),
+                'type_id' => $childProduct->getTypeId(),
+                'sku' => $childProduct->getSku()
+            ]
+        );
     }
 
     /**