#29174 Add API functional test for redirect generation for category updates

Author: amenk

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

@@ -218,6 +218,83 @@ public function testUpdate()
         $this->deleteCategory($categoryId);
     }
 
+    /**
+     * @magentoApiDataFixture Magento/Catalog/_files/category.php
+     */
+    public function testUpdateUrlKey()
+    {
+        $categoryId = 333;
+        $categoryData = [
+            'name' => 'Update Category Test Old Name',
+            'custom_attributes' => [
+                [
+                    'attribute_code' => 'url_key',
+                    'value' => "Update Category Test Old Name",
+                ],
+            ],
+        ];
+        $result = $this->updateCategory($categoryId, $categoryData);
+        $this->assertEquals($categoryId, $result['id']);
+
+
+        $categoryData = [
+            'name' => 'Update Category Test New Name',
+            'custom_attributes' => [
+                [
+                    'attribute_code' => 'url_key',
+                    'value' => "Update Category Test New Name",
+                ],
+                [
+                    'attribute_code' => 'save_rewrites_history',
+                    'value' => 1,
+                ],
+            ],
+        ];
+        $result = $this->updateCategory($categoryId, $categoryData);
+        $this->assertEquals($categoryId, $result['id']);
+        /** @var \Magento\Catalog\Model\Category $model */
+        $model = Bootstrap::getObjectManager()->get(\Magento\Catalog\Model\Category::class);
+        $category = $model->load($categoryId);
+        $this->assertEquals("Update Category Test New Name", $category->getName());
+        // delete category to clean up auto-generated url rewrites
+
+
+        $storage = Bootstrap::getObjectManager()->get(\Magento\UrlRewrite\Model\Storage\DbStorage::class);
+        $data = [
+            UrlRewrite::ENTITY_ID => $categoryId,
+            UrlRewrite::ENTITY_TYPE => CategoryUrlRewriteGenerator::ENTITY_TYPE,
+            UrlRewrite::REDIRECT_TYPE => 0,
+        ];
+
+        $urlRewrite = $storage->findOneByData($data);
+
+        // Assert that a url rewrite is auto-generated for the category created from the data fixture
+        $this->assertNotNull($urlRewrite);
+        $this->assertEquals(1, $urlRewrite->getIsAutogenerated());
+        $this->assertEquals($categoryId, $urlRewrite->getEntityId());
+        $this->assertEquals(CategoryUrlRewriteGenerator::ENTITY_TYPE, $urlRewrite->getEntityType());
+        $this->assertEquals('update-category-test-new-name.html', $urlRewrite->getRequestPath());
+
+
+        // check for the forward
+        $storage = Bootstrap::getObjectManager()->get(\Magento\UrlRewrite\Model\Storage\DbStorage::class);
+        $data = [
+            UrlRewrite::ENTITY_ID => $categoryId,
+            UrlRewrite::ENTITY_TYPE => CategoryUrlRewriteGenerator::ENTITY_TYPE,
+            UrlRewrite::REDIRECT_TYPE => 301,
+        ];
+
+        $urlRewrite = $storage->findOneByData($data);
+
+        $this->assertNotNull($urlRewrite);
+        $this->assertEquals(0, $urlRewrite->getIsAutogenerated());
+        $this->assertEquals($categoryId, $urlRewrite->getEntityId());
+        $this->assertEquals(CategoryUrlRewriteGenerator::ENTITY_TYPE, $urlRewrite->getEntityType());
+        $this->assertEquals('update-category-test-old-name.html', $urlRewrite->getRequestPath());
+
+        $this->deleteCategory($categoryId);
+    }
+
     protected function getSimpleCategoryData($categoryData = [])
     {
         return [