mirrors/arti.git/crates/tor-circmgr/src/preemptive.rs, branch arti-v1.1.7mirror of https://gitlab.torproject.org/tpo/core/arti
http://git.dilluti0n.com/mirrors/arti.git/atom?h=arti-v1.1.72023-07-10T12:49:51ZRun maint/add_warning to actually apply new lint allows2023-07-10T12:49:51ZIan Jacksonijackson@chiark.greenend.org.uk2023-07-10T11:19:32Zurn:sha1:473447a82e099fd52a674c28f5bc24b3e05208c0Allow clippy::unchecked_duration_subtraction in tests2023-01-27T13:28:02ZNick Mathewsonnickm@torproject.org2023-01-27T13:01:45Zurn:sha1:a6dd92843edf2bd4c58de62a0d562921e4191ada
This panics on error, and we're fine with a panic on misbehavior in
tests.
test lint blocks: Add many many automatically2022-12-12T18:00:30ZIan Jacksonijackson@chiark.greenend.org.uk2022-12-12T17:58:40Zurn:sha1:125a455bdcca841931f33f83f206e57637ae85c4
This is precisely the result of running the rune in
maint/adhoc-add-lint-blocks.
Shorten the duration needed in preemptive::test::does_not_predict_old_ports.2022-08-31T12:52:40ZAlexander Færøyahf@torproject.org2022-08-31T11:54:12Zurn:sha1:3e3a68683c96fd79e775e9dd707138398ca32b8c
This patch shortens the duration of the `does_not_predict_old_ports`
test in the preemptive module. AppVeyor spawns its VMs/containers per
build, so the `Instant::now()` call returns a value smaller than `60 *
60 + 1` which causes the subtraction to overflow and thus panic.
Thanks to @trinity-1686a for the help here.
See: tpo/core/arti#563.
Change builder list API2022-05-04T12:50:10ZIan Jacksonijackson@chiark.greenend.org.uk2022-05-04T12:32:35Zurn:sha1:4bca91271558d950092fb1ac51c1316655d293cb
The new API is (roughly) as discussed in
https://gitlab.torproject.org/tpo/core/arti/-/issues/451
This is quite a large commit and it is not convenient to split it up.
It contains the following changes:
* Redo the list builder and accessor macros implemnetation,
including docs and tests.
* Change uses of define_list_config_builder. In each case:
- Move the docs about the default value to the containing field.
- Remove the other docs (which were just recapitulations, and
are now not needed since the ListBuilder is no longer public).
- Rewmove or replace `pub` in the define_list_builder_helper call,
so that the builder is no longer public.
- Change the main macro call site to use define_list_builder_helper.
- Add a call to define_list_builder_accessors.
* Make the module `list_builder` pub so that we have somewhere to
put the overview documentation.
* Consequential changes:
- Change `outer.inner().replace(X)` to `outer.set_inner(X)`
- Consequential changes to imports (`use` statements).
Rename ThingListBuilder::replace (from set)2022-04-25T17:15:25ZIan Jacksonijackson@chiark.greenend.org.uk2022-04-25T15:55:21Zurn:sha1:1f215da1a32af7e7bbac7e8314858659c6d59c4e
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/471#note_2798024
Introduce PredictedPortsListBuilder2022-04-25T16:05:30ZIan Jacksonijackson@chiark.greenend.org.uk2022-04-25T13:33:03Zurn:sha1:24518675dbd2e649b554ecc0117ea62ede11b292
This means that `NetworkConfig::initial_predicted_ports` is now like
the other list-like things, returning `&mut list_builder` with the same
`set()` and `append()` methods.
move isolation in separate module2022-03-24T18:43:54Ztrinity-1686atrinity@deuxfleurs.fr2022-03-24T18:43:54Zurn:sha1:8dc6e958aa305236054a43c0db74dda9176e0702add trait to help test isolation related code2022-03-16T18:24:23Ztrinity-1686atrinity@deuxfleurs.fr2022-03-15T23:30:41Zurn:sha1:4eb90b72c04b34fb54baa197b1bf85734a769f64fix existing tests2022-03-16T18:23:21Ztrinity-1686atrinity@deuxfleurs.fr2022-03-12T09:32:48Zurn:sha1:234291f666c062a364685fda24aa46d59c6caf38