diff options
author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-08-11 19:08:16 -0700 |
---|---|---|
committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-08-11 23:24:47 -0300 |
commit | 786713a3abc375fab4d5c2f6f62a878c2ab51022 (patch) | |
tree | 70035fe7fa1d94d9e3e92726533e9a34c2570959 /activerecord/test | |
parent | 20f18cbc148270112c670189cf5489900a2271dc (diff) | |
download | rails-786713a3abc375fab4d5c2f6f62a878c2ab51022.tar.gz rails-786713a3abc375fab4d5c2f6f62a878c2ab51022.tar.bz2 rails-786713a3abc375fab4d5c2f6f62a878c2ab51022.zip |
Merge pull request #6073 from daveyeu/restore-state-on-record-invalid
Restore state on create when ActiveRecord::RecordInvalid is raised
Conflicts:
activerecord/CHANGELOG.md
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/transactions_test.rb | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/activerecord/test/cases/transactions_test.rb b/activerecord/test/cases/transactions_test.rb index 3efaaef7de..c2fc239167 100644 --- a/activerecord/test/cases/transactions_test.rb +++ b/activerecord/test/cases/transactions_test.rb @@ -204,6 +204,23 @@ class TransactionTest < ActiveRecord::TestCase end end + def test_callback_rollback_in_create_with_record_invalid_exception + begin + Topic.class_eval <<-eoruby, __FILE__, __LINE__ + 1 + remove_method(:after_create_for_transaction) + def after_create_for_transaction + raise ActiveRecord::RecordInvalid.new(Author.new) + end + eoruby + + new_topic = Topic.create(:title => "A new topic") + assert !new_topic.persisted?, "The topic should not be persisted" + assert_nil new_topic.id, "The topic should not have an ID" + ensure + remove_exception_raising_after_create_callback_to_topic + end + end + def test_nested_explicit_transactions Topic.transaction do Topic.transaction do |