Roytak
commented
5 months ago The test suite could really use some updating and that is something I plan to do in the future. I think this is a good step forward and a nice change. The CMake looks way cleaner this way.
michalvasko
Running tests in parallel was broken because these files all had some hard-coded port numbers. Some of these port numbers were OK, others were overlapping. Let's fix that by letting CMake manage these port numbers. However, to do that, one has to first convert the CMake setup to a "more idiomatic" approach with one function call per test. As a bonus, this gets rid of some "random variables" and the final test declarations are, hopefully, a bit easier to read. See the individual commits for descriptions.