提交 24b690ea 编写于 作者: R Rafael Mendonça França

Improve the test code for #8422

* Remove unneeded comments.
* Use key? instead of keys.include?.
上级 6ed5d515
......@@ -5,13 +5,13 @@
class DescendantsTrackerWithoutAutoloadingTest < ActiveSupport::TestCase
include DescendantsTrackerTestCases
# Regression test for #8422. https://github.com/rails/rails/issues/8442
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)
parent_instance.singleton_class.descendants
ActiveSupport::DescendantsTracker.clear
assert !ActiveSupport::DescendantsTracker.class_variable_get(:@@direct_descendants).key?(parent_instance.singleton_class)
end
end
end
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册