test: rework ClientDisconnect test

pistacheio / pistache

A high-performance REST toolkit written in C++

https://pistacheio.github.io/pistache/

Apache License 2.0

3.18k stars 698 forks source link

test: rework ClientDisconnect test #1071

Closed Tachi107 closed 2 years ago

Tachi107 commented 2 years ago

It was using a complex mix of condition variables, mutexes, synchronization stuff that was completely unneeded, and for some reason this made the test lock up when ran on old Linux distros like RHEL 8 or Ubuntu 20.10. This should fix the build failures in the Launchpad PPA.

Cherry-picked from #1070, as it is taking a while to get merged.

kiplingw commented 2 years ago

Well done @Tachi107. After all CI passes, you can go ahead and merge.