kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Merge pull request #6527 from frodsan/add_test_to_delete_by_fixnum_or_string

Browse source 
Add tests to delete by fixnum or string id with has many through associations
This commit is contained in:
Carlos Antonio da Silva 2012-05-28 21:04:40 -07:00
commit 3d4ede2959
1 changed files with 20 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
activerecord/test/cases/associations/join_model_test.rb
View file

@ -581,6 +581,26 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
assert_raise(ActiveRecord::AssociationTypeMismatch) { posts(:thinking).tags.delete(Object.new) }
end
def test_deleting_by_fixnum_id_from_has_many_through
post = posts(:thinking)
assert_difference 'post.tags.count', -1 do
assert_equal 1, post.tags.delete(1).size
end
assert_equal 0, post.tags.size
end
def test_deleting_by_string_id_from_has_many_through
post = posts(:thinking)
assert_difference 'post.tags.count', -1 do
assert_equal 1, post.tags.delete('1').size
end
assert_equal 0, post.tags.size
end
def test_has_many_through_sum_uses_calculations
assert_nothing_raised { authors(:david).comments.sum(:post_id) }
end