From 569fb1fffb216ad96721fe1f5d706535e9920154 Mon Sep 17 00:00:00 2001 From: Marc-Andre Lafortune Date: Sun, 29 Apr 2012 00:51:56 -0400 Subject: Generate appropriate error more judiciously --- activemodel/test/cases/observing_test.rb | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'activemodel/test') diff --git a/activemodel/test/cases/observing_test.rb b/activemodel/test/cases/observing_test.rb index df070d2d8e..1a80b5d854 100644 --- a/activemodel/test/cases/observing_test.rb +++ b/activemodel/test/cases/observing_test.rb @@ -70,6 +70,21 @@ class ObservingTest < ActiveModel::TestCase ObservedModel.instantiate_observers end + test "raises an appropriate error when a developer accidentally adds the wrong class (i.e. Widget instead of WidgetObserver)" do + assert_raise ArgumentError do + ObservedModel.observers = ['string'] + ObservedModel.instantiate_observers + end + assert_raise ArgumentError do + ObservedModel.observers = [:string] + ObservedModel.instantiate_observers + end + assert_raise ArgumentError do + ObservedModel.observers = [String] + ObservedModel.instantiate_observers + end + end + test "passes observers to subclasses" do FooObserver.instance bar = Class.new(Foo) -- cgit v1.2.3