Skip to content

Commit 0b94121

Browse files
committed
Minor change
1 parent 6cf26dc commit 0b94121

File tree

3 files changed

+44
-21
lines changed

3 files changed

+44
-21
lines changed

app/code/Magento/CatalogUrlRewrite/Test/Unit/Model/Product/AnchorUrlRewriteGeneratorTest.php

Lines changed: 42 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -3,53 +3,67 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6+
declare(strict_types=1);
7+
68
namespace Magento\CatalogUrlRewrite\Test\Unit\Model\Product;
79

10+
use Magento\Catalog\Api\CategoryRepositoryInterface;
11+
use Magento\Catalog\Model\Product;
12+
use Magento\CatalogUrlRewrite\Model\ObjectRegistry;
13+
use Magento\CatalogUrlRewrite\Model\Product\AnchorUrlRewriteGenerator;
14+
use Magento\CatalogUrlRewrite\Model\ProductUrlPathGenerator;
815
use Magento\CatalogUrlRewrite\Model\ProductUrlRewriteGenerator;
916
use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
17+
use Magento\UrlRewrite\Service\V1\Data\UrlRewrite;
18+
use Magento\UrlRewrite\Service\V1\Data\UrlRewriteFactory;
19+
use PHPUnit\Framework\TestCase;
20+
use PHPUnit_Framework_MockObject_MockObject as MockObject;
1021

11-
class AnchorUrlRewriteGeneratorTest extends \PHPUnit\Framework\TestCase
22+
class AnchorUrlRewriteGeneratorTest extends TestCase
1223
{
13-
/** @var \Magento\CatalogUrlRewrite\Model\Product\AnchorUrlRewriteGenerator */
24+
/** @var AnchorUrlRewriteGenerator */
1425
protected $anchorUrlRewriteGenerator;
1526

16-
/** @var \Magento\CatalogUrlRewrite\Model\ProductUrlPathGenerator|\PHPUnit_Framework_MockObject_MockObject */
27+
/** @var ProductUrlPathGenerator|MockObject */
1728
protected $productUrlPathGenerator;
1829

19-
/** @var \Magento\Catalog\Model\Product|\PHPUnit_Framework_MockObject_MockObject */
30+
/** @var Product|MockObject */
2031
protected $product;
2132

22-
/** @var \Magento\Catalog\Api\CategoryRepositoryInterface|\PHPUnit_Framework_MockObject_MockObject */
33+
/** @var CategoryRepositoryInterface|MockObject */
2334
private $categoryRepositoryInterface;
2435

25-
/** @var \Magento\CatalogUrlRewrite\Model\ObjectRegistry|\PHPUnit_Framework_MockObject_MockObject */
36+
/** @var ObjectRegistry|MockObject */
2637
protected $categoryRegistry;
2738

28-
/** @var \Magento\UrlRewrite\Service\V1\Data\UrlRewriteFactory|\PHPUnit_Framework_MockObject_MockObject */
39+
/** @var UrlRewriteFactory|MockObject */
2940
protected $urlRewriteFactory;
3041

31-
/** @var \Magento\UrlRewrite\Service\V1\Data\UrlRewrite|\PHPUnit_Framework_MockObject_MockObject */
42+
/** @var UrlRewrite|MockObject */
3243
protected $urlRewrite;
3344

45+
/**
46+
* @inheritDoc
47+
*/
3448
protected function setUp()
3549
{
36-
$this->urlRewriteFactory = $this->getMockBuilder(\Magento\UrlRewrite\Service\V1\Data\UrlRewriteFactory::class)
50+
$this->urlRewriteFactory = $this->getMockBuilder(UrlRewriteFactory::class)
3751
->setMethods(['create'])
3852
->disableOriginalConstructor()->getMock();
39-
$this->urlRewrite = $this->getMockBuilder(\Magento\UrlRewrite\Service\V1\Data\UrlRewrite::class)
53+
$this->urlRewrite = $this->getMockBuilder(UrlRewrite::class)
4054
->disableOriginalConstructor()->getMock();
41-
$this->product = $this->getMockBuilder(\Magento\Catalog\Model\Product::class)
55+
$this->product = $this->getMockBuilder(Product::class)
4256
->disableOriginalConstructor()->getMock();
4357
$this->categoryRepositoryInterface = $this->getMockBuilder(
44-
\Magento\Catalog\Api\CategoryRepositoryInterface::class
58+
CategoryRepositoryInterface::class
4559
)->disableOriginalConstructor()->getMock();
46-
$this->categoryRegistry = $this->getMockBuilder(\Magento\CatalogUrlRewrite\Model\ObjectRegistry::class)
60+
$this->categoryRegistry = $this->getMockBuilder(ObjectRegistry::class)
4761
->disableOriginalConstructor()->getMock();
4862
$this->productUrlPathGenerator = $this->getMockBuilder(
49-
\Magento\CatalogUrlRewrite\Model\ProductUrlPathGenerator::class
63+
ProductUrlPathGenerator::class
5064
)->disableOriginalConstructor()->getMock();
5165
$this->anchorUrlRewriteGenerator = (new ObjectManager($this))->getObject(
52-
\Magento\CatalogUrlRewrite\Model\Product\AnchorUrlRewriteGenerator::class,
66+
AnchorUrlRewriteGenerator::class,
5367
[
5468
'productUrlPathGenerator' => $this->productUrlPathGenerator,
5569
'urlRewriteFactory' => $this->urlRewriteFactory,
@@ -58,7 +72,12 @@ protected function setUp()
5872
);
5973
}
6074

61-
public function testGenerateEmpty()
75+
/**
76+
* Verify generate if category registry list is empty.
77+
*
78+
* @return void
79+
*/
80+
public function testGenerateEmpty(): void
6281
{
6382
$this->categoryRegistry->expects($this->any())->method('getList')->will($this->returnValue([]));
6483

@@ -68,7 +87,12 @@ public function testGenerateEmpty()
6887
);
6988
}
7089

71-
public function testGenerateCategories()
90+
/**
91+
* Verify generate product rewrites for anchor categories.
92+
*
93+
* @return void
94+
*/
95+
public function testGenerateCategories(): void
7296
{
7397
$urlPathWithCategory = 'category1/category2/category3/simple-product.html';
7498
$storeId = 10;
@@ -102,7 +126,7 @@ public function testGenerateCategories()
102126
->withConsecutive(
103127
[$categoryIds[0], $storeId],
104128
[$categoryIds[1], $storeId],
105-
[$categoryIds[2], $storeId],
129+
[$categoryIds[2], $storeId]
106130
)
107131
->will($this->returnValue($category));
108132
$this->categoryRegistry->expects($this->any())->method('getList')

dev/tests/integration/testsuite/Magento/CatalogUrlRewrite/Model/Product/AnchorUrlRewriteGeneratorTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
use Magento\TestFramework\Helper\Bootstrap;
1616
use PHPUnit\Framework\TestCase;
1717

18-
/**3
18+
/**
1919
* Verify generate url rewrites for anchor categories.
2020
*/
2121
class AnchorUrlRewriteGeneratorTest extends TestCase

dev/tests/integration/testsuite/Magento/CatalogUrlRewrite/_files/categories_with_stores.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@
1111
use Magento\TestFramework\Helper\Bootstrap;
1212

1313
require __DIR__ . '/../../../Magento/Store/_files/second_store.php';
14-
Bootstrap::getInstance()
15-
->loadArea(FrontNameResolver::AREA_CODE);
14+
Bootstrap::getInstance()->loadArea(FrontNameResolver::AREA_CODE);
1615

1716
/**
1817
* After installation system has categories:

0 commit comments

Comments
 (0)