Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

No need for instance vars on single tests

  • Loading branch information...
commit 07ff3525d5275de47960c3a012a1fa261bbfdfc8 1 parent da4b5e8
@carlosantoniodasilva carlosantoniodasilva authored
Showing with 8 additions and 8 deletions.
  1. +8 −8 activerecord/test/cases/transaction_callbacks_test.rb
View
16 activerecord/test/cases/transaction_callbacks_test.rb
@@ -93,11 +93,11 @@ def test_only_call_after_commit_on_destroy_after_transaction_commits_for_destroy
end
def test_only_call_after_commit_on_create_after_transaction_commits_for_new_record
- @new_record = TopicWithCallbacks.new(:title => "New topic", :written_on => Date.today)
- add_transaction_execution_blocks @new_record
+ new_record = TopicWithCallbacks.new(:title => "New topic", :written_on => Date.today)
+ add_transaction_execution_blocks new_record
- @new_record.save!
- assert_equal [:commit_on_create], @new_record.history
+ new_record.save!
+ assert_equal [:commit_on_create], new_record.history
end
def test_only_call_after_commit_on_create_after_transaction_commits_for_new_record_if_create_succeeds_creating_through_association
@@ -160,15 +160,15 @@ def test_only_call_after_rollback_on_destroy_after_transaction_rollsback_for_des
end
def test_only_call_after_rollback_on_create_after_transaction_rollsback_for_new_record
- @new_record = TopicWithCallbacks.new(:title => "New topic", :written_on => Date.today)
- add_transaction_execution_blocks @new_record
+ new_record = TopicWithCallbacks.new(:title => "New topic", :written_on => Date.today)
+ add_transaction_execution_blocks new_record
Topic.transaction do
- @new_record.save!
+ new_record.save!
raise ActiveRecord::Rollback
end
- assert_equal [:rollback_on_create], @new_record.history
+ assert_equal [:rollback_on_create], new_record.history
end
def test_call_after_rollback_when_commit_fails
Please sign in to comment.
Something went wrong with that request. Please try again.