MC-15941: Product media_gallery_entries / types only present if image, thumbnail, small_image is requested

- add api-functional test

Author: danielrenaud

dev/tests/api-functional/testsuite/Magento/GraphQl/Catalog/MediaGalleryTest.php

@@ -9,6 +9,9 @@
 
 use Magento\TestFramework\TestCase\GraphQlAbstract;
 
+/**
+ * Test media gallery queries
+ */
 class MediaGalleryTest extends GraphQlAbstract
 {
     /**
@@ -45,6 +48,43 @@ public function testProductSmallImageUrlWithExistingImage()
         self::assertTrue($this->checkImageExists($response['products']['items'][0]['small_image']['url']));
     }
 
+    /**
+     * @magentoApiDataFixture Magento/Catalog/_files/product_with_multiple_images.php
+     */
+    public function testMediaGalleryTypesAreCorrect()
+    {
+        $productSku = 'simple';
+        $query = <<<QUERY
+{
+  products(filter: {sku: {eq: "{$productSku}"}}) {
+    items {    
+      media_gallery_entries {
+      	label
+        media_type
+        file
+        types
+      }
+    }
+  }    
+}
+QUERY;
+        $response = $this->graphQlQuery($query);
+
+        $this->assertNotEmpty($response['products']['items'][0]['media_gallery_entries']);
+        $mediaGallery = $response['products']['items'][0]['media_gallery_entries'];
+        $this->assertCount(2, $mediaGallery);
+
+        $this->assertEquals('Image Alt Text', $mediaGallery[0]['label']);
+        $this->assertEquals('image', $mediaGallery[0]['media_type']);
+        $this->assertContains('magento_image', $mediaGallery[0]['file']);
+        $this->assertEquals(['image', 'small_image'], $mediaGallery[0]['types']);
+
+        $this->assertEquals('Thumbnail Image', $mediaGallery[1]['label']);
+        $this->assertEquals('image', $mediaGallery[1]['media_type']);
+        $this->assertContains('magento_thumbnail', $mediaGallery[1]['file']);
+        $this->assertEquals(['thumbnail', 'swatch_image'], $mediaGallery[1]['types']);
+    }
+
     /**
      * @param string $url
      * @return bool

dev/tests/integration/testsuite/Magento/Catalog/_files/product_image.php

@@ -18,7 +18,14 @@
 $mediaDirectory->create($targetDirPath);
 $mediaDirectory->create($targetTmpDirPath);
 
-$targetTmpFilePath = $mediaDirectory->getAbsolutePath() . DIRECTORY_SEPARATOR . $targetTmpDirPath
-    . DIRECTORY_SEPARATOR . 'magento_image.jpg';
-copy(__DIR__ . '/magento_image.jpg', $targetTmpFilePath);
-// Copying the image to target dir is not necessary because during product save, it will be moved there from tmp dir
+$images = ['magento_image.jpg', 'magento_small_image.jpg', 'magento_thumbnail.jpg'];
+
+foreach ($images as $image) {
+    $targetTmpFilePath = $mediaDirectory->getAbsolutePath() . DIRECTORY_SEPARATOR . $targetTmpDirPath
+        . DIRECTORY_SEPARATOR . $image;
+
+    $sourceFilePath = __DIR__ . DIRECTORY_SEPARATOR . $image;
+
+    copy($sourceFilePath, $targetTmpFilePath);
+    // Copying the image to target dir is not necessary because during product save, it will be moved there from tmp dir
+}

dev/tests/integration/testsuite/Magento/Catalog/_files/product_with_multiple_images.php

@@ -0,0 +1,37 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+require __DIR__ . '/product_image.php';
+require __DIR__ . '/product_simple.php';
+
+$objectManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
+$productRepository = $objectManager->create(\Magento\Catalog\Api\ProductRepositoryInterface::class);
+$product = $productRepository->get('simple');
+
+/** @var $product \Magento\Catalog\Model\Product */
+$product->setStoreId(0)
+    ->setImage('/m/a/magento_image.jpg')
+    ->setSmallImage('/m/a/magento_image.jpg')
+    ->setThumbnail('/m/a/magento_thumbnail.jpg')
+    ->setSwatchImage('/m/a/magento_thumbnail.jpg')
+    ->setData('media_gallery', ['images' => [
+        [
+            'file' => '/m/a/magento_image.jpg',
+            'position' => 1,
+            'label' => 'Image Alt Text',
+            'disabled' => 0,
+            'media_type' => 'image'
+        ],
+        [
+            'file' => '/m/a/magento_thumbnail.jpg',
+            'position' => 2,
+            'label' => 'Thumbnail Image',
+            'disabled' => 0,
+            'media_type' => 'image'
+        ],
+    ]])
+    ->setCanSaveCustomOptions(true)
+    ->save();

dev/tests/integration/testsuite/Magento/Catalog/_files/product_with_multiple_images_rollback.php

@@ -0,0 +1,7 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+require __DIR__ . '/product_with_image_rollback.php';