Skip to content

Commit 7a42f42

Browse files
committed
MAGETWO-49469: Contribute Ogre Bugfixes and Stories for Sprint 44
- Fixing test
1 parent 6a3f1ce commit 7a42f42

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

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

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -170,14 +170,13 @@ public function __construct(
170170
$this->searchRequestName = $searchRequestName;
171171
}
172172

173-
174173
/**
175174
* @deprecated
176175
* @return \Magento\Search\Api\SearchInterface
177176
*/
178177
private function getSearch()
179178
{
180-
if ($this->search !== null) {
179+
if ($this->search === null) {
181180
$this->search = ObjectManager::getInstance()->get('\Magento\Search\Api\SearchInterface');
182181
}
183182
return $this->search;
@@ -194,10 +193,11 @@ public function setSearch(\Magento\Search\Api\SearchInterface $object)
194193

195194
/**
196195
* @deprecated
196+
* @return \Magento\Framework\Api\Search\SearchCriteriaBuilder
197197
*/
198198
private function getSearchCriteriaBuilder()
199199
{
200-
if ($this->searchCriteriaBuilder !== null) {
200+
if ($this->searchCriteriaBuilder === null) {
201201
$this->searchCriteriaBuilder = ObjectManager::getInstance()
202202
->get('\Magento\Framework\Api\Search\SearchCriteriaBuilder');
203203
}
@@ -208,18 +208,18 @@ private function getSearchCriteriaBuilder()
208208
* @deprecated
209209
* @param \Magento\Framework\Api\Search\SearchCriteriaBuilder $object
210210
*/
211-
public function setSearchCriteriaBuilder($object)
211+
public function setSearchCriteriaBuilder(\Magento\Framework\Api\Search\SearchCriteriaBuilder $object)
212212
{
213213
$this->searchCriteriaBuilder = $object;
214214
}
215215

216-
217216
/**
218217
* @deprecated
218+
* @return \Magento\Framework\Api\FilterBuilder
219219
*/
220220
private function getFilterBuilder()
221221
{
222-
if ($this->filterBuilder !== null) {
222+
if ($this->filterBuilder === null) {
223223
$this->filterBuilder = ObjectManager::getInstance()->get('\Magento\Framework\Api\FilterBuilder');
224224
}
225225
return $this->filterBuilder;
@@ -229,7 +229,7 @@ private function getFilterBuilder()
229229
* @deprecated
230230
* @param \Magento\Framework\Api\FilterBuilder $object
231231
*/
232-
public function setFilterBuilder($object)
232+
public function setFilterBuilder(\Magento\Framework\Api\FilterBuilder $object)
233233
{
234234
$this->filterBuilder = $object;
235235
}

app/code/Magento/CatalogSearch/Test/Unit/Model/ResourceModel/Fulltext/CollectionTest.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,14 @@ protected function setUp()
3838
'storeManager' => $storeManager,
3939
'universalFactory' => $universalFactory,
4040
'scopeConfig' => $scopeConfig,
41-
'searchCriteriaBuilder' => $criteriaBuilder,
42-
'filterBuilder' => $filterBuilder,
4341
]
4442
);
43+
44+
$search = $this->getMockForAbstractClass('\Magento\Search\Api\SearchInterface');
45+
$this->model->setSearchCriteriaBuilder($criteriaBuilder);
46+
$this->model->setSearch($search);
47+
$this->model->setFilterBuilder($filterBuilder);
48+
4549
}
4650

4751
/**

0 commit comments

Comments
 (0)