MC-39598: Create automated test for: "Assign/Unassign product to websites via API"

Author: ysapiga

dev/tests/api-functional/testsuite/Magento/Catalog/Api/ProductWebsiteLinkRepositoryTest.php

@@ -46,32 +46,19 @@ protected function setUp(): void
         $this->websiteRepository = $this->objectManager->get(WebsiteRepositoryInterface::class);
     }
 
-    /**
-     * @magentoApiDataFixture Magento/Catalog/_files/second_product_simple.php
-     *
-     * @return void
-     */
-    public function testSaveWebsiteLinkWithoutWebsiteId(): void
-    {
-        $serviceInfo = $this->fillServiceInfo('/V1/products/:sku/websites', Request::HTTP_METHOD_POST, 'Save');
-        $requestData = ['productWebsiteLink' => ['sku' => 'simple2']];
-        $this->expectException(\Exception::class);
-        $this->expectErrorMessage((string)__('There are not websites for assign to product'));
-        $this->_webApiCall($serviceInfo, $requestData);
-    }
-
     /**
      * @magentoApiDataFixture Magento/Catalog/_files/second_product_simple.php
      *
      * @return void
      */
     public function testSaveWebsiteLinkWithUnexistingWebsiteId(): void
     {
+        $pattern = '/(Could\\snot\\sassign\\sproduct)+([\\s\\S]*)(to\\swebsites)+([\\s\\S]*)/';
         $unexistingWebsiteId = 8932568989;
         $serviceInfo = $this->fillServiceInfo('/V1/products/:sku/websites', Request::HTTP_METHOD_POST, 'Save');
         $requestData = ['productWebsiteLink' => ['sku' => 'simple2', 'websiteId' => $unexistingWebsiteId]];
         $this->expectException(\Exception::class);
-        $this->expectExceptionMessageMatches('/Could not assign product \\\"%1\\\" to websites \\\"%2\\\"/');
+        $this->expectExceptionMessageMatches($pattern);
         $this->_webApiCall($serviceInfo, $requestData);
     }
 
@@ -85,8 +72,8 @@ public function testDeleteWebsiteLink(): void
         $productSku = 'unique-simple-azaza';
         $websiteId = (int)$this->websiteRepository->get('second_website')->getId();
         $resourcePath = sprintf('/V1/products/%s/websites/%u', $productSku, $websiteId);
-        $serviceInfo = $this->fillServiceInfo($resourcePath, Request::HTTP_METHOD_DELETE, 'Delete');
-        $this->_webApiCall($serviceInfo);
+        $serviceInfo = $this->fillServiceInfo($resourcePath, Request::HTTP_METHOD_DELETE, 'DeleteById');
+        $this->_webApiCall($serviceInfo, ['sku' => $productSku, 'websiteId' => $websiteId]);
         $product = $this->productRepository->get($productSku, false, null, true);
         $this->assertNotContains($websiteId, $product->getWebsiteIds());
     }

dev/tests/integration/testsuite/Magento/Catalog/Model/ProductWebsiteLinkRepositoryTest.php

@@ -10,6 +10,7 @@
 use Magento\Catalog\Api\Data\ProductWebsiteLinkInterfaceFactory;
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\Catalog\Api\ProductWebsiteLinkRepositoryInterface;
+use Magento\Framework\Exception\InputException;
 use Magento\Framework\ObjectManagerInterface;
 use Magento\Store\Api\WebsiteRepositoryInterface;
 use Magento\TestFramework\Helper\Bootstrap;
@@ -55,7 +56,21 @@ protected function setUp(): void
     }
 
     /**
-     * @magentoApiDataFixture Magento/Catalog/_files/product_with_two_websites.php
+     * @magentoDataFixture Magento/Catalog/_files/second_product_simple.php
+     *
+     * @return void
+     */
+    public function testSaveWithoutWebsiteId(): void
+    {
+        $productWebsiteLink = $this->productWebsiteLinkFactory->create();
+        $productWebsiteLink->setSku('unique-simple-azaza');
+        $this->expectException(InputException::class);
+        $this->expectErrorMessage((string)__('There are not websites for assign to product'));
+        $this->productWebsiteLinkRepository->save($productWebsiteLink);
+    }
+
+    /**
+     * @magentoDataFixture Magento/Catalog/_files/product_with_two_websites.php
      *
      * @return void
      */