Revise test_clearing_weakrefs_in_gc test

Types are not treated specially, it only matters if the weakref has a callback or not.
nascheme · Aug 8, 2025 · Jun 22, 2025 · Jun 22, 2025 · Jun 24, 2025 · Jul 4, 2025
commit 0339d544908326dfb5a9c0c3bb138f40b654c65f
@@ -1045,30 +1045,24 @@ def callback(obj):
         self.assertNotRegex(stderr, "_Py_Dealloc: Deallocator of type 'TestObj'")
 
     def test_clearing_weakrefs_in_gc(self):
-        # This test checks that:
-        # 1. weakrefs for types with callbacks are cleared before the
-        # finalizer is called
-        # 2. weakrefs for types without callbacks are cleared after the
-        # finalizer is called
-        # 3. other weakrefs cleared before the finalizer is called
+        # This test checks that when finalizers are called:
+        # 1. weakrefs with callbacks have been cleared
+        # 2. weakrefs without callbacks have not been cleared
         errors = []
         def test():
             class Class:
                 def __init__(self):
                     self._self = self
                     self.wr1 = weakref.ref(Class, lambda x: None)
                     self.wr2 = weakref.ref(Class)
-                    self.wr3 = weakref.ref(self)
 
                 def __del__(self):
                     # we can't use assert* here, because gc will swallow
                     # exceptions
                     if self.wr1() is not None:
-                        errors.append("Type weakref with callback is not None")
+                        errors.append("weakref with callback as cleared")
                     if self.wr2() is not Class:
-                        errors.append("Type weakref is not Class")
-                    if self.wr3() is not None:
-                        errors.append("Instance weakref is not None")
+                        errors.append("weakref without callback was cleared")
 
             Class()