summaryrefslogtreecommitdiff
path: root/test/pathname
diff options
context:
space:
mode:
authorKJ Tsanaktsidis <kj@kjtsanaktsidis.id.au>2023-05-15 14:37:28 +1000
committerGitHub <noreply@github.com>2023-05-15 13:37:28 +0900
commit8bd4d8867a0222a3c30a0c7ee1f69b06baa8e91a (patch)
treedd908161307738ce7350cc0b988617348f9001e1 /test/pathname
parentd9033d374991c8bf65e70320b4a9f2791dd668ce (diff)
downloadruby-8bd4d8867a0222a3c30a0c7ee1f69b06baa8e91a.tar.gz
Unskip the test skipped in #4173 (#7809)
This test was skipped 12 years ago because it was flaky on FreeBSD and OpenBSD. Since then, Ruby's SIGCHLD handling has been substantially re-written (mostly by Eric Wong @normalperson in 44fc3d08). These tests now in fact pass reliably on Ruby master on FreeBSD 13.2 and OpenBSD 7.3. I stress-tested the test_wait_and_sigchild test on my laptop by running four copies of the test in a loop on a 8-core VM; both by itself and also as part of the whole test_process.rb file. I did not see any failures. Let's unskip the test and close [#4173] out. I'll keep an eye out on Ruby CI for any flakes in this file on BSD after this gets merged, but if we don't see any I'm going to assume 44fc3d08 or related changes around that time accidently fixed this bug. It's also probably important to unskip this test so that if another attempt at removing the special SIGCHLD handling is made (like was reverted in https://github.com/ruby/ruby/pull/7517), we get signal if that breaks on FreeBSD/OpenBSD. [Fixes #4173]
Diffstat (limited to 'test/pathname')
0 files changed, 0 insertions, 0 deletions