#1823: Cover GetAssetByIdInterface with integration test - replaced the mock class with existing mock class and modified the sample data for the existing mock class

Author: yolouiese

AdobeStockAsset/Test/Integration/Model/ClientMock.php

@@ -35,7 +35,7 @@ public function search(SearchCriteriaInterface $searchCriteria): SearchResultInt
         $searchResult = new SearchResult();
         $searchResult->setSearchCriteria($searchCriteria);
         $searchResult->setItems($items);
-        $searchResult->setTotalCount(3);
+        $searchResult->setTotalCount(1);
 
         return $searchResult;
     }
@@ -64,40 +64,6 @@ private function getStockFiles(): array
                     ],
                     'category_id' => 1
                 ]
-            ],
-            [
-                'id' => 2,
-                'custom_attributes' => [
-                    'id_field_name' => 'id',
-                    'id' => 2,
-                    'thumbnail_240_url' => 'https://test.url/2',
-                    'width' => 120,
-                    'height' => 220,
-                    'comp_url' => 'https://test.url/2',
-                    'category' => [
-                        'id' => 1,
-                        'name' => 'Test',
-                        'link' => null
-                    ],
-                    'category_id' => 1
-                ]
-            ],
-            [
-                'id' => 3,
-                'custom_attributes' => [
-                    'id_field_name' => 'id',
-                    'id' => 3,
-                    'thumbnail_240_url' => 'https://test.url/3',
-                    'width' => 130,
-                    'height' => 230,
-                    'comp_url' => 'https://test.url/3',
-                    'category' => [
-                        'id' => 1,
-                        'name' => 'Test',
-                        'link' => null
-                    ],
-                    'category_id' => 1
-                ],
             ]
         ];
 

AdobeStockAsset/Test/Integration/Model/GetAssetByIdTest.php

@@ -8,7 +8,7 @@
 namespace Magento\AdobeStockAsset\Test\Integration\Model;
 
 use Magento\AdobeStockAssetApi\Api\GetAssetByIdInterface;
-use Magento\AdobeStockAssetApi\Api\GetAssetListInterface;
+use Magento\AdobeStockClientApi\Api\ClientInterface;
 use Magento\Framework\Api\Search\DocumentInterface;
 use Magento\TestFramework\Helper\Bootstrap;
 use PHPUnit\Framework\TestCase;
@@ -24,7 +24,7 @@ protected function setUp(): void
     {
         Bootstrap::getObjectManager()->configure([
             'preferences' => [
-                GetAssetListInterface::class => GetAssetListMock::class
+                ClientInterface::class => ClientMock::class
             ]
         ]);
 
@@ -34,10 +34,10 @@ protected function setUp(): void
     public function testExecute(): void
     {
         /** @var DocumentInterface $searchResults */
-        $searchResults = $this->getAssetById->execute(123455678);
+        $searchResults = $this->getAssetById->execute(1);
 
         $this->assertInstanceOf(DocumentInterface::class, $searchResults);
         $this->assertNotEmpty($searchResults);
-        $this->assertEquals(123455678, $searchResults->getId());
+        $this->assertEquals(1, $searchResults->getId());
     }
 }

AdobeStockAsset/Test/Integration/Model/GetAssetListMock.php

@@ -63,7 +63,7 @@ public function testExecute(): void
         $searchResults = $this->getAssetList->execute($searchCriteria);
 
         $this->assertInstanceOf(SearchResultInterface::class, $searchResults);
-        $this->assertEquals(3, $searchResults->getTotalCount());
-        $this->assertCount(3, array_values($searchResults->getItems()));
+        $this->assertEquals(1, $searchResults->getTotalCount());
+        $this->assertCount(1, array_values($searchResults->getItems()));
     }
 }