[2.0] Simplifying code. Including OneToManyBidirectionalAssociation test in AllTests

tests/Doctrine/Tests/Models/ECommerce/ECommerceProduct.php

@@ -89,12 +89,10 @@ class ECommerceProduct
     }
 
     public function removeFeature(ECommerceFeature $feature) {
-        foreach ($this->features as $index => $current) {
-            if ($current === $feature) {
-                unset($this->features[$index]);
-                $current->removeProduct();
-                return true;
-            }
+        $removed = $this->features->removeElement($feature);
+        if ($removed !== null) {
+            $removed->removeProduct();
+            return true;
         }
         return false;
     }

tests/Doctrine/Tests/ORM/Functional/AllTests.php

@@ -28,6 +28,7 @@ class AllTests
         $suite->addTestSuite('Doctrine\Tests\ORM\Functional\QueryTest');
         $suite->addTestSuite('Doctrine\Tests\ORM\Functional\OneToOneUnidirectionalAssociationTest');
         $suite->addTestSuite('Doctrine\Tests\ORM\Functional\OneToOneBidirectionalAssociationTest');
+        $suite->addTestSuite('Doctrine\Tests\ORM\Functional\OneToManyBidirectionalAssociationTest');
 
         return $suite;
     }