diff options
| author | Franky W <frankywahl@users.noreply.github.com> | 2014-11-06 09:07:29 -0800 |
|---|---|---|
| committer | Franky W <frankywahl@users.noreply.github.com> | 2014-11-06 19:38:58 -0800 |
| commit | d424ded6fd4b923802e9dab62b463a6f3439417f (patch) | |
| tree | 88c054c1cede1b3abe8d22615f4ea2d82eee769d /activerecord/test/cases | |
| parent | 9a517b5a0b97b8872af1e66b2a5e3fd765f0d0a1 (diff) | |
| download | rails-d424ded6fd4b923802e9dab62b463a6f3439417f.tar.gz rails-d424ded6fd4b923802e9dab62b463a6f3439417f.tar.bz2 rails-d424ded6fd4b923802e9dab62b463a6f3439417f.zip | |
Print out a meaningful error when ActiveRecord::ReadOnlyRecord is raised
Currently, there is no messages which get printed out. Convoluted system
may have hooks that create other objects in which case we only fail with
no messages. This commit changes this information allowing you to know
which object is the one that actually raised the error.
Diffstat (limited to 'activerecord/test/cases')
| -rw-r--r-- | activerecord/test/cases/readonly_test.rb | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/activerecord/test/cases/readonly_test.rb b/activerecord/test/cases/readonly_test.rb index 2afd25c989..37d277eaa1 100644 --- a/activerecord/test/cases/readonly_test.rb +++ b/activerecord/test/cases/readonly_test.rb @@ -22,9 +22,12 @@ 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 |