From 99bf3c112d1408a0920d16e22c0ddfd8b13bec45 Mon Sep 17 00:00:00 2001 From: Etan Kissling Date: Wed, 14 Aug 2024 20:33:17 +0200 Subject: [PATCH] Avoid generating empty transactions in tests Transactions cannot be empty, they always have at least 1 byte. Random tests should produce valid CL data by default. There are still individual tests for invalid transactions. --- tests/core/pyspec/eth2spec/test/helpers/execution_payload.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/core/pyspec/eth2spec/test/helpers/execution_payload.py b/tests/core/pyspec/eth2spec/test/helpers/execution_payload.py index 1fbb12d7b..99399abd4 100644 --- a/tests/core/pyspec/eth2spec/test/helpers/execution_payload.py +++ b/tests/core/pyspec/eth2spec/test/helpers/execution_payload.py @@ -64,7 +64,7 @@ def compute_trie_root_from_indexed_data(data): t = HexaryTrie(db={}) for i, obj in enumerate(data): k = encode(i, big_endian_int) - t.set(k, obj) + t.set(k, obj) # Implicitly skipped if `obj == b''` (invalid RLP) return t.root_hash @@ -359,4 +359,4 @@ def build_state_with_execution_payload_header(spec, state, execution_payload_hea def get_random_tx(rng): - return get_random_bytes_list(rng, rng.randint(0, 1000)) + return get_random_bytes_list(rng, rng.randint(1, 1000))