@@ -102,11 +102,11 @@ public function testAddNonGlobalAttributeToCollectionNoProducts()
102102 $ this ->conditionProduct ->addToCollection ($ collection );
103103 $ collectedAttributes = $ this ->conditionProduct ->getRule ()->getCollectedAttributes ();
104104 $ this ->assertArrayHasKey ('visibility ' , $ collectedAttributes );
105- $ query = (string )$ collection ->getSelect ();
106- $ this ->assertStringNotContainsString ('visibility ' , $ query );
107- $ this ->assertEquals ('' , $ this ->conditionProduct ->getMappedSqlField ());
105+ $ this ->assertEquals (0 , $ collection ->getSize ());
106+ $ this ->assertStringContainsString ('visibility ' , (string )$ this ->conditionProduct ->getMappedSqlField ());
108107 $ this ->assertFalse ($ this ->conditionProduct ->hasValueParsed ());
109- $ this ->assertFalse ($ this ->conditionProduct ->hasValue ());
108+ $ this ->assertTrue ($ this ->conditionProduct ->hasValue ());
109+ $ this ->assertEquals ('4 ' , $ this ->conditionProduct ->getValue ());
110110 }
111111
112112 /**
@@ -121,9 +121,11 @@ public function testAddNonGlobalAttributeToCollection()
121121 $ this ->conditionProduct ->addToCollection ($ collection );
122122 $ collectedAttributes = $ this ->conditionProduct ->getRule ()->getCollectedAttributes ();
123123 $ this ->assertArrayHasKey ('visibility ' , $ collectedAttributes );
124- $ query = (string )$ collection ->getSelect ();
125- $ this ->assertStringNotContainsString ('visibility ' , $ query );
126- $ this ->assertEquals ('e.entity_id ' , $ this ->conditionProduct ->getMappedSqlField ());
124+ $ this ->assertEquals (1 , $ collection ->getSize ());
125+ $ this ->assertStringContainsString ('visibility ' , (string )$ this ->conditionProduct ->getMappedSqlField ());
126+ $ this ->assertFalse ($ this ->conditionProduct ->hasValueParsed ());
127+ $ this ->assertTrue ($ this ->conditionProduct ->hasValue ());
128+ $ this ->assertEquals ('4 ' , $ this ->conditionProduct ->getValue ());
127129 }
128130
129131 /**
0 commit comments