Skip to content

Commit 51c4be2

Browse files
author
Stanislav Idolov
authored
ENGCOM-1369: [Forwardport] #4389 Widget cache error #14816
2 parents 08a72a5 + 121f40d commit 51c4be2

File tree

2 files changed

+15
-7
lines changed

2 files changed

+15
-7
lines changed

app/code/Magento/CatalogWidget/Block/Product/ProductsList.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,9 @@ public function getCacheKeyInfo()
167167
intval($this->getRequest()->getParam($this->getData('page_var_name'), 1)),
168168
$this->getProductsPerPage(),
169169
$conditions,
170-
$this->json->serialize($this->getRequest()->getParams())
170+
$this->json->serialize($this->getRequest()->getParams()),
171+
$this->getTemplate(),
172+
$this->getTitle()
171173
];
172174
}
173175

app/code/Magento/CatalogWidget/Test/Unit/Block/Product/ProductsListTest.php

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,8 @@ protected function setUp()
8787
{
8888
$this->collectionFactory =
8989
$this->getMockBuilder(\Magento\Catalog\Model\ResourceModel\Product\CollectionFactory::class)
90-
->setMethods(['create'])
91-
->disableOriginalConstructor()->getMock();
90+
->setMethods(['create'])
91+
->disableOriginalConstructor()->getMock();
9292
$this->visibility = $this->getMockBuilder(\Magento\Catalog\Model\Product\Visibility::class)
9393
->setMethods(['getVisibleInCatalogIds'])
9494
->disableOriginalConstructor()
@@ -144,6 +144,8 @@ public function testGetCacheKeyInfo()
144144
$this->productsList->setData('conditions', 'some_serialized_conditions');
145145

146146
$this->productsList->setData('page_var_name', 'page_number');
147+
$this->productsList->setTemplate('test_template');
148+
$this->productsList->setData('title', 'test_title');
147149
$this->request->expects($this->once())->method('getParam')->with('page_number')->willReturn(1);
148150

149151
$this->request->expects($this->once())->method('getParams')->willReturn('request_params');
@@ -166,7 +168,9 @@ public function testGetCacheKeyInfo()
166168
1,
167169
5,
168170
'some_serialized_conditions',
169-
json_encode('request_params')
171+
json_encode('request_params'),
172+
'test_template',
173+
'test_title'
170174
];
171175
$this->assertEquals($cacheKey, $this->productsList->getCacheKeyInfo());
172176
}
@@ -251,9 +255,10 @@ public function testGetPagerHtml()
251255
* Test public `createCollection` method and protected `getPageSize` method via `createCollection`
252256
*
253257
* @param bool $pagerEnable
254-
* @param int $productsCount
255-
* @param int $productsPerPage
256-
* @param int $expectedPageSize
258+
* @param int $productsCount
259+
* @param int $productsPerPage
260+
* @param int $expectedPageSize
261+
*
257262
* @dataProvider createCollectionDataProvider
258263
*/
259264
public function testCreateCollection($pagerEnable, $productsCount, $productsPerPage, $expectedPageSize)
@@ -382,6 +387,7 @@ public function testGetIdentities()
382387

383388
/**
384389
* @param $collection
390+
*
385391
* @return \PHPUnit_Framework_MockObject_MockObject
386392
*/
387393
private function getConditionsForCollection($collection)

0 commit comments

Comments
 (0)