aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases
diff options
context:
space:
mode:
authorCraig Demyanovich <cdemyanovich@mutuallyhuman.com>2008-06-03 13:38:00 -0500
committerJoshua Peek <josh@joshpeek.com>2008-06-03 13:38:00 -0500
commitaa1771668877f20ca044e8f45a9736fbb7c8402e (patch)
tree8144566f85fcff4c94e6ccd993ad35c6d4688a1c /activerecord/test/cases
parenta977f3e88e989cd7d795c46504451b4b1b4db79d (diff)
downloadrails-aa1771668877f20ca044e8f45a9736fbb7c8402e.tar.gz
rails-aa1771668877f20ca044e8f45a9736fbb7c8402e.tar.bz2
rails-aa1771668877f20ca044e8f45a9736fbb7c8402e.zip
Callbacks fire before notifying observers [#230 state:resolved]
Signed-off-by: Joshua Peek <josh@joshpeek.com>
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r--activerecord/test/cases/callbacks_observers_test.rb38
1 files changed, 38 insertions, 0 deletions
diff --git a/activerecord/test/cases/callbacks_observers_test.rb b/activerecord/test/cases/callbacks_observers_test.rb
new file mode 100644
index 0000000000..87de524923
--- /dev/null
+++ b/activerecord/test/cases/callbacks_observers_test.rb
@@ -0,0 +1,38 @@
+require "cases/helper"
+
+class Comment < ActiveRecord::Base
+ attr_accessor :callers
+
+ before_validation :record_callers
+
+ def after_validation
+ record_callers
+ end
+
+ def record_callers
+ callers << self.class if callers
+ end
+end
+
+class CommentObserver < ActiveRecord::Observer
+ attr_accessor :callers
+
+ def after_validation(model)
+ callers << self.class if callers
+ end
+end
+
+class CallbacksObserversTest < ActiveRecord::TestCase
+ def test_model_callbacks_fire_before_observers_are_notified
+ callers = []
+
+ comment = Comment.new
+ comment.callers = callers
+
+ CommentObserver.instance.callers = callers
+
+ comment.valid?
+
+ assert_equal [Comment, Comment, CommentObserver], callers, "model callbacks did not fire before observers were notified"
+ end
+end