diff options
| author | Yves Senn <yves.senn@gmail.com> | 2014-11-07 09:00:28 +0100 |
|---|---|---|
| committer | Yves Senn <yves.senn@gmail.com> | 2014-11-07 09:03:25 +0100 |
| commit | 888ea6bf4452971bdb32a92e854116c8cd053497 (patch) | |
| tree | 7f9298266904c874b63cbebc8dbf6c04d8827ec5 /activerecord/test | |
| parent | 9a517b5a0b97b8872af1e66b2a5e3fd765f0d0a1 (diff) | |
| parent | d424ded6fd4b923802e9dab62b463a6f3439417f (diff) | |
| download | rails-888ea6bf4452971bdb32a92e854116c8cd053497.tar.gz rails-888ea6bf4452971bdb32a92e854116c8cd053497.tar.bz2 rails-888ea6bf4452971bdb32a92e854116c8cd053497.zip | |
Merge pull request #17541 from OpenSourceProjects/meaninful_message
Print out a meaningful error when ActiveRecord::ReadOnlyRecord is raised
Diffstat (limited to 'activerecord/test')
| -rw-r--r-- | activerecord/test/cases/readonly_test.rb | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/activerecord/test/cases/readonly_test.rb b/activerecord/test/cases/readonly_test.rb index 2afd25c989..5b903e4f2c 100644 --- a/activerecord/test/cases/readonly_test.rb +++ b/activerecord/test/cases/readonly_test.rb @@ -22,9 +22,15 @@ class ReadOnlyTest < ActiveRecord::TestCase assert !dev.save dev.name = 'Forbidden.' end - assert_raise(ActiveRecord::ReadOnlyRecord) { dev.save } - assert_raise(ActiveRecord::ReadOnlyRecord) { dev.save! } - assert_raise(ActiveRecord::ReadOnlyRecord) { dev.destroy } + + e = assert_raise(ActiveRecord::ReadOnlyRecord) { dev.save } + assert_equal "Developer is marked as readonly", e.message + + e = assert_raise(ActiveRecord::ReadOnlyRecord) { dev.save! } + assert_equal "Developer is marked as readonly", e.message + + e = assert_raise(ActiveRecord::ReadOnlyRecord) { dev.destroy } + assert_equal "Developer is marked as readonly", e.message end |