diff --git a/light-client/src/store/sled.rs b/light-client/src/store/sled.rs index a6e9ac462..38219e45d 100644 --- a/light-client/src/store/sled.rs +++ b/light-client/src/store/sled.rs @@ -23,7 +23,6 @@ pub struct SledStore { verified_db: HeightIndexedDb, trusted_db: HeightIndexedDb, failed_db: HeightIndexedDb, - db: sled::Db, } impl SledStore { @@ -39,7 +38,6 @@ impl SledStore { verified_db: HeightIndexedDb::new(db.open_tree(VERIFIED).unwrap()), trusted_db: HeightIndexedDb::new(db.open_tree(TRUSTED).unwrap()), failed_db: HeightIndexedDb::new(db.open_tree(FAILED).unwrap()), - db, } } diff --git a/light-client/src/supervisor.rs b/light-client/src/supervisor.rs index df396d2a1..425acaddf 100644 --- a/light-client/src/supervisor.rs +++ b/light-client/src/supervisor.rs @@ -521,10 +521,7 @@ mod tests { let mut peer_list = PeerList::builder(); if let Some(primary) = primary { - let io = MockIo::new( - primary[0].signed_header.header.chain_id.to_string(), - primary.clone(), - ); + let io = MockIo::new(primary.clone()); let primary_instance = make_instance(primary[0].provider, trust_options.clone(), io, now); @@ -535,10 +532,7 @@ mod tests { if let Some(witnesses) = witnesses { for provider in witnesses.into_iter() { let peer_id = provider[0].provider; - let io = MockIo::new( - provider[0].signed_header.header.chain_id.to_string(), - provider, - ); + let io = MockIo::new(provider); let instance = make_instance(peer_id, trust_options.clone(), io.clone(), now); peer_list.witness(peer_id, instance); } diff --git a/light-client/src/tests.rs b/light-client/src/tests.rs index c988bc0f9..660c09a8b 100644 --- a/light-client/src/tests.rs +++ b/light-client/src/tests.rs @@ -84,13 +84,12 @@ impl Clock for MockClock { #[derive(Clone)] pub struct MockIo { - chain_id: String, light_blocks: HashMap, latest_height: Height, } impl MockIo { - pub fn new(chain_id: String, light_blocks: Vec) -> Self { + pub fn new(light_blocks: Vec) -> Self { let latest_height = light_blocks.iter().map(|lb| lb.height()).max().unwrap(); let light_blocks = light_blocks @@ -99,7 +98,6 @@ impl MockIo { .collect(); Self { - chain_id, light_blocks, latest_height, } diff --git a/light-client/tests/backward.rs b/light-client/tests/backward.rs index af7798692..ce1617266 100644 --- a/light-client/tests/backward.rs +++ b/light-client/tests/backward.rs @@ -45,7 +45,6 @@ struct TestCase { fn make(chain: LightChain, trusted_height: Height) -> (LightClient, State) { let primary = default_peer_id(); - let chain_id = "testchain-1".parse().unwrap(); let clock = MockClock { /// Set the current time to be ahead of the latest block in the chain @@ -65,7 +64,7 @@ fn make(chain: LightChain, trusted_height: Height) -> (LightClient, State) { .map(testgen_to_lb) .collect(); - let io = MockIo::new(chain_id, light_blocks); + let io = MockIo::new(light_blocks); let trusted_state = io .fetch_light_block(AtHeight::At(trusted_height)) diff --git a/light-client/tests/light_client.rs b/light-client/tests/light_client.rs index ed69765f5..1f2f626d5 100644 --- a/light-client/tests/light_client.rs +++ b/light-client/tests/light_client.rs @@ -48,7 +48,7 @@ fn run_test(tc: LightClientTest) -> BisectionTestResult { }; let provider = tc.primary; - let io = MockIo::new(provider.chain_id, provider.lite_blocks); + let io = MockIo::new(provider.lite_blocks); let trusted_height = tc.trust_options.height; let trusted_state = io diff --git a/light-client/tests/supervisor.rs b/light-client/tests/supervisor.rs index 4186414d7..0a7dcfc3f 100644 --- a/light-client/tests/supervisor.rs +++ b/light-client/tests/supervisor.rs @@ -69,7 +69,7 @@ fn run_multipeer_test(tc: LightClientTest) { None => false, }; - let io = MockIo::new(tc.primary.chain_id, tc.primary.lite_blocks); + let io = MockIo::new(tc.primary.lite_blocks); let primary_instance = make_instance(primary, tc.trust_options.clone(), io.clone(), tc.now); let mut peer_list = PeerList::builder(); @@ -78,7 +78,7 @@ fn run_multipeer_test(tc: LightClientTest) { for provider in tc.witnesses.into_iter() { let peer_id = provider.value.lite_blocks[0].provider; println!("Witness: {}", peer_id); - let io = MockIo::new(provider.value.chain_id, provider.value.lite_blocks); + let io = MockIo::new(provider.value.lite_blocks); let instance = make_instance(peer_id, tc.trust_options.clone(), io.clone(), tc.now); peer_list.witness(peer_id, instance); }