Skip to content

Commit

Permalink
Address test_multihop flakiness
Browse files Browse the repository at this point in the history
Limit relay selection in `test_multihop` to reduce flakiness.

Hopefully this should be able to (at least partially) mitigate timeout
related issues.
  • Loading branch information
MarkusPettersson98 committed Nov 12, 2024
1 parent 7d6d4e7 commit 8923705
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion test/test-manager/src/tests/tunnel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -294,7 +294,16 @@ pub async fn test_multihop(
rpc: ServiceClient,
mut mullvad_client: MullvadProxyClient,
) -> Result<(), Error> {
let query = RelayQueryBuilder::new().wireguard().multihop().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()
.multihop()
.location(LowLatency)
.entry(LowLatency)
.build();

apply_settings_from_relay_query(&mut mullvad_client, query).await?;

Expand Down

0 comments on commit 8923705

Please sign in to comment.