Skip to content

Commit 79d15ad

Browse files
magento#10045 Fix coding standards
1 parent d990c95 commit 79d15ad

File tree

4 files changed

+10
-9
lines changed

4 files changed

+10
-9
lines changed

app/code/Magento/Sitemap/Test/Unit/Model/CategorySitemapItemResolverTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ public function testGetItems(array $categories)
5050
$resolver = new CategorySitemapItemResolver($storeConfigMock, $cmsPageFactoryMock, $itemFactoryMock);
5151
$items = $resolver->getItems(1);
5252
self::assertTrue(count($items) == count($categories));
53-
foreach($categories as $index => $category) {
53+
foreach ($categories as $index => $category) {
5454
self::assertSame($category->getUpdatedAt(), $items[$index]->getUpdatedAt());
5555
self::assertSame('daily', $items[$index]->getChangeFrequency());
5656
self::assertSame('1.0', $items[$index]->getPriority());

app/code/Magento/Sitemap/Test/Unit/Model/CmsPageSitemapItemResolverTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,4 +149,4 @@ private function getCmsPageCollectionMock($returnValue)
149149

150150
return $sitemapCmsPageMock;
151151
}
152-
}
152+
}

app/code/Magento/Sitemap/Test/Unit/Model/CompositeSitemapItemResolverTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,8 @@
1212

1313
class CompositeSitemapItemResolverTest extends \PHPUnit_Framework_TestCase
1414
{
15-
public function testNoResolvers() {
15+
public function testNoResolvers()
16+
{
1617
$resolver = new CompositeSitemapItemResolver();
1718
self::assertSame([], $resolver->getItems(1));
1819
}

app/code/Magento/Sitemap/Test/Unit/Model/ProductSitemapItemResolverTest.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,24 +34,24 @@ public function testGetItemsEmpty()
3434

3535
/**
3636
* @dataProvider productProvider
37-
* @param array $categories
37+
* @param array $products
3838
*/
39-
public function testGetItems(array $categories)
39+
public function testGetItems(array $products)
4040
{
4141
$storeConfigMock = $this->getStoreConfigMock([
4242
ProductSitemapItemResolver::XML_PATH_PRODUCT_CHANGEFREQ => 'daily',
4343
ProductSitemapItemResolver::XML_PATH_PRODUCT_PRIORITY => '1.0',
4444
]);
4545

46-
$ProductMock = $this->getProductCollectionMock($categories);
46+
$ProductMock = $this->getProductCollectionMock($products);
4747

4848
$cmsPageFactoryMock = $this->getProductFactoryMock($ProductMock);
4949
$itemFactoryMock = $this->getItemFactoryMock();
5050

5151
$resolver = new ProductSitemapItemResolver($storeConfigMock, $cmsPageFactoryMock, $itemFactoryMock);
5252
$items = $resolver->getItems(1);
53-
self::assertTrue(count($items) == count($categories));
54-
foreach($categories as $index => $product) {
53+
self::assertTrue(count($items) == count($products));
54+
foreach ($products as $index => $product) {
5555
self::assertSame($product->getUpdatedAt(), $items[$index]->getUpdatedAt());
5656
self::assertSame('daily', $items[$index]->getChangeFrequency());
5757
self::assertSame('1.0', $items[$index]->getPriority());
@@ -171,4 +171,4 @@ private function getProductCollectionMock($returnValue)
171171

172172
return $sitemapCmsPageMock;
173173
}
174-
}
174+
}

0 commit comments

Comments
 (0)