Merge pull request #3048 from magento-epam/EPAM-PR-3

[epam] MAGETWO-91620: It is not possible to use function setInAllAttributeSetsFilter() together with getAllIds() for class \Magento\Catalog\Model\ResourceModel\Product\Attribute\Collection

Author: rganin

app/code/Magento/Eav/Model/ResourceModel/Entity/Attribute/Collection.php

@@ -222,7 +222,7 @@ public function setInAllAttributeSetsFilter(array $setIds)
                     $setIds
                 )
                 ->group('entity_attribute.attribute_id')
-                ->having('count = ' . count($setIds));
+                ->having(new \Zend_Db_Expr('COUNT(*)') . ' = ' . count($setIds));
         }
 
         //$this->getSelect()->distinct(true);

app/code/Magento/Eav/Test/Unit/Model/ResourceModel/Entity/Attribute/CollectionTest.php

@@ -150,8 +150,8 @@ public function testSetInAllAttributeSetsFilter()
         $this->selectMock->expects($this->atLeastOnce())->method('group')->with('entity_attribute.attribute_id')
             ->willReturnSelf();
 
-        $this->selectMock->expects($this->atLeastOnce())->method('having')->with('count = ' . count($setIds))
-            ->willReturnSelf();
+        $this->selectMock->expects($this->atLeastOnce())->method('having')
+            ->with(new \Zend_Db_Expr('COUNT(*)') . ' = ' . count($setIds))->willReturnSelf();
 
         $this->model->setInAllAttributeSetsFilter($setIds);
     }

dev/tests/integration/testsuite/Magento/Eav/Model/ResourceModel/Entity/Attribute/CollectionTest.php

@@ -52,6 +52,20 @@ public function testSetAttributeGroupFilter()
         $this->assertEquals([$includeGroupId], $groups);
     }
 
+    /**
+     * Test if getAllIds method return results after using setInAllAttributeSetsFilter method
+     *
+     * @covers \Magento\Eav\Model\ResourceModel\Entity\Attribute\Collection::setInAllAttributeSetsFilter()
+     * @covers \Magento\Eav\Model\ResourceModel\Entity\Attribute\Collection::getAllIds()
+     */
+    public function testSetInAllAttributeSetsFilterWithGetAllIds()
+    {
+        $sets = [1];
+        $this->_model->setInAllAttributeSetsFilter($sets);
+        $attributeIds = $this->_model->getAllIds();
+        $this->assertGreaterThan(0, count($attributeIds));
+    }
+
     /**
      * Returns array of group ids, present in collection attributes
      *