chore(ruff): changes to tests/frontier/.

Author: spencer-tb

tests/frontier/__init__.py

@@ -1,3 +1 @@
-"""
-Test cases for EVM functionality introduced in Frontier.
-"""
+"""Test cases for EVM functionality introduced in Frontier."""

tests/frontier/opcodes/__init__.py

@@ -1,3 +1 @@
-"""
-Test for opcodes introduced in Frontier.
-"""
+"""Test for opcodes introduced in Frontier."""

tests/frontier/opcodes/test_all_opcodes.py

@@ -17,16 +17,15 @@
     StateTestFiller,
     Transaction,
 )
-from ethereum_test_tools.vm.opcode import Opcode
+from ethereum_test_tools.vm.opcode import Opcode, UndefinedOpcodes
 from ethereum_test_tools.vm.opcode import Opcodes as Op
-from ethereum_test_tools.vm.opcode import UndefinedOpcodes
 
 REFERENCE_SPEC_GIT_PATH = "N/A"
 REFERENCE_SPEC_VERSION = "N/A"
 
 
 def prepare_stack(opcode: Opcode) -> Bytecode:
-    """Prepare valid stack for opcode"""
+    """Prepare valid stack for opcode."""
     if opcode == Op.CREATE:
         return Op.MSTORE(0, 0x6001600155) + Op.PUSH1(5) + Op.PUSH1(27) + Op.PUSH1(5)
     if opcode == Op.CREATE2:
@@ -39,7 +38,7 @@ def prepare_stack(opcode: Opcode) -> Bytecode:
 
 
 def prepare_suffix(opcode: Opcode) -> Bytecode:
-    """Prepare after opcode instructions"""
+    """Prepare after opcode instructions."""
     if opcode == Op.JUMPI or opcode == Op.JUMP:
         return Op.JUMPDEST
     return Op.STOP
@@ -95,7 +94,7 @@ def test_all_opcodes(state_test: StateTestFiller, pre: Alloc, fork: Fork):
 
 @pytest.mark.valid_from("Cancun")
 def test_cover_revert(state_test: StateTestFiller, pre: Alloc):
-    """Cover state revert from original tests for the coverage script"""
+    """Cover state revert from original tests for the coverage script."""
     tx = Transaction(
         sender=pre.fund_eoa(),
         gas_limit=1_000_000,

tests/frontier/opcodes/test_call_and_callcode_gas_calculation.py

@@ -70,24 +70,20 @@ def callee_code(callee_opcode: Op) -> Bytecode:
         PUSH1 0x01 <- for positive value transfer
         PUSH2 Contract.nonexistent
         GAS <- value doesn't matter
-        CALL/CALLCODE
+        CALL/CALLCODE.
     """
     return callee_opcode(Op.GAS(), 0xFEFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF, 1, 0, 0, 0, 0)
 
 
 @pytest.fixture
 def sender(pre: Alloc) -> EOA:
-    """
-    Sender for all transactions.
-    """
+    """Sender for all transactions."""
     return pre.fund_eoa(0x0BA1A9CE)
 
 
 @pytest.fixture
 def callee_address(pre: Alloc, callee_code: Bytecode) -> Address:
-    """
-    Address of the callee.
-    """
+    """Address of the callee."""
     return pre.deploy_contract(callee_code, balance=0x03)
 
 
@@ -100,7 +96,7 @@ def caller_code(caller_gas_limit: int, callee_address: Address) -> Bytecode:
         PUSH2 caller_gas <- gas limit set for CALL to callee contract
         CALL
         PUSH1 0x00
-        SSTORE
+        SSTORE.
     """
     return Op.SSTORE(0, Op.CALL(caller_gas_limit, callee_address, 0, 0, 0, 0, 0))
 
@@ -114,7 +110,7 @@ def caller_address(pre: Alloc, caller_code: Bytecode) -> Address:
         PUSH2 caller_gas <- gas limit set for CALL to callee contract
         CALL
         PUSH1 0x00
-        SSTORE
+        SSTORE.
     """
     return pre.deploy_contract(caller_code, balance=0x03)
 
@@ -156,7 +152,5 @@ def test_value_transfer_gas_calculation(
     caller_tx: Transaction,
     post: Dict[str, Account],
 ):
-    """
-    Tests the nested CALL/CALLCODE opcode gas consumption with a positive value transfer.
-    """
+    """Tests the nested CALL/CALLCODE opcode gas consumption with a positive value transfer."""
     state_test(env=Environment(), pre=pre, post=post, tx=caller_tx)

tests/frontier/opcodes/test_dup.py

@@ -7,9 +7,8 @@
 import pytest
 
 from ethereum_test_forks import Frontier, Homestead
-from ethereum_test_tools import Account, Alloc, Environment
+from ethereum_test_tools import Account, Alloc, Environment, StateTestFiller, Storage, Transaction
 from ethereum_test_tools import Opcodes as Op
-from ethereum_test_tools import StateTestFiller, Storage, Transaction
 
 
 @pytest.mark.parametrize(
@@ -97,7 +96,7 @@ def test_dup(
     DUP1 copies the first element of the stack (0x10).
     DUP16 copies the 16th element of the stack (0x01).
     """
-    s: Storage.StorageDictType = dict(zip(range(1, 17), range(16, 0, -1)))
+    s: Storage.StorageDictType = dict(zip(range(1, 17), range(16, 0, -1), strict=False))
     s[0] = 16 - (dup_opcode.int() - 0x80)
 
     post[account] = Account(storage=s)

tests/frontier/opcodes/test_selfdestruct.py

@@ -1,12 +1,17 @@
-"""
-Test the SELFDESTRUCT opcode.
-"""
+"""Test the SELFDESTRUCT opcode."""
 
 import pytest
 
-from ethereum_test_tools import Account, Alloc, Block, BlockchainTestFiller, Environment, Initcode
+from ethereum_test_tools import (
+    Account,
+    Alloc,
+    Block,
+    BlockchainTestFiller,
+    Environment,
+    Initcode,
+    Transaction,
+)
 from ethereum_test_tools import Opcodes as Op
-from ethereum_test_tools import Transaction
 
 
 @pytest.mark.valid_from("Frontier")