Alias ActiveSupport::OrderedHash#update to ActiveSupport::OrderedHash.merge!

This ensures that an OrderedHash's keys are set up appropriately when using update.

[#4973 state:committed]

Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
This commit is contained in:
Paul Mucur 2010-06-25 15:35:11 +01:00 committed by Jeremy Kemper
parent 5e2b473b47
commit 158e22dae0
2 changed files with 8 additions and 0 deletions

View File

@ -149,6 +149,8 @@ module ActiveSupport
self
end
alias_method :update, :merge!
def merge(other_hash, &block)
dup.merge!(other_hash, &block)
end

View File

@ -244,4 +244,10 @@ class OrderedHashTest < Test::Unit::TestCase
assert_equal @ordered_hash.keys, @deserialized_ordered_hash.keys
assert_equal @ordered_hash.values, @deserialized_ordered_hash.values
end
def test_update_sets_keys
@updated_ordered_hash = ActiveSupport::OrderedHash.new
@updated_ordered_hash.update(:name => "Bob")
assert_equal [:name], @updated_ordered_hash.keys
end
end