fix(tests): Fix test expectations for BAL oog tests (#2293)

* fix(tests): Fix test expectations for BAL oog tests

* chore: update Amsterdam commit hash for resolver

* chore: add changelog entry

Author: fselmo

.github/configs/eels_resolutions.json

@@ -52,6 +52,6 @@
     "Amsterdam": {
         "git_url": "https://github.com/fselmo/execution-specs.git",
         "branch": "feat/amsterdam-fork-and-block-access-lists",
-        "commit": "5d98d409da10f763754dea8d60e855395f944498"
+        "commit": "f3ad59980a68fe5974244f41bcda7b22294bf98b"
     }
 }

docs/CHANGELOG.md

@@ -18,6 +18,8 @@ Test fixtures for use by clients are available for each release on the [Github r
 
 ### 🧪 Test Cases
 
+- 🐞Fix BALs opcode OOG test vectors by updating the Amsterdam commit hash in specs and validating appropriately on the testing side ([#2293](https://github.com/ethereum/execution-spec-tests/pull/2293)).
+
 ## [v5.3.0](https://github.com/ethereum/execution-spec-tests/releases/tag/v5.3.0) - 2025-10-09
 
 ## 🇯🇵 Summary

src/pytest_plugins/eels_resolutions.json

@@ -55,6 +55,6 @@
     "Amsterdam": {
       "git_url": "https://github.com/fselmo/execution-specs.git",
       "branch": "feat/amsterdam-fork-and-block-access-lists",
-      "commit": "5d98d409da10f763754dea8d60e855395f944498"
+      "commit": "f3ad59980a68fe5974244f41bcda7b22294bf98b"
     }
 }

tests/amsterdam/eip7928_block_level_access_lists/test_block_access_lists_opcodes.py

@@ -233,7 +233,7 @@ def test_bal_balance_and_oog(
             account_expectations={
                 balance_checker: BalAccountExpectation.empty(),
                 # Bob should only appear in BAL if BALANCE succeeded
-                **({} if fails_at_balance else {bob: BalAccountExpectation.empty()}),
+                **({bob: None} if fails_at_balance else {bob: BalAccountExpectation.empty()}),
             }
         ),
     )
@@ -303,7 +303,7 @@ def test_bal_extcodesize_and_oog(
                 codesize_checker: BalAccountExpectation.empty(),
                 # Target should only appear if EXTCODESIZE succeeded
                 **(
-                    {}
+                    {target_contract: None}
                     if fails_at_extcodesize
                     else {target_contract: BalAccountExpectation.empty()}
                 ),
@@ -375,7 +375,7 @@ def test_bal_call_and_oog(
             account_expectations={
                 call_contract: BalAccountExpectation.empty(),
                 # Bob should only appear if CALL succeeded
-                **({} if fails_at_call else {bob: BalAccountExpectation.empty()}),
+                **({bob: None} if fails_at_call else {bob: BalAccountExpectation.empty()}),
             }
         ),
     )
@@ -450,7 +450,7 @@ def test_bal_delegatecall_and_oog(
                 delegatecall_contract: BalAccountExpectation.empty(),
                 # Target should only appear if DELEGATECALL succeeded
                 **(
-                    {}
+                    {target_contract: None}
                     if fails_at_delegatecall
                     else {target_contract: BalAccountExpectation.empty()}
                 ),
@@ -504,9 +504,10 @@ def test_bal_extcodecopy_and_oog(
 
     # Costs:
     # - 4 PUSH operations = G_VERY_LOW * 4
-    # - EXTCODECOPY cold = G_COLD_ACCOUNT_ACCESS + G_COPY (base cost)
+    # - EXTCODECOPY cold = G_COLD_ACCOUNT_ACCESS + (G_COPY * words)
+    #   where words = ceil32(size) // 32 = ceil32(0) // 32 = 0
     push_cost = gas_costs.G_VERY_LOW * 4
-    extcodecopy_cold_cost = gas_costs.G_COLD_ACCOUNT_ACCESS + gas_costs.G_COPY
+    extcodecopy_cold_cost = gas_costs.G_COLD_ACCOUNT_ACCESS  # + (G_COPY * 0) = 0
     tx_gas_limit = intrinsic_gas_cost + push_cost + extcodecopy_cold_cost
 
     if fails_at_extcodecopy:
@@ -526,7 +527,7 @@ def test_bal_extcodecopy_and_oog(
                 extcodecopy_contract: BalAccountExpectation.empty(),
                 # Target should only appear if EXTCODECOPY succeeded
                 **(
-                    {}
+                    {target_contract: None}
                     if fails_at_extcodecopy
                     else {target_contract: BalAccountExpectation.empty()}
                 ),