]> granicus.if.org Git - python/commitdiff
bpo-36155: Check for identity on test_gc.test_get_objects (GH-12116)
authorPablo Galindo <Pablogsal@gmail.com>
Fri, 1 Mar 2019 09:12:27 +0000 (01:12 -0800)
committerGitHub <noreply@github.com>
Fri, 1 Mar 2019 09:12:27 +0000 (01:12 -0800)
Lib/test/test_gc.py

index 65e74d4759ce3ad336ac80291a41926ddf6b9426..2dab53004452aefc7dbb9286af43d95b574827c8 100644 (file)
@@ -770,21 +770,45 @@ class GCTests(unittest.TestCase):
         gc.collect()
         l = []
         l.append(l)
-        self.assertIn(l, gc.get_objects(generation=0))
-        self.assertNotIn(l, gc.get_objects(generation=1))
-        self.assertNotIn(l, gc.get_objects(generation=2))
+        self.assertTrue(
+                any(l is element for element in gc.get_objects(generation=0))
+        )
+        self.assertFalse(
+                any(l is element for element in  gc.get_objects(generation=1))
+        )
+        self.assertFalse(
+                any(l is element for element in gc.get_objects(generation=2))
+        )
         gc.collect(generation=0)
-        self.assertNotIn(l, gc.get_objects(generation=0))
-        self.assertIn(l, gc.get_objects(generation=1))
-        self.assertNotIn(l, gc.get_objects(generation=2))
+        self.assertFalse(
+                any(l is element for element in gc.get_objects(generation=0))
+        )
+        self.assertTrue(
+                any(l is element for element in  gc.get_objects(generation=1))
+        )
+        self.assertFalse(
+                any(l is element for element in gc.get_objects(generation=2))
+        )
         gc.collect(generation=1)
-        self.assertNotIn(l, gc.get_objects(generation=0))
-        self.assertNotIn(l, gc.get_objects(generation=1))
-        self.assertIn(l, gc.get_objects(generation=2))
+        self.assertFalse(
+                any(l is element for element in gc.get_objects(generation=0))
+        )
+        self.assertFalse(
+                any(l is element for element in  gc.get_objects(generation=1))
+        )
+        self.assertTrue(
+                any(l is element for element in gc.get_objects(generation=2))
+        )
         gc.collect(generation=2)
-        self.assertNotIn(l, gc.get_objects(generation=0))
-        self.assertNotIn(l, gc.get_objects(generation=1))
-        self.assertIn(l, gc.get_objects(generation=2))
+        self.assertFalse(
+                any(l is element for element in gc.get_objects(generation=0))
+        )
+        self.assertFalse(
+                any(l is element for element in  gc.get_objects(generation=1))
+        )
+        self.assertTrue(
+                any(l is element for element in gc.get_objects(generation=2))
+        )
         del l
         gc.collect()