| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Run maint/add_warning to actually apply new lint allows | Ian Jackson | 2023-07-10 | 1 | -0/+1 |
| | | |||||
| * | tor-dirmgr: bridge descriptor tests: Add a missing #[traced_test] | Ian Jackson | 2023-07-06 | 1 | -0/+1 |
| | | | | | This seems to have been overlooked. | ||||
| * | tor-dirmgr: bridge descriptor tests: Use MockRuntime (fmt) | Ian Jackson | 2023-07-06 | 1 | -270/+271 |
| | | | | | Run rustfmt. | ||||
| * | tor-dirmgr: bridge descriptor tests: Use MockRuntime | Ian Jackson | 2023-07-06 | 1 | -35/+25 |
| | | | | | This abolishes a bodge sleep. It should make the tests deterministic. | ||||
| * | tor-dirmgr: bridge descriptor tests: Fix a v. low prob. race | Ian Jackson | 2023-07-06 | 1 | -2/+5 |
| | | | | | | | | | | | Drain a number of events, not just one. The stream might yield many events, as explained in this new comment. This fails every time with MockExecutor::try_test_with_various(). I think it might fail with the tokio exeuctor too, but evidently not with high probability or we would have noticed. | ||||
| * | Use bool::then_some() as appropriate | Nick Mathewson | 2023-04-11 | 1 | -7/+7 |
| | | | | | | | Now that we require a version of Rust that allows `b.then_some(v)`, clippy complains about our use of `b.then(|| v)`. | ||||
| * | Sort out some TODOs in bdtest.rs | Ian Jackson | 2023-03-21 | 1 | -5/+4 |
| | | |||||
| * | Allow clippy::unchecked_duration_subtraction in tests | Nick Mathewson | 2023-01-27 | 1 | -0/+1 |
| | | | | | | This panics on error, and we're fine with a panic on misbehavior in tests. | ||||
| * | bridge desc mgr: Test dormancy | Ian Jackson | 2022-11-23 | 1 | -0/+42 |
| | | |||||
| * | BridgeConfig: Replace all Arc<BridgeConfig> with BridgeConfig | Ian Jackson | 2022-11-22 | 1 | -2/+1 |
| | | | | | BridgeConfig is itself an Arc now, so these are redundant. | ||||
| * | Fix a bunch of "needless borrow" warnings on nightly | Nick Mathewson | 2022-11-18 | 1 | -1/+1 |
| | | | | | | It looks like, despite a few false starts, they've got this warning right; there weren't any false positives. | ||||
| * | bridge descriptor dormancy: Accept the dormancy value | Ian Jackson | 2022-11-11 | 1 | -0/+1 |
| | | | | | But right now, don't do anything with it. That will come in a future MR. | ||||
| * | bridge descs: Add tests for process_document | Ian Jackson | 2022-11-08 | 1 | -0/+100 |
| | | |||||
| * | bridge descs: Rename BridgeDescMgr from BridgeDescManager | Ian Jackson | 2022-11-08 | 1 | -2/+2 |
| | | | | | | | This is more consistent with our naming elsewhere. Suggested-by: Nick Mathewson <[email protected]> | ||||
| * | bridge desc tests: Test caching and if-modified-since | Ian Jackson | 2022-11-04 | 1 | -2/+121 |
| | | |||||
| * | bridge desc tests: Break out queues_are_empty helper | Ian Jackson | 2022-11-04 | 1 | -2/+6 |
| | | |||||
| * | bridge desc tests: Provide a sqlite connection | Ian Jackson | 2022-11-04 | 1 | -2/+5 |
| | | | | | The current test case doesn't use this. | ||||
| * | bridge desc: Provide a Store to the bridge desc manager | Ian Jackson | 2022-11-04 | 1 | -5/+10 |
| | | | | | | | | | | | | | | | | | We use the one in the dirmgr. That means that our constructor now has to take a dirmgr. And, the dirmgr must have a circmgr. This is all rather odd, TBH. Add .. to the binding of the return values from setup, as future-proofing. The tests now need to provide a Store too. Make the sqlite::new_empty function pub(crate) so we can use it. We must retain the _db_tmp_dir, since when it goes away the tmp directory is deleted and the db goes readonly. | ||||
| * | bridge desc tests: Apply tracing_test::traced_test | Ian Jackson | 2022-11-04 | 1 | -0/+2 |
| | | | | | This lets us see the log messages. They look fine, as it happens. | ||||
| * | bridgedesc: Prepare Mockable trait for if-modified-since | Ian Jackson | 2022-11-03 | 1 | -2/+3 |
| | | |||||
| * | bridge descriptors tests: Fix flaky test | Ian Jackson | 2022-11-03 | 1 | -1/+11 |
| | | |||||
| * | bridge descriptors tests: Test set_bridges more thoroughly (fix) | Ian Jackson | 2022-11-03 | 1 | -1/+1 |
| | | | | | | | The RetryDelays are being triggered for the 2nd time here, so their timeouts can be longer. We must bump the sleep to make sure we don't have a flaky test. | ||||
| * | bridge descriptors tests: Test set_bridges more thoroughly | Ian Jackson | 2022-11-03 | 1 | -0/+34 |
| | | | | | | | | | Prior to the previous commit, set_bridges would malfunction if there were bridges which where (i) in current (ii) in queued or running (iii) in the new bridge set. This test failed then and passes now. | ||||
| * | bridge descriptors tests: Add another couple of test cases | Ian Jackson | 2022-11-03 | 1 | -1/+31 |
| | | |||||
| * | bridge descriptors tests: stream_drain_ready returns event count | Ian Jackson | 2022-11-03 | 1 | -1/+4 |
| | | | | | We'll use this in a moment. | ||||
| * | bridge descriptors tests: Break out bad_bridge helper function | Ian Jackson | 2022-11-03 | 1 | -7/+7 |
| | | |||||
| * | bridge descriptors: Change default max_refetch | Ian Jackson | 2022-11-03 | 1 | -1/+1 |
| | | | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/820#note_2850269 | ||||
| * | bridge descriptors: Tests | Ian Jackson | 2022-11-03 | 1 | -0/+254 |
