Skip to content

Commit

Permalink
tests/6780: Removed TODO comments now clarified
Browse files Browse the repository at this point in the history
  • Loading branch information
marioevz committed Jul 25, 2023
1 parent 713ed56 commit f14e81e
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions tests/cancun/eip6780_selfdestruct/test_selfdestruct.py
Original file line number Diff line number Diff line change
Expand Up @@ -937,9 +937,6 @@ def test_selfdestruct_created_same_block_different_tx(
)

entry_code += Op.SSTORE(
# entry_code_storage.store_next(keccak256(selfdestruct_code if eip_enabled else b""))
# TODO: Don't really understand why this works. It should be empty if EIP is disabled,
# but it works if it's not
entry_code_storage.store_next(keccak256(bytes(selfdestruct_code))),
Op.EXTCODEHASH(Op.PUSH20(selfdestruct_contract_address)),
)
Expand Down Expand Up @@ -1100,7 +1097,6 @@ def test_delegatecall_from_new_contract_to_pre_existing_contract(
),
)

# TODO: Why is this correct ??
sendall_amount += i

entry_code += Op.SSTORE(
Expand Down Expand Up @@ -1256,7 +1252,6 @@ def test_delegatecall_from_pre_existing_contract_to_new_contract(
),
)

# TODO: Why is this correct ??
sendall_amount += i

entry_code += Op.SSTORE(
Expand Down

0 comments on commit f14e81e

Please sign in to comment.