rename: build -> build_block_access_list

Author: fselmo

src/ethereum/amsterdam/block_access_lists/__init__.py

@@ -10,7 +10,7 @@
     add_storage_read,
     add_storage_write,
     add_touched_account,
-    build,
+    build_block_access_list,
 )
 from .rlp_utils import (
     compute_block_access_list_hash,
@@ -37,7 +37,7 @@
     "add_storage_read",
     "add_storage_write",
     "add_touched_account",
-    "build",
+    "build_block_access_list",
     "compute_block_access_list_hash",
     "set_transaction_index",
     "rlp_encode_block_access_list",

src/ethereum/amsterdam/block_access_lists/builder.py

@@ -344,7 +344,9 @@ def add_touched_account(
     ensure_account(builder, address)
 
 
-def build(builder: BlockAccessListBuilder) -> BlockAccessList:
+def build_block_access_list(
+    builder: BlockAccessListBuilder
+) -> BlockAccessList:
     """
     Build the final [`BlockAccessList`] from accumulated changes.
 

src/ethereum/amsterdam/block_access_lists/rlp_utils.py

@@ -216,10 +216,12 @@ def validate_block_access_list_against_execution(
     # 4. If Block Access List builder provided, validate against it
     # by comparing hashes
     if block_access_list_builder is not None:
-        from .builder import build
+        from .builder import build_block_access_list
 
         # Build a Block Access List from the builder
-        expected_block_access_list = build(block_access_list_builder)
+        expected_block_access_list = build_block_access_list(
+            block_access_list_builder
+        )
 
         # Compare hashes
         if compute_block_access_list_hash(

src/ethereum/amsterdam/fork.py

@@ -30,7 +30,7 @@
 )
 
 from . import vm
-from .block_access_lists.builder import build
+from .block_access_lists.builder import build_block_access_list
 from .block_access_lists.rlp_utils import compute_block_access_list_hash
 from .block_access_lists.tracker import (
     set_transaction_index,
@@ -797,7 +797,7 @@ def apply_body(
         block_env=block_env,
         block_output=block_output,
     )
-    block_output.block_access_list = build(
+    block_output.block_access_list = build_block_access_list(
         block_env.state.change_tracker.block_access_list_builder
     )
 

src/ethereum_spec_tools/evm_tools/loaders/fork_loader.py

@@ -127,7 +127,7 @@ def signing_hash_155(self) -> Any:
     @property
     def build(self) -> Any:
         """build function of the fork"""
-        return self._module("block_access_lists").build
+        return self._module("block_access_lists").build_block_access_list
 
     @property
     def compute_block_access_list_hash(self) -> Any:

tests/amsterdam/test_bal_implementation.py

@@ -23,7 +23,7 @@
     add_storage_read,
     add_storage_write,
     add_touched_account,
-    build,
+    build_block_access_list,
 )
 from ethereum.amsterdam.block_access_lists.tracker import (
     capture_pre_state,
@@ -160,7 +160,7 @@ def test_bal_builder_build_complete(self) -> None:
         add_touched_account(builder, address2)
 
         # Build BAL
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
 
         assert isinstance(block_access_list, BlockAccessList)
         assert len(block_access_list.account_changes) == 2
@@ -364,7 +364,7 @@ def test_system_contract_indices(self) -> None:
             Bytes32(b"\x02" * 32),
         )
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
 
         for account in block_access_list.account_changes:
             if account.address in [beacon_roots_addr, history_addr]:
@@ -384,7 +384,7 @@ def test_transaction_indices(self) -> None:
                 builder, address, BlockAccessIndex(tx_num), U256(0)
             )
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
 
         assert len(block_access_list.account_changes) == 3
         for i, account in enumerate(block_access_list.account_changes):
@@ -407,7 +407,7 @@ def test_post_execution_index(self) -> None:
             U256(0),
         )
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
 
         for account in block_access_list.account_changes:
             if account.address == withdrawal_addr:
@@ -434,7 +434,7 @@ def test_mixed_indices_ordering(self) -> None:
         )
         add_balance_change(builder, address, BlockAccessIndex(0), U256(0))
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
 
         assert len(block_access_list.account_changes) == 1
         account = block_access_list.account_changes[0]
@@ -475,7 +475,7 @@ def test_rlp_encode_simple_bal(self) -> None:
 
         add_balance_change(builder, address, BlockAccessIndex(1), U256(0))
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
         encoded = rlp_encode_block_access_list(block_access_list)
 
         # Should produce valid RLP bytes
@@ -499,7 +499,7 @@ def test_bal_hash_computation(self) -> None:
             Bytes32(b"\x03" * 32),
         )
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
         hash_val = compute_block_access_list_hash(block_access_list)
 
         # Should produce a 32-byte hash
@@ -535,7 +535,7 @@ def test_rlp_encode_complex_bal(self) -> None:
             builder, address, BlockAccessIndex(2), Bytes(b"\x60\x80")
         )
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
         encoded = rlp_encode_block_access_list(block_access_list)
 
         # Should produce valid RLP bytes
@@ -549,7 +549,7 @@ class TestEdgeCases:
     def test_empty_bal(self) -> None:
         """Test building an empty BAL."""
         builder = BlockAccessListBuilder()
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
 
         assert isinstance(block_access_list, BlockAccessList)
         assert len(block_access_list.account_changes) == 0
@@ -571,7 +571,7 @@ def test_multiple_changes_same_slot(self) -> None:
             builder, address, slot, BlockAccessIndex(2), Bytes32(b"\x02" * 32)
         )
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
 
         assert len(block_access_list.account_changes) == 1
         account = block_access_list.account_changes[0]
@@ -604,7 +604,7 @@ def test_address_sorting(self) -> None:
         for addr in addresses:
             add_touched_account(builder, addr)
 
-        block_access_list = build(builder)
+        block_access_list = build_block_access_list(builder)
 
         # Should be sorted lexicographically
         sorted_addresses = sorted(addresses)