MAGETWO-75935: Search results is not the same for the same search queries

Author: AVoskoboinikov

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

@@ -364,6 +364,13 @@ protected function _renderFiltersBefore()
         if ($this->order && 'relevance' === $this->order['field']) {
             $this->getSelect()->order('search_result.'. TemporaryStorage::FIELD_SCORE . ' ' . $this->order['dir']);
         }
+
+        /*
+         * This order is required to force search results be the same
+         * for the same requests and products with the same relevance
+         */
+        $this->setOrder('entity_id');
+
         return parent::_renderFiltersBefore();
     }
 

dev/tests/integration/testsuite/Magento/CatalogSearch/Model/ResourceModel/Fulltext/CollectionTest.php

@@ -31,6 +31,37 @@ public function testLoadWithFilterSearch($request, $filters, $expectedCount)
         $this->assertCount($expectedCount, $items);
     }
 
+    /**
+     * @magentoDataFixture Magento/Framework/Search/_files/products_with_the_same_search_score.php
+     */
+    public function testSearchResultsAreTheSameForSameRequests()
+    {
+        $howManySearchRequests = 3;
+        $previousResult = null;
+
+        $objManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
+
+        foreach (range(1, $howManySearchRequests) as $_) {
+            /** @var  \Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection $fulltextCollection */
+            $fulltextCollection = $objManager->create(
+                \Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection::class,
+                ['searchRequestName' => 'quick_search_container']
+            );
+
+            $fulltextCollection->addFieldToFilter('search_term', 'shorts');
+            $fulltextCollection->setOrder('relevance');
+            $fulltextCollection->load();
+            $items = $fulltextCollection->getItems();
+            $this->assertGreaterThan(0, count($items), "Search result must not be empty");
+
+            if ($previousResult) {
+                $this->assertEquals($previousResult, array_keys($items), "Search result must be the same for the same requests");
+            }
+
+            $previousResult = array_keys($items);
+        }
+    }
+
     public function filtersDataProviderSearch()
     {
         return [
@@ -42,4 +73,4 @@ public function filtersDataProviderSearch()
             ['catalog_view_container', [], 0],
         ];
     }
-}
+}

dev/tests/integration/testsuite/Magento/Framework/Search/_files/products_with_the_same_search_score.php

@@ -0,0 +1,45 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+use Magento\Catalog\Api\ProductRepositoryInterface;
+use Magento\Catalog\Model\Product;
+use Magento\Catalog\Model\Product\Attribute\Source\Status;
+use Magento\Catalog\Model\Product\Type;
+use Magento\Catalog\Model\Product\Visibility;
+use Magento\TestFramework\Helper\Bootstrap;
+
+Bootstrap::getInstance()->reinitialize();
+
+/** @var ProductRepositoryInterface $productRepository */
+$productRepository = Bootstrap::getObjectManager()
+    ->create(ProductRepositoryInterface::class);
+
+$howManyProducts = 5;
+
+foreach (range(1, $howManyProducts) as $productId) {
+    $product = Bootstrap::getObjectManager()->create(Product::class);
+    $product->setTypeId(Type::TYPE_SIMPLE)
+        ->setAttributeSetId(4)
+        ->setWebsiteIds([1])
+        ->setName('Cool short' . $productId)
+        ->setSku('cool_shorts_' . $productId)
+        ->setPrice(42)
+        ->setShortDescription("Some description about shorts")
+        ->setTaxClassId(0)
+        ->setDescription('Some description about <b>shorts</b>')
+        ->setVisibility(Visibility::VISIBILITY_BOTH)
+        ->setStatus(Status::STATUS_ENABLED)
+        ->setStockData(
+            [
+                'use_config_manage_stock' => 1,
+                'qty' => 100,
+                'is_qty_decimal' => 0,
+                'is_in_stock' => 1,
+            ]
+        );
+
+    $productRepository->save($product);
+}

dev/tests/integration/testsuite/Magento/Framework/Search/_files/products_with_the_same_search_score_rollback.php

@@ -0,0 +1,21 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+/** @var \Magento\Framework\Registry $registry */
+$objectManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
+$registry = $objectManager->get(\Magento\Framework\Registry::class);
+
+$registry->unregister('isSecureArea');
+$registry->register('isSecureArea', true);
+
+/** @var \Magento\Catalog\Model\ResourceModel\Product\Collection $collection */
+$collection = $objectManager->create(\Magento\Catalog\Model\ResourceModel\Product\Collection::class);
+$collection->addAttributeToSelect('id')->load();
+if ($collection->count() > 0) {
+    $collection->delete();
+}
+
+$registry->unregister('isSecureArea');
+$registry->register('isSecureArea', false);