Fix CollectionAssociation#replace to return new target (closes #6231)

activerecord/lib/active_record/associations/collection_association.rb

@@ -468,6 +468,8 @@ def replace_records(new_target, original_target)
             raise RecordNotSaved, "Failed to replace #{reflection.name} because one or more of the " \
                                   "new records could not be saved."
           end
+
+          new_target
         end
 
         def concat_records(records)

activerecord/test/cases/associations/has_many_associations_test.rb

@@ -1685,6 +1685,18 @@ def test_replace
     assert_equal [bulb2], car.reload.bulbs
   end
 
+  def test_replace_returns_new_target
+    car = Car.create(:name => 'honda')
+    bulb1 = car.bulbs.create
+    bulb2 = car.bulbs.create
+    bulb3 = Bulb.create
+
+    assert_equal [bulb1, bulb2], car.bulbs
+    result = car.bulbs.replace([bulb1, bulb3])
+    assert_equal [bulb1, bulb3], car.bulbs
+    assert_equal [bulb1, bulb3], result
+  end
+
   def test_building_has_many_association_with_restrict_dependency
     option_before = ActiveRecord::Base.dependent_restrict_raises
     ActiveRecord::Base.dependent_restrict_raises = true