ApiTester::new_from_config doesn't work properly in Electra environment #7005
Description
Background
lighthouse/beacon_node/http_api/tests/tests.rs
Lines 6251 to 6263 in 6ab6eae
I was working on #7003, and found that current unit test(get_events_electra) doesn't work properly due to the merkle_proof package issue.
Reproduction
-
Intentionally disabled
#![cfg(not(debug_assertions))]for testing purpose. -
Run following command:
RUST_BACKTRACE=1 RUST_MIN_STACK=16777216 cargo test get_events_electra -p http_api --test bn_http_api_tests
Backtrace
...
thread 'tokio-runtime-worker' panicked at consensus/merkle_proof/src/lib.rs:317:9:
assertion failed: current_node.is_leaf()
stack backtrace:
0: rust_begin_unwind
at /rustc/e71f9a9a98b0faf423844bf0ba7438f29dc27d58/library/std/src/panicking.rs:665:5
1: core::panicking::panic_fmt
at /rustc/e71f9a9a98b0faf423844bf0ba7438f29dc27d58/library/core/src/panicking.rs:76:14
2: core::panicking::panic
at /rustc/e71f9a9a98b0faf423844bf0ba7438f29dc27d58/library/core/src/panicking.rs:148:5
3: merkle_proof::MerkleTree::generate_proof
at /Users/junsong/projects/ethereum-core/cl/lighthouse/consensus/merkle_proof/src/lib.rs:319:9
4: types::beacon_state::BeaconState<E>::generate_proof
at /Users/junsong/projects/ethereum-core/cl/lighthouse/consensus/types/src/beacon_state.rs:2631:26
5: types::beacon_state::BeaconState<E>::compute_finalized_root_proof
at /Users/junsong/projects/ethereum-core/cl/lighthouse/consensus/types/src/beacon_state.rs:2619:25
6: beacon_chain::light_client_server_cache::LightClientCachedData<E>::from_state
at /Users/junsong/projects/ethereum-core/cl/lighthouse/beacon_node/beacon_chain/src/light_client_server_cache.rs:424:13
7: beacon_chain::light_client_server_cache::LightClientServerCache<T>::cache_state_data
at /Users/junsong/projects/ethereum-core/cl/lighthouse/beacon_node/beacon_chain/src/light_client_server_cache.rs:70:31
8: beacon_chain::beacon_chain::BeaconChain<T>::import_block
at /Users/junsong/projects/ethereum-core/cl/lighthouse/beacon_node/beacon_chain/src/beacon_chain.rs:4038:13
9: beacon_chain::beacon_chain::BeaconChain<T>::import_available_block::{{closure}}::{{closure}}
at /Users/junsong/projects/ethereum-core/cl/lighthouse/beacon_node/beacon_chain/src/beacon_chain.rs:3749:21
...
Additional comment
lighthouse/consensus/merkle_proof/src/lib.rs
Line 289 in 6ab6eae
generate_proof function with index = 169, depth = 6 would fall current_node back into Zero node. So this makes assertion(is_leaf()) fail.