mab122
/
radicle-heartwood-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

node: Fix E2E tests 

The tests were flaky because the routes are sampled at a moment
when gossip could still be happening. Instead, we check the
local inventory directly to build the expected routing table.

Signed-off-by: Alexis Sellier <self@cloudhead.io>
This commit is contained in:
Alexis Sellier 2023-01-17 12:58:01 +01:00
parent 3bedb4438e
commit df9346fefb
No known key found for this signature in database
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
radicle-node/src/tests/e2e.rs
View File

@ -10,7 +10,7 @@ use radicle::git::refname;
use radicle::identity::Id; use radicle::identity::Id;
use radicle::node::Handle; use radicle::node::Handle;
use radicle::profile::Home; use radicle::profile::Home;
use radicle::storage::WriteStorage; use radicle::storage::{ReadStorage, WriteStorage};
use radicle::test::fixtures; use radicle::test::fixtures;
use radicle::Storage; use radicle::Storage;
use radicle::{assert_matches, rad}; use radicle::{assert_matches, rad};
@ -126,10 +126,10 @@ fn check<'a>(nodes: impl IntoIterator<Item = &'a Node>) -> BTreeSet<(Id, NodeId)
// First build the set of all routes. // First build the set of all routes.
for node in &nodes { for node in &nodes {
let routing = node.handle.routing().unwrap(); let inv = node.storage.inventory().unwrap();
for (rid, seed) in routing.try_iter() { for rid in inv {
all_routes.insert((rid, seed)); all_routes.insert((rid, node.id));
} }
} }