diff options
| author | Yuki Nishijima <mail@yukinishijima.net> | 2015-01-06 04:31:08 -0800 |
|---|---|---|
| committer | Yuki Nishijima <mail@yukinishijima.net> | 2015-05-01 19:01:41 -0700 |
| commit | ad5824bde0f63d5bb2e33b7fe86904d1fbd7ff95 (patch) | |
| tree | 6a3dc7ce4fce9afe9e54c11d9a573bff8d3297fe /activerecord/test/cases/associations/has_many_associations_test.rb | |
| parent | 2e7fd4a8661960434ccaad71c8afb4ef26281d21 (diff) | |
| download | rails-ad5824bde0f63d5bb2e33b7fe86904d1fbd7ff95.tar.gz rails-ad5824bde0f63d5bb2e33b7fe86904d1fbd7ff95.tar.bz2 rails-ad5824bde0f63d5bb2e33b7fe86904d1fbd7ff95.zip | |
AR::RecordNotSaved & RecordNotDestroyed should include an error message
When `AR::Base.save!` or `AR::Base.destroy!` is called and an exception
is raised, the exception doesn't have any error message or has a weird
message like `#<FailedBulb:0x0000000907b4b8>`. Give a better message so
we can easily understand why it's failing to save/destroy.
Diffstat (limited to 'activerecord/test/cases/associations/has_many_associations_test.rb')
| -rw-r--r-- | activerecord/test/cases/associations/has_many_associations_test.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/activerecord/test/cases/associations/has_many_associations_test.rb b/activerecord/test/cases/associations/has_many_associations_test.rb index 14cdf37f46..2c4e2a875c 100644 --- a/activerecord/test/cases/associations/has_many_associations_test.rb +++ b/activerecord/test/cases/associations/has_many_associations_test.rb @@ -2131,11 +2131,12 @@ class HasManyAssociationsTest < ActiveRecord::TestCase car = Car.create! original_child = FailedBulb.create!(car: car) - assert_raise(ActiveRecord::RecordNotDestroyed) do + error = assert_raise(ActiveRecord::RecordNotDestroyed) do car.failed_bulbs = [FailedBulb.create!] end assert_equal [original_child], car.reload.failed_bulbs + assert_equal "Failed to destroy the record", error.message end test 'updates counter cache when default scope is given' do |