fix(tests): Use pytest.mark.valid_before for EIP-8037

Author: marioevz

tests/osaka/eip7825_transaction_gas_limit_cap/test_tx_gas_limit.py

@@ -90,7 +90,7 @@ def tx_gas_limit_cap_tests(fork: Fork) -> List[ParameterSet]:
 @pytest.mark.parametrize_by_fork("tx_gas_limit,error", tx_gas_limit_cap_tests)
 @pytest.mark.with_all_tx_types
 @pytest.mark.valid_from("Prague")
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_transaction_gas_limit_cap(
     state_test: StateTestFiller,
     pre: Alloc,
@@ -345,7 +345,7 @@ def total_cost_floor_per_token(fork: Fork) -> int:
 )
 @pytest.mark.parametrize("zero_byte", [True, False])
 @pytest.mark.valid_from("Osaka")
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 @pytest.mark.eels_base_coverage
 def test_tx_gas_limit_cap_full_calldata(
     state_test: StateTestFiller,
@@ -480,7 +480,7 @@ def test_tx_gas_limit_cap_contract_creation(
     ],
 )
 @pytest.mark.valid_from("Osaka")
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_tx_gas_limit_cap_access_list_with_diff_keys(
     state_test: StateTestFiller,
     exceed_tx_gas_limit: bool,
@@ -567,7 +567,7 @@ def intrinsic_cost_for_num_storage_keys(storage_key_count: int) -> int:
     ],
 )
 @pytest.mark.valid_from("Osaka")
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_tx_gas_limit_cap_access_list_with_diff_addr(
     state_test: StateTestFiller,
     pre: Alloc,
@@ -648,7 +648,7 @@ def intrinsic_cost_for_num_accounts(account_count: int) -> int:
     ],
 )
 @pytest.mark.valid_from("Osaka")
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_tx_gas_limit_cap_authorized_tx(
     state_test: StateTestFiller,
     pre: Alloc,

tests/prague/eip6110_deposits/test_deposits.py

@@ -716,7 +716,7 @@
                 ),
             ],
             id="single_deposit_from_contract_call_depth_high",
-            marks=pytest.mark.valid_until("EIP8037"),
+            marks=pytest.mark.valid_before("EIP8037"),
         ),
         pytest.param(
             [

tests/prague/eip7623_increase_calldata_cost/test_execution_gas.py

@@ -72,7 +72,7 @@ def to(
                 True,
                 [Address(1)],
                 id="type_4",
-                marks=pytest.mark.valid_until("EIP8037"),
+                marks=pytest.mark.valid_before("EIP8037"),
             ),
         ],
         indirect=["authorization_list"],

tests/prague/eip7623_increase_calldata_cost/test_refunds.py

@@ -342,7 +342,7 @@ def tx_gas_limit(
 )
 # TODO[EIP-8037]: Authorization state gas split affects
 # refund calculations for Amsterdam.
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_gas_refunds_from_data_floor(
     state_test: StateTestFiller,
     pre: Alloc,

tests/prague/eip7623_increase_calldata_cost/test_transaction_validity.py

@@ -159,7 +159,7 @@ def test_transaction_validity_type_0(
 # TODO[EIP-8037]: Contract creation state gas
 # (G_TRANSACTION_CREATE) split affects intrinsic gas
 # calculation for Amsterdam.
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_transaction_validity_type_1_type_2(
     state_test: StateTestFiller,
     pre: Alloc,

tests/prague/eip7702_set_code_tx/test_gas.py

@@ -841,7 +841,7 @@ def gas_test_parameter_args(
     )
 )
 @pytest.mark.slow()
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_gas_cost(
     state_test: StateTestFiller,
     pre: Alloc,
@@ -1120,7 +1120,7 @@ def test_account_warming(
 # TODO[EIP-8037]: EELS uses PER_EMPTY_ACCOUNT_COST=25,000
 # per auth for intrinsic gas check, but Amsterdam
 # G_AUTHORIZATION=165,990 includes state gas. EELS bug?
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_intrinsic_gas_cost(
     state_test: StateTestFiller,
     pre: Alloc,

tests/prague/eip7702_set_code_tx/test_set_code_txs.py

@@ -2945,7 +2945,7 @@ def test_set_code_to_precompile(
 
 
 @pytest.mark.with_all_precompiles
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_set_code_to_precompile_not_enough_gas_for_precompile_execution(
     state_test: StateTestFiller,
     pre: Alloc,

tests/prague/eip7702_set_code_tx/test_set_code_txs_2.py

@@ -38,7 +38,7 @@
 @pytest.mark.valid_from("Prague")
 # TODO[EIP-8037]: Amsterdam expected_loop_count needs
 # recalculating due to state gas.
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 # TODO[EIP-8037]: Fix Storage.KeyValueMismatchError for
 # contract_loop expected values.
 @pytest.mark.skip(
@@ -694,7 +694,7 @@ class AccessListTo(Enum):
     [AccessListTo.POINTER_ADDRESS, AccessListTo.CONTRACT_ADDRESS],
 )
 @pytest.mark.valid_from("Prague")
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_gas_diff_pointer_vs_direct_call(
     blockchain_test: BlockchainTestFiller,
     pre: Alloc,
@@ -909,7 +909,7 @@ def test_gas_diff_pointer_vs_direct_call(
 
 
 @pytest.mark.valid_from("Prague")
-@pytest.mark.valid_until("EIP8037")
+@pytest.mark.valid_before("EIP8037")
 def test_pointer_call_followed_by_direct_call(
     state_test: StateTestFiller,
     pre: Alloc,

tests/shanghai/eip3860_initcode/test_initcode.py

@@ -404,7 +404,7 @@ def post(
     # TODO[EIP-8037]: Code deposit and G_CREATE become
     # state gas under Amsterdam.
     # Gas calculations need updating for two-dimensional gas.
-    @pytest.mark.valid_until("EIP8037")
+    @pytest.mark.valid_before("EIP8037")
     @pytest.mark.slow()
     def test_gas_usage(
         self,
@@ -611,7 +611,7 @@ def code_deposit_gas(self, fork: Fork, initcode: Initcode) -> int:
     # TODO[EIP-8037]: Code deposit and G_CREATE become
     # state gas under Amsterdam.
     # Gas calculations need updating for two-dimensional gas.
-    @pytest.mark.valid_until("EIP8037")
+    @pytest.mark.valid_before("EIP8037")
     @pytest.mark.xdist_group(name="bigmem")
     @pytest.mark.slow()
     def test_create_opcode_initcode(

tests/tangerine_whistle/eip150_operation_gas_costs/test_eip150_selfdestruct.py

@@ -329,7 +329,7 @@ def build_post_state(
     [0, 1],
     ids=["dead_beneficiary", "alive_beneficiary"],
 )
-@pytest.mark.valid_until("EIP8037")  # TODO[EIP-8037]: Fix for Amsterdam
+@pytest.mark.valid_before("EIP8037")  # TODO[EIP-8037]: Fix for Amsterdam
 def test_selfdestruct_to_account(
     pre: Alloc,
     blockchain_test: BlockchainTestFiller,
@@ -583,7 +583,7 @@ def test_selfdestruct_state_access_boundary(
     ],
 )
 @pytest.mark.valid_from("TangerineWhistle")
-@pytest.mark.valid_until("EIP8037")  # TODO[EIP-8037]: Fix for Amsterdam
+@pytest.mark.valid_before("EIP8037")  # TODO[EIP-8037]: Fix for Amsterdam
 def test_selfdestruct_to_precompile(
     pre: Alloc,
     blockchain_test: BlockchainTestFiller,