switch to assertTrue(all(...))

ebonnal · ebonnal · commit 1c3d01ac1b69 · 2025-03-25T01:42:12.000Z
diff --git a/Lib/test/test_concurrent_futures/executor.py b/Lib/test/test_concurrent_futures/executor.py
@@ -69,9 +69,9 @@ def test_map_exception(self):
 
         # a failed future must not be captured in its
         # future._exception.__traceback__ to avoid a reference cycle
-        self.assertFalse(
-            any(
-                isinstance(referrer, futures.Future)
+        self.assertTrue(
+            all(
+                not isinstance(referrer, futures.Future)
                 for referrer in gc.get_referrers(error)
             )
         )

Original file line number	Diff line number	Diff line change
`@@ -69,9 +69,9 @@ def test_map_exception(self):`
`69`	`69`
`70`	`70`	`# a failed future must not be captured in its`
`71`	`71`	`# future._exception.__traceback__ to avoid a reference cycle`
`72`		`- self.assertFalse(`
`73`		`- any(`
`74`		`- isinstance(referrer, futures.Future)`
	`72`	`+ self.assertTrue(`
	`73`	`+ all(`
	`74`	`+ not isinstance(referrer, futures.Future)`
`75`	`75`	`for referrer in gc.get_referrers(error)`
`76`	`76`	`)`
`77`	`77`	`)`