make fixture accessors private

prevent to be run fixture accessor (e.g. test_foos for TestFoo model) as a test case Signed-off-by: José Valim <jose.valim@gmail.com>
rails · May 16, 2010 · e535b45 · e535b45
1 parent 3570f3e
commit e535b45
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/activerecord/lib/active_record/fixtures.rb b/activerecord/lib/active_record/fixtures.rb
@@ -891,6 +891,7 @@ def setup_fixture_accessors(table_names = nil)
 
             instances.size == 1 ? instances.first : instances
           end
+          private table_name
         end
       end
 

diff --git a/activerecord/test/cases/fixtures_test.rb b/activerecord/test/cases/fixtures_test.rb
@@ -254,6 +254,11 @@ def test_fixtures_from_root_yml_without_instantiation
     assert_nil @unknown
   end
 
+  def test_visibility_of_accessor_method
+    assert_equal false, respond_to?(:topics, false), "should be private method"
+    assert_equal true, respond_to?(:topics, true), "confirm to respond surely"
+  end
+
   def test_accessor_methods
     assert_equal "The First Topic", topics(:first).title
     assert_equal "Jamis", developers(:jamis).name