Changeset 8875

Timestamp:

02/15/08 23:09:51 (9 months ago)

Author:

nzkoz

Message:

Add test for Observer#observer_class and change the implementation slightly. Closes #11099 [ernesto.jimenez]

Files:

• trunk/activerecord/lib/active_record/observer.rb (modified) (1 diff)
• trunk/activerecord/test/cases/lifecycle_test.rb (modified) (1 diff)

trunk/activerecord/lib/active_record/observer.rb

@@ -152 +152 @@
-
-      # The class observed by default is inferred from the observer's class name:
-[Person]
+Person
-      def observed_class
-name.scan(/(.*)Observer/)[0
-[0]
+/(.*)Observer/.match(name)[1
+
-        else
-          nil

trunk/activerecord/test/cases/lifecycle_test.rb

@@ -135 +135 @@
-    assert_raise(ArgumentError) { Topic.observers = Object.new; Topic.instantiate_observers }
-  end
+
+  def test_getting_observed_class_from_class_name
+    assert_equal Topic, TopicObserver.observed_class
+  end
-end