kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Fixes transaction callbacks tests

Browse source 
[#4640 state:committed]

Signed-off-by: wycats <wycats@gmail.com>
This commit is contained in:
Santiago Pastorino 2010-05-18 02:45:06 -03:00 committed by wycats
parent b439d85a19
commit 03c3d1eb84
1 changed files with 19 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

42
activerecord/test/cases/transaction_callbacks_test.rb
View file

@ -58,26 +58,24 @@ class TransactionCallbacksTest < ActiveRecord::TestCase
end
def test_only_call_after_commit_on_update_after_transaction_commits_for_existing_record
commit_callback = []
@first.after_commit_block(:create){|r| r.history << :commit_on_create}
@first.after_commit_block(:update){|r| r.history << :commit_on_update}
@first.after_commit_block(:destroy){|r| r.history << :commit_on_destroy}
@first.after_commit_block(:create){|r| r.history << :rollback_on_create}
@first.after_commit_block(:update){|r| r.history << :rollback_on_update}
@first.after_commit_block(:destroy){|r| r.history << :rollback_on_destroy}
@first.after_rollback_block(:create){|r| r.history << :rollback_on_create}
@first.after_rollback_block(:update){|r| r.history << :rollback_on_update}
@first.after_rollback_block(:destroy){|r| r.history << :rollback_on_destroy}
@first.save!
assert_equal [:commit_on_update], @first.history
end
def test_only_call_after_commit_on_destroy_after_transaction_commits_for_destroyed_record
commit_callback = []
@first.after_commit_block(:create){|r| r.history << :commit_on_create}
@first.after_commit_block(:update){|r| r.history << :commit_on_update}
@first.after_commit_block(:destroy){|r| r.history << :commit_on_destroy}
@first.after_commit_block(:create){|r| r.history << :rollback_on_create}
@first.after_commit_block(:update){|r| r.history << :rollback_on_update}
@first.after_commit_block(:destroy){|r| r.history << :rollback_on_destroy}
@first.after_rollback_block(:create){|r| r.history << :rollback_on_create}
@first.after_rollback_block(:update){|r| r.history << :rollback_on_update}
@first.after_rollback_block(:destroy){|r| r.history << :rollback_on_destroy}
@first.destroy
assert_equal [:commit_on_destroy], @first.history
@ -88,9 +86,9 @@ class TransactionCallbacksTest < ActiveRecord::TestCase
@new_record.after_commit_block(:create){|r| r.history << :commit_on_create}
@new_record.after_commit_block(:update){|r| r.history << :commit_on_update}
@new_record.after_commit_block(:destroy){|r| r.history << :commit_on_destroy}
@new_record.after_commit_block(:create){|r| r.history << :rollback_on_create}
@new_record.after_commit_block(:update){|r| r.history << :rollback_on_update}
@new_record.after_commit_block(:destroy){|r| r.history << :rollback_on_destroy}
@new_record.after_rollback_block(:create){|r| r.history << :rollback_on_create}
@new_record.after_rollback_block(:update){|r| r.history << :rollback_on_update}
@new_record.after_rollback_block(:destroy){|r| r.history << :rollback_on_destroy}
@new_record.save!
assert_equal [:commit_on_create], @new_record.history
@ -109,13 +107,12 @@ class TransactionCallbacksTest < ActiveRecord::TestCase
end
def test_only_call_after_rollback_on_update_after_transaction_rollsback_for_existing_record
commit_callback = []
@first.after_commit_block(:create){|r| r.history << :commit_on_create}
@first.after_commit_block(:update){|r| r.history << :commit_on_update}
@first.after_commit_block(:destroy){|r| r.history << :commit_on_destroy}
@first.after_commit_block(:create){|r| r.history << :rollback_on_create}
@first.after_commit_block(:update){|r| r.history << :rollback_on_update}
@first.after_commit_block(:destroy){|r| r.history << :rollback_on_destroy}
@first.after_rollback_block(:create){|r| r.history << :rollback_on_create}
@first.after_rollback_block(:update){|r| r.history << :rollback_on_update}
@first.after_rollback_block(:destroy){|r| r.history << :rollback_on_destroy}
Topic.transaction do
@first.save!
@ -126,13 +123,12 @@ class TransactionCallbacksTest < ActiveRecord::TestCase
end
def test_only_call_after_rollback_on_destroy_after_transaction_rollsback_for_destroyed_record
commit_callback = []
@first.after_commit_block(:create){|r| r.history << :commit_on_create}
@first.after_commit_block(:update){|r| r.history << :commit_on_update}
@first.after_commit_block(:destroy){|r| r.history << :commit_on_update}
@first.after_commit_block(:create){|r| r.history << :rollback_on_create}
@first.after_commit_block(:update){|r| r.history << :rollback_on_update}
@first.after_commit_block(:destroy){|r| r.history << :rollback_on_destroy}
@first.after_rollback_block(:create){|r| r.history << :rollback_on_create}
@first.after_rollback_block(:update){|r| r.history << :rollback_on_update}
@first.after_rollback_block(:destroy){|r| r.history << :rollback_on_destroy}
Topic.transaction do
@first.destroy
@ -147,9 +143,9 @@ class TransactionCallbacksTest < ActiveRecord::TestCase
@new_record.after_commit_block(:create){|r| r.history << :commit_on_create}
@new_record.after_commit_block(:update){|r| r.history << :commit_on_update}
@new_record.after_commit_block(:destroy){|r| r.history << :commit_on_destroy}
@new_record.after_commit_block(:create){|r| r.history << :rollback_on_create}
@new_record.after_commit_block(:update){|r| r.history << :rollback_on_update}
@new_record.after_commit_block(:destroy){|r| r.history << :rollback_on_destroy}
@new_record.after_rollback_block(:create){|r| r.history << :rollback_on_create}
@new_record.after_rollback_block(:update){|r| r.history << :rollback_on_update}
@new_record.after_rollback_block(:destroy){|r| r.history << :rollback_on_destroy}
Topic.transaction do
@new_record.save!
@ -198,7 +194,7 @@ class TransactionCallbacksTest < ActiveRecord::TestCase
assert_equal 1, @first.commits
assert_equal 0, @first.rollbacks
assert_equal 1, @second.commits
assert_equal 0, @second.commits
assert_equal 1, @second.rollbacks
end