Browse files

Test for after_create callback order in ActiveSupport [#5703 state:re…

…solved]

Signed-off-by: José Valim <jose.valim@gmail.com>
  • Loading branch information...
1 parent 72c1e19 commit bfc986811c5cbcfcc856916a1b09fcbf6551ecf5 Neeraj Singh committed with josevalim Sep 25, 2010
Showing with 23 additions and 0 deletions.
  1. +23 −0 activesupport/test/callbacks_test.rb
View
23 activesupport/test/callbacks_test.rb
@@ -149,6 +149,27 @@ def test_optimized_first_compile
end
end
+ class AfterSaveConditionalPerson < Record
+ after_save Proc.new { |r| r.history << [:after_save, :string1] }
+ after_save Proc.new { |r| r.history << [:after_save, :string2] }
+ def save
+ run_callbacks :save
+ end
+ end
+
+ class AfterSaveConditionalPersonCallbackTest < Test::Unit::TestCase
+ def test_after_save_runs_in_the_reverse_order
+ person = AfterSaveConditionalPerson.new
+ person.save
+ assert_equal [
+ [:after_save, :string2],
+ [:after_save, :string1]
+ ], person.history
+ end
+ end
+
+
+
class ConditionalPerson < Record
# proc
before_save Proc.new { |r| r.history << [:before_save, :proc] }, :if => Proc.new { |r| true }
@@ -352,6 +373,8 @@ def test_save_conditional_person
end
end
+
+
class ResetCallbackTest < Test::Unit::TestCase
def test_save_conditional_person
person = CleanPerson.new

0 comments on commit bfc9868

Please sign in to comment.