mirrors/arti.git/crates/tor-circmgr/src/path/exitpath.rs, branch arti-v1.2.8mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v1.2.82024-09-09T17:26:37Ztor-netdir: Allow access to the `ConsensusBuilder` when building test netdirs.2024-09-09T17:26:37ZGabriela Moldovangabi@torproject.org2024-09-05T15:42:08Zurn:sha1:b2b75302ab095bccdc7ccecb627e4648a3ae9419
This allows us to set SRVs for example (needed because by default, the
test `NetDir` is built from a consensus that doesn't contain any SRVs).
tor-circmgr: removed unused 'skip_guard_subnet_check' workaround in tests2024-08-15T17:54:58ZSteven Englersmengler3@gmail.com2024-08-15T15:35:49Zurn:sha1:4af7ab716019c85a0373fd82ee22daab3c74a0cetor-circmgr: removed unused "chosen exit" functionality2024-08-15T17:54:58ZSteven Englersmengler3@gmail.com2024-08-15T15:32:46Zurn:sha1:e03a2b6cc1f9ee07404c6ac0cd736fc4eba86a99tor-circmgr: fix flaky `path::exitpath::test::by_ports` test2024-08-13T18:44:24ZSteven Englersmengler3@gmail.com2024-08-13T18:10:10Zurn:sha1:e0b7f6582283e8cdb721b2df23e99257e4191eb0
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 guard2024-08-13T01:13:17ZSteven Englersmengler3@gmail.com2024-08-13T01:12:32Zurn:sha1:2a4208aa0675c966c58fbb368745254545e6e923tor-circmgr: make `GuardMgr` mandatory2024-08-13T00:21:53ZSteven Englersmengler3@gmail.com2024-08-13T00:09:27Zurn:sha1:acf1e81bb8621ec322ebd56b89c47c28958d3bfb
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 requirement2024-08-12T22:54:22ZSteven Englersmengler3@gmail.com2024-08-12T22:26:35Zurn:sha1:7f13bce5a12e3edbfb6f58d758673a2336ad20ee
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:48ZNick Mathewsonnickm@torproject.org2024-03-28T12:35:48Zurn:sha1:da4ec973d45a1600a2b262f10a37a4b896cf3065Remove temporary functions in CheckedRelay.2024-03-28T12:21:14ZNick Mathewsonnickm@torproject.org2024-03-27T18:09:37Zurn:sha1:dd53eca3a35d5918e13cac50a13f2bb91ce622dator-circmgr: Move ExitPathBuilder::from_chosen_exit under the test mod.2024-03-20T17:48:14ZGabriela Moldovangabi@torproject.org2024-03-20T16:32:25Zurn:sha1:6ad6bc5eb10b4c70fb41268de696e23c66d8fd53
`ExitPathBuilder` is now `pub(crate)`, so we need to move this under the
`test` module to fix the "unused" clippy warning.