Skip to content

Commit 74be9e2

Browse files
[3.13] tests: use errno.EBADF instead of hardcoded number in _close_file() (GH-148345) (#148411)
tests: use errno.EBADF instead of hardcoded number in _close_file() (GH-148345) test_interpreters: use errno.EBADF instead of hardcoded number in _close_file() Replace the hardcoded `9` check in `Lib/test/test_interpreters/utils.py` with `errno.EBADF`. Using `errno.EBADF` makes the helper portable across platforms with different errno numbering while preserving the intended behavior. (cherry picked from commit cef334f) Co-authored-by: Artem Yarulin <artem.yarulin@kapteko.com>
1 parent 9628a79 commit 74be9e2

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

Lib/test/test_interpreters/utils.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
from collections import namedtuple
22
import contextlib
3+
import errno
34
import json
45
import io
56
import logging
@@ -55,7 +56,7 @@ def _close_file(file):
5556
else:
5657
os.close(file)
5758
except OSError as exc:
58-
if exc.errno != 9:
59+
if exc.errno != errno.EBADF:
5960
raise # re-raise
6061
# It was closed already.
6162

0 commit comments

Comments
 (0)