Fix static test

Author: engcom-Echo

app/code/Magento/CatalogUrlRewrite/Model/ProductUrlPathGenerator.php

@@ -3,20 +3,19 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+declare(strict_types=1);
 
 namespace Magento\CatalogUrlRewrite\Model;
 
-use Magento\Store\Model\Store;
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\Catalog\Model\Category;
 use Magento\Catalog\Model\Product;
-use Magento\CatalogUrlRewrite\Model\CategoryUrlPathGenerator;
 use Magento\Framework\App\Config\ScopeConfigInterface;
 use Magento\Store\Model\ScopeInterface;
 use Magento\Store\Model\StoreManagerInterface;
 
 /**
- * Class ProductUrlPathGenerator
+ * Model product url path generator
  */
 class ProductUrlPathGenerator
 {

app/code/Magento/CatalogUrlRewrite/Test/Unit/Model/ProductUrlPathGeneratorTest.php

@@ -7,42 +7,50 @@
 
 namespace Magento\CatalogUrlRewrite\Test\Unit\Model;
 
+use Magento\Catalog\Api\ProductRepositoryInterface;
+use Magento\Catalog\Model\Category;
+use Magento\Catalog\Model\Product;
+use Magento\CatalogUrlRewrite\Model\CategoryUrlPathGenerator;
 use Magento\CatalogUrlRewrite\Model\ProductUrlPathGenerator;
+use Magento\Framework\App\Config\ScopeConfigInterface;
 use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
 use Magento\Store\Model\ScopeInterface;
+use Magento\Store\Model\StoreManagerInterface;
+use PHPUnit\Framework\TestCase;
+use PHPUnit_Framework_MockObject_MockObject as MockObject;
 
 /**
- * Class ProductUrlPathGeneratorTest
+ * Verify ProductUrlPathGenerator class
  */
-class ProductUrlPathGeneratorTest extends \PHPUnit\Framework\TestCase
+class ProductUrlPathGeneratorTest extends TestCase
 {
-    /** @var \Magento\CatalogUrlRewrite\Model\ProductUrlPathGenerator */
+    /** @var ProductUrlPathGenerator */
     protected $productUrlPathGenerator;
 
-    /** @var \Magento\Store\Model\StoreManagerInterface|\PHPUnit_Framework_MockObject_MockObject */
+    /** @var StoreManagerInterface|MockObject */
     protected $storeManager;
 
-    /** @var \Magento\Framework\App\Config\ScopeConfigInterface|\PHPUnit_Framework_MockObject_MockObject */
+    /** @var ScopeConfigInterface|MockObject */
     protected $scopeConfig;
 
-    /** @var \Magento\CatalogUrlRewrite\Model\CategoryUrlPathGenerator|\PHPUnit_Framework_MockObject_MockObject */
+    /** @var CategoryUrlPathGenerator|MockObject */
     protected $categoryUrlPathGenerator;
 
-    /** @var \Magento\Catalog\Model\Product|\PHPUnit_Framework_MockObject_MockObject */
+    /** @var Product|MockObject */
     protected $product;
 
-    /** @var \Magento\Catalog\Api\ProductRepositoryInterface|\PHPUnit_Framework_MockObject_MockObject */
+    /** @var ProductRepositoryInterface|MockObject */
     protected $productRepository;
 
-    /** @var \Magento\Catalog\Model\Category|\PHPUnit_Framework_MockObject_MockObject */
+    /** @var Category|MockObject */
     protected $category;
 
     /**
      * @inheritdoc
      */
     protected function setUp(): void
     {
-        $this->category = $this->createMock(\Magento\Catalog\Model\Category::class);
+        $this->category = $this->createMock(Category::class);
         $productMethods = [
             '__wakeup',
             'getData',
@@ -54,17 +62,17 @@ protected function setUp(): void
             'setStoreId',
         ];
 
-        $this->product = $this->createPartialMock(\Magento\Catalog\Model\Product::class, $productMethods);
-        $this->storeManager = $this->createMock(\Magento\Store\Model\StoreManagerInterface::class);
-        $this->scopeConfig = $this->createMock(\Magento\Framework\App\Config\ScopeConfigInterface::class);
+        $this->product = $this->createPartialMock(Product::class, $productMethods);
+        $this->storeManager = $this->createMock(StoreManagerInterface::class);
+        $this->scopeConfig = $this->createMock(ScopeConfigInterface::class);
         $this->categoryUrlPathGenerator = $this->createMock(
-            \Magento\CatalogUrlRewrite\Model\CategoryUrlPathGenerator::class
+            CategoryUrlPathGenerator::class
         );
-        $this->productRepository = $this->createMock(\Magento\Catalog\Api\ProductRepositoryInterface::class);
+        $this->productRepository = $this->createMock(ProductRepositoryInterface::class);
         $this->productRepository->expects($this->any())->method('getById')->willReturn($this->product);
 
         $this->productUrlPathGenerator = (new ObjectManager($this))->getObject(
-            \Magento\CatalogUrlRewrite\Model\ProductUrlPathGenerator::class,
+            ProductUrlPathGenerator::class,
             [
                 'storeManager' => $this->storeManager,
                 'scopeConfig' => $this->scopeConfig,
@@ -75,6 +83,8 @@ protected function setUp(): void
     }
 
     /**
+     * Data provider for testGetUrlPath.
+     *
      * @return array
      */
     public function getUrlPathDataProvider(): array
@@ -89,6 +99,8 @@ public function getUrlPathDataProvider(): array
     }
 
     /**
+     * Verify get url path.
+     *
      * @dataProvider getUrlPathDataProvider
      * @param string|null|bool $urlKey
      * @param string|null|bool $productName
@@ -109,6 +121,8 @@ public function testGetUrlPath($urlKey, $productName, $formatterCalled, $result)
     }
 
     /**
+     * Verify get url key.
+     *
      * @param string|bool $productUrlKey
      * @param string|bool $expectedUrlKey
      * @return void
@@ -122,6 +136,8 @@ public function testGetUrlKey($productUrlKey, $expectedUrlKey): void
     }
 
     /**
+     * Data provider for testGetUrlKey.
+     *
      * @return array
      */
     public function getUrlKeyDataProvider(): array
@@ -133,6 +149,8 @@ public function getUrlKeyDataProvider(): array
     }
 
     /**
+     * Verify get url path with default utl key.
+     *
      * @param string|null|bool $storedUrlKey
      * @param string|null|bool $productName
      * @param string $expectedUrlKey
@@ -150,6 +168,8 @@ public function testGetUrlPathDefaultUrlKey($storedUrlKey, $productName, $expect
     }
 
     /**
+     * Data provider for testGetUrlPathDefaultUrlKey.
+     *
      * @return array
      */
     public function getUrlPathDefaultUrlKeyDataProvider(): array
@@ -161,6 +181,8 @@ public function getUrlPathDefaultUrlKeyDataProvider(): array
     }
 
     /**
+     * Verify get url path with category.
+     *
      * @return void
      */
     public function testGetUrlPathWithCategory(): void
@@ -177,6 +199,8 @@ public function testGetUrlPathWithCategory(): void
     }
 
     /**
+     * Verify get url path with suffix.
+     *
      * @return void
      */
     public function testGetUrlPathWithSuffix(): void
@@ -198,6 +222,8 @@ public function testGetUrlPathWithSuffix(): void
     }
 
     /**
+     * Verify get url path with suffix and category and store.
+     *
      * @return void
      */
     public function testGetUrlPathWithSuffixAndCategoryAndStore(): void
@@ -219,6 +245,8 @@ public function testGetUrlPathWithSuffixAndCategoryAndStore(): void
     }
 
     /**
+     * Verify get canonical url path.
+     *
      * @return void
      */
     public function testGetCanonicalUrlPath(): void
@@ -232,6 +260,8 @@ public function testGetCanonicalUrlPath(): void
     }
 
     /**
+     * Verify get canonical path with category.
+     *
      * @return void
      */
     public function testGetCanonicalUrlPathWithCategory(): void