Skip to content
This repository
Browse code

Add tests to delete by fixnum or string id with has many through asso…

…ciations
  • Loading branch information...
commit b89553cf41e3376d46f43dafcaed99df3f7721d3 1 parent 8795ac5
Francesco Rodríguez frodsan authored

Showing 1 changed file with 20 additions and 0 deletions. Show diff stats Hide diff stats

  1. +20 0 activerecord/test/cases/associations/join_model_test.rb
20 activerecord/test/cases/associations/join_model_test.rb
@@ -581,6 +581,26 @@ def test_deleting_junk_from_has_many_through_should_raise_type_mismatch
581 581 assert_raise(ActiveRecord::AssociationTypeMismatch) { posts(:thinking).tags.delete(Object.new) }
582 582 end
583 583
  584 + def test_deleting_by_fixnum_id_from_has_many_through
  585 + post = posts(:thinking)
  586 +
  587 + assert_difference 'post.tags.count', -1 do
  588 + assert_equal 1, post.tags.delete(1).size
  589 + end
  590 +
  591 + assert_equal 0, post.tags.size
  592 + end
  593 +
  594 + def test_deleting_by_string_id_from_has_many_through
  595 + post = posts(:thinking)
  596 +
  597 + assert_difference 'post.tags.count', -1 do
  598 + assert_equal 1, post.tags.delete('1').size
  599 + end
  600 +
  601 + assert_equal 0, post.tags.size
  602 + end
  603 +
584 604 def test_has_many_through_sum_uses_calculations
585 605 assert_nothing_raised { authors(:david).comments.sum(:post_id) }
586 606 end

0 comments on commit b89553c

Please sign in to comment.
Something went wrong with that request. Please try again.