Rémi Lapeyre <remi.lape...@henki.fr> added the comment:

Hi Pablo, while this patch should fix both problems, I'm not sure how to write 
a regression test for this, `assert_python_ok` in 
https://github.com/python/cpython/pull/7865/files#diff-7d4645839a05727ebdf39226fd56e29cR97
 forks the interpreter so I'm not sure I can check for unclosed fd during the 
test.

Should I call 
https://github.com/remilapeyre/cpython/blob/04b2ade751b318460c1f0f9566676ef519358328/Lib/json/tool.py#L18
 directly and mock `io.open` and `os.close`?

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue33927>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to