From a2d2ec69e327da09b32e0e90a148d371e37d4f3a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Philipp=20Kr=C3=BCger?= Date: Fri, 9 Aug 2024 14:36:01 +0200 Subject: [PATCH] test(iroh): Reduce entry amount in `sync_gossip_bulk` (#2608) ## Description The test currently runs 55s on our windows CI machines, it runs 40s when I run it on my laptop locally. If anyone is curious: In release mode, the test runs 0.4s. :upside_down_face: The test basically sync 1000 entries twice. The number is configurable and the default was arbitrarily chosen according to @Frando, so I've reduced it by a factor of 10. With `n_entries = 100` the test runs ~4.5s for me, much more reasonable. ## Breaking Changes None ## Notes & open questions ## Change checklist - [x] Self-review. - ~~[ ] Documentation updates following the [style guide](https://rust-lang.github.io/rfcs/1574-more-api-documentation-conventions.html#appendix-a-full-conventions-text), if relevant.~~ - [x] Tests if relevant. - [x] All breaking changes documented. --- iroh/tests/sync.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/iroh/tests/sync.rs b/iroh/tests/sync.rs index cf94c4926a..026d14c6fc 100644 --- a/iroh/tests/sync.rs +++ b/iroh/tests/sync.rs @@ -156,7 +156,7 @@ async fn sync_subscribe_no_sync() -> Result<()> { async fn sync_gossip_bulk() -> Result<()> { let n_entries: usize = std::env::var("N_ENTRIES") .map(|x| x.parse().expect("N_ENTRIES must be a number")) - .unwrap_or(1000); + .unwrap_or(100); let mut rng = test_rng(b"sync_gossip_bulk"); setup_logging();