Skip to content

Commit 2884972

Browse files
committed
ENGCOM-3502: Unit test fix.
1 parent 97244a1 commit 2884972

File tree

4 files changed

+52
-11
lines changed

4 files changed

+52
-11
lines changed

app/code/Magento/CatalogSearch/Model/ResourceModel/Advanced/Collection.php

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,23 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6+
67
namespace Magento\CatalogSearch\Model\ResourceModel\Advanced;
78

9+
use Magento\Catalog\Model\Indexer\Category\Product\TableMaintainer;
10+
use Magento\Catalog\Model\Indexer\Product\Price\PriceTableResolver;
811
use Magento\Catalog\Model\Product;
12+
use Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitationFactory;
913
use Magento\Framework\Api\FilterBuilder;
1014
use Magento\Framework\Api\Search\SearchCriteriaBuilder;
1115
use Magento\Framework\Api\Search\SearchResultFactory;
1216
use Magento\Framework\EntityManager\MetadataPool;
1317
use Magento\Framework\Exception\LocalizedException;
18+
use Magento\Framework\Indexer\DimensionFactory;
1419
use Magento\Framework\Model\ResourceModel\ResourceModelPoolInterface;
1520
use Magento\Framework\Search\Adapter\Mysql\TemporaryStorage;
1621
use Magento\Framework\Search\Request\EmptyRequestDataException;
1722
use Magento\Framework\Search\Request\NonExistingRequestNameException;
18-
use Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitationFactory;
1923

2024
/**
2125
* Advanced search collection
@@ -88,8 +92,10 @@ class Collection extends \Magento\Catalog\Model\ResourceModel\Product\Collection
8892
* @param \Magento\Framework\DB\Adapter\AdapterInterface|null $connection
8993
* @param SearchResultFactory|null $searchResultFactory
9094
* @param ProductLimitationFactory|null $productLimitationFactory
91-
* @param MetadataPool|null $metadataPool
92-
*
95+
* @param MetadataPool|null $metadataPool *
96+
* @param TableMaintainer|null $tableMaintainer
97+
* @param PriceTableResolver|null $priceTableResolver
98+
* @param DimensionFactory|null $dimensionFactory
9399
* @param ResourceModelPoolInterface|null $resourceModelPool
94100
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
95101
*/
@@ -120,6 +126,9 @@ public function __construct(
120126
SearchResultFactory $searchResultFactory = null,
121127
ProductLimitationFactory $productLimitationFactory = null,
122128
MetadataPool $metadataPool = null,
129+
TableMaintainer $tableMaintainer = null,
130+
PriceTableResolver $priceTableResolver = null,
131+
DimensionFactory $dimensionFactory = null,
123132
ResourceModelPoolInterface $resourceModelPool = null
124133
) {
125134
$this->requestBuilder = $requestBuilder;
@@ -152,6 +161,9 @@ public function __construct(
152161
$connection,
153162
$productLimitationFactory,
154163
$metadataPool,
164+
$tableMaintainer,
165+
$priceTableResolver,
166+
$dimensionFactory,
155167
$resourceModelPool
156168
);
157169
}

app/code/Magento/CatalogSearch/Model/ResourceModel/Fulltext/Collection.php

Lines changed: 19 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,21 +3,25 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6+
67
namespace Magento\CatalogSearch\Model\ResourceModel\Fulltext;
78

9+
use Magento\Catalog\Model\Indexer\Category\Product\TableMaintainer;
10+
use Magento\Catalog\Model\Indexer\Product\Price\PriceTableResolver;
11+
use Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitationFactory;
812
use Magento\CatalogSearch\Model\Search\RequestGenerator;
13+
use Magento\Framework\Api\Search\SearchResultFactory;
14+
use Magento\Framework\App\ObjectManager;
915
use Magento\Framework\DB\Select;
1016
use Magento\Framework\EntityManager\MetadataPool;
17+
use Magento\Framework\Exception\LocalizedException;
1118
use Magento\Framework\Exception\StateException;
19+
use Magento\Framework\Indexer\DimensionFactory;
1220
use Magento\Framework\Model\ResourceModel\ResourceModelPoolInterface;
1321
use Magento\Framework\Search\Adapter\Mysql\TemporaryStorage;
14-
use Magento\Framework\Search\Response\QueryResponse;
1522
use Magento\Framework\Search\Request\EmptyRequestDataException;
1623
use Magento\Framework\Search\Request\NonExistingRequestNameException;
17-
use Magento\Framework\Api\Search\SearchResultFactory;
18-
use Magento\Framework\Exception\LocalizedException;
19-
use Magento\Framework\App\ObjectManager;
20-
use Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitationFactory;
24+
use Magento\Framework\Search\Response\QueryResponse;
2125

2226
/**
2327
* Fulltext Collection
@@ -133,7 +137,9 @@ class Collection extends \Magento\Catalog\Model\ResourceModel\Product\Collection
133137
* @param SearchResultFactory|null $searchResultFactory
134138
* @param ProductLimitationFactory|null $productLimitationFactory
135139
* @param MetadataPool|null $metadataPool
136-
*
140+
* @param TableMaintainer|null $tableMaintainer
141+
* @param PriceTableResolver|null $priceTableResolver
142+
* @param DimensionFactory|null $dimensionFactory
137143
* @param ResourceModelPoolInterface|null $resourceModelPool
138144
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
139145
*/
@@ -166,6 +172,9 @@ public function __construct(
166172
SearchResultFactory $searchResultFactory = null,
167173
ProductLimitationFactory $productLimitationFactory = null,
168174
MetadataPool $metadataPool = null,
175+
TableMaintainer $tableMaintainer = null,
176+
PriceTableResolver $priceTableResolver = null,
177+
DimensionFactory $dimensionFactory = null,
169178
ResourceModelPoolInterface $resourceModelPool = null
170179
) {
171180
$this->queryFactory = $catalogSearchData;
@@ -196,6 +205,9 @@ public function __construct(
196205
$connection,
197206
$productLimitationFactory,
198207
$metadataPool,
208+
$tableMaintainer,
209+
$priceTableResolver,
210+
$dimensionFactory,
199211
$resourceModelPool
200212
);
201213
$this->requestBuilder = $requestBuilder;
@@ -382,7 +394,7 @@ protected function _renderFiltersBefore()
382394

383395
if ($this->relevanceOrderDirection) {
384396
$this->getSelect()->order(
385-
'search_result.'. TemporaryStorage::FIELD_SCORE . ' ' . $this->relevanceOrderDirection
397+
'search_result.' . TemporaryStorage::FIELD_SCORE . ' ' . $this->relevanceOrderDirection
386398
);
387399
}
388400
return parent::_renderFiltersBefore();

app/code/Magento/Reports/Model/ResourceModel/Product/Collection.php

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,11 @@
1111
*/
1212
namespace Magento\Reports\Model\ResourceModel\Product;
1313

14+
use Magento\Catalog\Model\Indexer\Category\Product\TableMaintainer;
15+
use Magento\Catalog\Model\Indexer\Product\Price\PriceTableResolver;
1416
use Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitationFactory;
1517
use Magento\Framework\EntityManager\MetadataPool;
18+
use Magento\Framework\Indexer\DimensionFactory;
1619
use Magento\Framework\Model\ResourceModel\ResourceModelPoolInterface;
1720

1821
/**
@@ -93,8 +96,13 @@ class Collection extends \Magento\Catalog\Model\ResourceModel\Product\Collection
9396
* @param \Magento\Catalog\Model\Product\Type $productType
9497
* @param \Magento\Quote\Model\ResourceModel\Quote\Collection $quoteResource
9598
* @param mixed $connection
96-
*
99+
* @param ProductLimitationFactory|null $productLimitationFactory
100+
* @param MetadataPool|null $metadataPool
101+
* @param TableMaintainer|null $tableMaintainer
102+
* @param PriceTableResolver|null $priceTableResolver
103+
* @param DimensionFactory|null $dimensionFactory
97104
* @param ResourceModelPoolInterface $resourceModelPool
105+
* @throws \Magento\Framework\Exception\LocalizedException
98106
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
99107
*/
100108
public function __construct(
@@ -124,6 +132,9 @@ public function __construct(
124132
\Magento\Framework\DB\Adapter\AdapterInterface $connection = null,
125133
ProductLimitationFactory $productLimitationFactory = null,
126134
MetadataPool $metadataPool = null,
135+
TableMaintainer $tableMaintainer = null,
136+
PriceTableResolver $priceTableResolver = null,
137+
DimensionFactory $dimensionFactory = null,
127138
ResourceModelPoolInterface $resourceModelPool = null
128139
) {
129140
$this->setProductEntityId($product->getEntityIdField());
@@ -152,6 +163,9 @@ public function __construct(
152163
$connection,
153164
$productLimitationFactory,
154165
$metadataPool,
166+
$tableMaintainer,
167+
$priceTableResolver,
168+
$dimensionFactory,
155169
$resourceModelPool
156170
);
157171
$this->_eventTypeFactory = $eventTypeFactory;

app/code/Magento/Reports/Test/Unit/Model/ResourceModel/Product/CollectionTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,9 @@ protected function setUp()
149149
$this->connectionMock,
150150
$this->createMock(ProductLimitationFactory::class),
151151
$this->createMock(MetadataPool::class),
152+
$this->createMock(\Magento\Catalog\Model\Indexer\Category\Product\TableMaintainer::class),
153+
$this->createMock(\Magento\Catalog\Model\Indexer\Product\Price\PriceTableResolver::class),
154+
$this->createMock(\Magento\Framework\Indexer\DimensionFactory::class),
152155
$resourceModelPoolMock
153156
);
154157
}

0 commit comments

Comments
 (0)