Skip to content

Commit 82ab8fb

Browse files
[3.14] tests: use errno.EBADF instead of hardcoded number in _close_file() (GH-148345) (#148410)
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 942d302 commit 82ab8fb

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 logging
56
import os
@@ -52,7 +53,7 @@ def _close_file(file):
5253
else:
5354
os.close(file)
5455
except OSError as exc:
55-
if exc.errno != 9:
56+
if exc.errno != errno.EBADF:
5657
raise # re-raise
5758
# It was closed already.
5859

0 commit comments

Comments
 (0)