From b67023c42b24beda810cc43d5c967501a6c5c4f9 Mon Sep 17 00:00:00 2001 From: Markus Pettersson Date: Fri, 8 Nov 2024 13:00:28 +0100 Subject: [PATCH] Address `test_wireguard_over_shadowsocks` flakiness Limit relay selection in `test_wireguard_over_shadowsocks` to reduce flakiness. Hopefully this should be able to (at least partially) mitigate timeout related issues. --- test/test-manager/src/tests/tunnel.rs | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/test/test-manager/src/tests/tunnel.rs b/test/test-manager/src/tests/tunnel.rs index beaeb63ca54a..fde8c7304954 100644 --- a/test/test-manager/src/tests/tunnel.rs +++ b/test/test-manager/src/tests/tunnel.rs @@ -195,7 +195,15 @@ pub async fn test_wireguard_over_shadowsocks( rpc: ServiceClient, mut mullvad_client: MullvadProxyClient, ) -> anyhow::Result<()> { - let query = RelayQueryBuilder::new().wireguard().shadowsocks().build(); + // NOTE: We have experienced flakiness due to timeout issues if distant relays are selected. + // This is an attempt to try to reduce this type of flakiness. + use helpers::custom_lists::LowLatency; + + let query = RelayQueryBuilder::new() + .wireguard() + .shadowsocks() + .location(LowLatency) + .build(); apply_settings_from_relay_query(&mut mullvad_client, query).await?;