mirrors/arti.git/crates/tor-circmgr/src/path/exitpath.rs, branch arti-v1.2.7

tor-circmgr: removed unused 'skip_guard_subnet_check' workaround in tests

2024-08-15T17:54:58Z

tor-circmgr: removed unused "chosen exit" functionality

2024-08-15T17:54:58Z

tor-circmgr: fix flaky `path::exitpath::test::by_ports` test

2024-08-13T18:44:24Z

The `path::exitpath::test::by_ports` test sometimes failed now that the test is using a `GuardMgr` since `select_guard`, when given a chosen exit, only ensures that the guard and chosen exit are not in the same family. It does not ensure that the guard and exit do not share an extended family. This commit relaxes an assertion in the test. ```text thread 'path::exitpath::test::by_ports' panicked at crates/tor-circmgr/src/path/exitpath.rs:295:9: assertion failed: r1.can_share_circuit(r3, subnet_config) ``` This "chosen exit" functionality isn't actually being used anywhere (`ExitPathBuilderInner::ChosenExit` is only ever constructed in tests).

tor-circmgr: assert in test that exit path begins with guard

2024-08-13T01:13:17Z

tor-circmgr: make `GuardMgr` mandatory

2024-08-13T00:21:53Z

Functions that took `Option<&GuardMgr>` now take only `&GuardMgr`. Three unit tests were removed that covered behaviour when no guard manager was set.

tor-circmg: prepare tests for runtime requirement

2024-08-12T22:54:22Z

This wraps some unit tests with `tor_rtcompat::test_with_all_runtimes!`. This is its own commit to get the indentation changes out of the way and declutter the following commit.

exitpath: Refactor a test to use tor-relay-usage.

2024-03-28T12:35:48Z

Remove temporary functions in CheckedRelay.

2024-03-28T12:21:14Z

tor-circmgr: Move ExitPathBuilder::from_chosen_exit under the test mod.

2024-03-20T17:48:14Z

`ExitPathBuilder` is now `pub(crate)`, so we need to move this under the `test` module to fix the "unused" clippy warning.

tor-circmgr: Make the path module crate-private.

2024-03-20T17:48:14Z

We're not currently using any of the path builders outside of `tor-circmgr`, so let's make them crate-private for now. Care must be taken if/when we decide to make them public again. For instance, the `HsPathBuilder` exposes two path building functions, one that uses vanguards, and one that doesn't. We want to strongly encourage the use of the vanguards-aware version of the function whenever the `vanguards` feature is enabled, without breaking any of its existing non-vanguard uses. Prompted by https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/2046#note_3010217