diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2012-04-30 16:43:22 -0700 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2012-04-30 16:43:22 -0700 |
commit | 206b43a954dc6c6c0b4b3916cade2561413efdb5 (patch) | |
tree | 7bf51668bbe0a13e31052433dfc4838b412b07ca /activemodel/test | |
parent | d7d379967a7bc6b2e562d271c673c3e142294224 (diff) | |
parent | 24c068d67dab4559ab24f77482cd671563161ec2 (diff) | |
download | rails-206b43a954dc6c6c0b4b3916cade2561413efdb5.tar.gz rails-206b43a954dc6c6c0b4b3916cade2561413efdb5.tar.bz2 rails-206b43a954dc6c6c0b4b3916cade2561413efdb5.zip |
Merge pull request #6063 from marcandre/observer_extra_args
Allow extra arguments for Observers
Diffstat (limited to 'activemodel/test')
-rw-r--r-- | activemodel/test/cases/observing_test.rb | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/activemodel/test/cases/observing_test.rb b/activemodel/test/cases/observing_test.rb index 9891be8d77..c91938a7ee 100644 --- a/activemodel/test/cases/observing_test.rb +++ b/activemodel/test/cases/observing_test.rb @@ -14,8 +14,8 @@ class FooObserver < ActiveModel::Observer attr_accessor :stub - def on_spec(record) - stub.event_with(record) if stub + def on_spec(record, *args) + stub.event_with(record, *args) if stub end def around_save(record) @@ -141,6 +141,13 @@ class ObserverTest < ActiveModel::TestCase Foo.send(:notify_observers, :on_spec, foo) end + test "passes extra arguments" do + foo = Foo.new + FooObserver.instance.stub = stub + FooObserver.instance.stub.expects(:event_with).with(foo, :bar) + Foo.send(:notify_observers, :on_spec, foo, :bar) + end + test "skips nonexistent observer event" do foo = Foo.new Foo.send(:notify_observers, :whatever, foo) |