From c712b082c54157e31e4f959f110c8ede11315b55 Mon Sep 17 00:00:00 2001 From: Iuri Gagnidze Date: Sun, 9 Dec 2012 16:56:10 -0600 Subject: [PATCH] Added test to ensure that DescendantsTracker does not leak memory on singleton classes --- .../descendants_tracker_without_autoloading_test.rb | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/activesupport/test/descendants_tracker_without_autoloading_test.rb b/activesupport/test/descendants_tracker_without_autoloading_test.rb index 74669aaca1..4f1ee3e760 100644 --- a/activesupport/test/descendants_tracker_without_autoloading_test.rb +++ b/activesupport/test/descendants_tracker_without_autoloading_test.rb @@ -4,4 +4,14 @@ class DescendantsTrackerWithoutAutoloadingTest < ActiveSupport::TestCase include DescendantsTrackerTestCases + + def test_clear_without_autoloaded_singleton_parent + mark_as_autoloaded do + parent_instance = Parent.new + parent_instance.singleton_class.descendants #adds singleton class in @@direct_descendants + ActiveSupport::DescendantsTracker.clear #clear is supposed to remove singleton class keys so GC can remove them. + assert !ActiveSupport::DescendantsTracker.class_variable_get(:@@direct_descendants).keys.include?(parent_instance.singleton_class) + end + end + end -- GitLab