https://github.com/python/cpython/commit/6f7fb6c95ba9e4d5c1b86d0d4ad83d0cd495976b
commit: 6f7fb6c95ba9e4d5c1b86d0d4ad83d0cd495976b
branch: main
author: Serhiy Storchaka <[email protected]>
committer: serhiy-storchaka <[email protected]>
date: 2026-08-09T19:57:45+03:00
summary:
gh-155423: Make BaseFileTest.tearDown() in test_logging robust (GH-155424)
Always call BaseTest.tearDown(), which removes the handlers added by
setUp(), even if unlinking the log files fails. Tolerate an already
removed file.
files:
M Lib/test/test_logging.py
diff --git a/Lib/test/test_logging.py b/Lib/test/test_logging.py
index d74670609ec0199..90c38557d2e21b4 100644
--- a/Lib/test/test_logging.py
+++ b/Lib/test/test_logging.py
@@ -6446,11 +6446,12 @@ def setUp(self):
self.rmfiles = []
def tearDown(self):
- for fn in self.rmfiles:
- os.unlink(fn)
- if os.path.exists(self.fn):
- os.unlink(self.fn)
- BaseTest.tearDown(self)
+ try:
+ for fn in self.rmfiles:
+ os_helper.unlink(fn)
+ os_helper.unlink(self.fn)
+ finally:
+ BaseTest.tearDown(self)
def assertLogFile(self, filename):
"Assert a log file is there and register it for deletion"
_______________________________________________
Python-checkins mailing list -- [email protected]
To unsubscribe send an email to [email protected]
https://mail.python.org/mailman3//lists/python-checkins.python.org
Member address: [email protected]
