From c9c43e8f661c003cf4e515cf8f9b36979cda8e31 Mon Sep 17 00:00:00 2001 From: Hsiao-Wei Wang Date: Thu, 18 Jan 2024 13:27:34 +0800 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Lion - dapplion <35266934+dapplion@users.noreply.github.com> --- .../pyspec/eth2spec/test/altair/transition/test_transition.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/core/pyspec/eth2spec/test/altair/transition/test_transition.py b/tests/core/pyspec/eth2spec/test/altair/transition/test_transition.py index ef5bc8fbd..b40142de7 100644 --- a/tests/core/pyspec/eth2spec/test/altair/transition/test_transition.py +++ b/tests/core/pyspec/eth2spec/test/altair/transition/test_transition.py @@ -425,11 +425,11 @@ def test_non_empty_historical_roots(state, fork_epoch, spec, post_spec, pre_tag, """ Test with non-empty pre-state `state.historical_roots`. - Since Capella froze `historical_roots`, Capealla spec doesn't invoke `process_historical_roots_update` anymore. + Since Capella froze `historical_roots`, Capella spec doesn't invoke `process_historical_roots_update` anymore. Therefore, we need to fill in `historical_roots` with non-empty value. """ # fill in historical_roots with non-empty values - pre_historical_roots = [b'\x56' * 32 for _ in range(spec.SLOTS_PER_HISTORICAL_ROOT)] + pre_historical_roots = [b'\x56' * 32] state.historical_roots = pre_historical_roots transition_until_fork(spec, state, fork_epoch)