Fixes #31103
On some CI runs, the seed node timer in ThreadOpenConnection was only started after the mocktime was set.
Fix this by waiting for the first connection attempt, which happens after the timer was started.
Also I noticed that the “unreachable” connections are not in fact unreachable, so that the functional test could attempt connections to random IPs on the internet. This was already noted in #29605 (review) but the suggested fix never made it in, so I added it to this PR.
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For detailed information about the code coverage, see the test coverage report.
See the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
On some CI runs, the timer in ThreadOpenConnection was only started *after*
the mocktime was set. Fix this by waiting for the first connection attempt,
which happens after the timer was started.
Also convert some comments into log messages/add a log, so that the test
isn't completely silent.
These addrs aren't unreachable as the test claims.
Specify a (non-working) proxy to make sure the connections fails
even if the addr was reachable.
Co-authored-by: Vasil Dimov <vd@FreeBSD.org>
cr and light test ACK 6c9fe7b73ea1572b8b56c716ab13d9866f91c6e9
Nice addition of the non-working proxy.
