]> granicus.if.org Git - python/commit
bpo-34130: Fix test_signal.test_warn_on_full_buffer() (GH-8327)
authorVictor Stinner <vstinner@redhat.com>
Wed, 18 Jul 2018 16:29:54 +0000 (18:29 +0200)
committerGitHub <noreply@github.com>
Wed, 18 Jul 2018 16:29:54 +0000 (18:29 +0200)
commit686b4b5ff219ed66714f3b811815776dafadc23b
tree9516e763de1fe17521370710994ec8b11eadf824
parent99bb6df66a42625367c4f38e6802c8bb527baf4a
bpo-34130: Fix test_signal.test_warn_on_full_buffer() (GH-8327)

On Windows, sometimes test_signal.test_warn_on_full_buffer() fails to
fill the socketpair buffer. In that case, the C signal handler
succeed to write into the socket, it doesn't log the expected send
error, and so the test fail.

On Windows, the test now uses a timeout of 50 ms to fill the
socketpair buffer to fix this race condition.

Other changes:

* Begin with large chunk size to fill the buffer to speed up the
  test.
* Add error messages to assertion errors to more easily identify
  which assertion failed.
* Don't set the read end of the socketpair as non-blocking.
Lib/test/test_signal.py