Commit 779ae7f4 authored by Antoine Pitrou's avatar Antoine Pitrou

Issue #20876: correctly close temporary file in test.support.fs_is_case_insensitive()

parent 516f14f1
...@@ -2133,16 +2133,15 @@ def skip_unless_xattr(test): ...@@ -2133,16 +2133,15 @@ def skip_unless_xattr(test):
def fs_is_case_insensitive(directory): def fs_is_case_insensitive(directory):
"""Detects if the file system for the specified directory is case-insensitive.""" """Detects if the file system for the specified directory is case-insensitive."""
base_fp, base_path = tempfile.mkstemp(dir=directory) with tempfile.NamedTemporaryFile(dir=directory) as base:
case_path = base_path.upper() base_path = base.name
if case_path == base_path: case_path = base_path.upper()
case_path = base_path.lower() if case_path == base_path:
try: case_path = base_path.lower()
return os.path.samefile(base_path, case_path) try:
except FileNotFoundError: return os.path.samefile(base_path, case_path)
return False except FileNotFoundError:
finally: return False
os.unlink(base_path)
class SuppressCrashReport: class SuppressCrashReport:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment