chores(fill, pytest): mark slow tests that were not marked as such yet (#2220)

felix314159 · marioevz · web-flow · commit 322fd26ac129 · 2025-10-03T21:29:28.000+02:00
* mark slow tests that were not marked as such yet

* Apply suggestions from code review

---------

Co-authored-by: Mario Vega &lt;marioevz@gmail.com&gt;
diff --git a/tests/berlin/eip2929_gas_cost_increases/test_precompile_warming.py b/tests/berlin/eip2929_gas_cost_increases/test_precompile_warming.py
@@ -79,6 +79,7 @@ def precompile_addresses_in_predecessor_successor(
 )
 @EIPChecklist.Precompile.Test.ForkTransition.Before.Cold(eip=[7951])
 @EIPChecklist.Precompile.Test.ForkTransition.After.Warm(eip=[7951])
+@pytest.mark.slow()
 def test_precompile_warming(
     blockchain_test: BlockchainTestFiller,
     fork: Fork,
diff --git a/tests/berlin/eip2930_access_list/test_tx_intrinsic_gas.py b/tests/berlin/eip2930_access_list/test_tx_intrinsic_gas.py
@@ -145,6 +145,7 @@
     ],
 )
 @pytest.mark.with_all_tx_types(selector=lambda tx_type: tx_type in [1, 2])
+@pytest.mark.slow()
 def test_tx_intrinsic_gas(
     state_test: StateTestFiller,
     tx_type: int,
diff --git a/tests/cancun/eip4844_blobs/test_blob_txs.py b/tests/cancun/eip4844_blobs/test_blob_txs.py
@@ -629,6 +629,7 @@ def test_invalid_block_blob_count(
 @pytest.mark.parametrize("tx_error", [TransactionException.INSUFFICIENT_ACCOUNT_FUNDS], ids=[""])
 @pytest.mark.exception_test
 @pytest.mark.valid_from("Cancun")
+@pytest.mark.slow()
 def test_insufficient_balance_blob_tx(
     state_test: StateTestFiller,
     state_env: Environment,
@@ -1405,6 +1406,7 @@ def test_blob_tx_attribute_gasprice_opcode(
 )
 @pytest.mark.exception_test
 @pytest.mark.valid_at_transition_to("Cancun")
+@pytest.mark.slow()
 def test_blob_type_tx_pre_fork(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/cancun/eip4844_blobs/test_excess_blob_gas.py b/tests/cancun/eip4844_blobs/test_excess_blob_gas.py
@@ -480,6 +480,7 @@ def all_invalid_blob_gas_used_combinations(fork: Fork) -> Iterator[Tuple[int, in
 )
 @pytest.mark.parametrize("parent_blobs", [0])
 @pytest.mark.exception_test
+@pytest.mark.slow()
 def test_invalid_blob_gas_used_in_header(
     blockchain_test: BlockchainTestFiller,
     env: Environment,
@@ -759,6 +760,7 @@ def test_invalid_excess_blob_gas_change(
     lambda fork: range(fork.target_blobs_per_block()),
 )
 @pytest.mark.exception_test
+@pytest.mark.slow()
 def test_invalid_negative_excess_blob_gas(
     blockchain_test: BlockchainTestFiller,
     env: Environment,
@@ -809,6 +811,7 @@ def test_invalid_negative_excess_blob_gas(
     lambda fork: [fork.target_blobs_per_block() + 1],
 )
 @pytest.mark.exception_test
+@pytest.mark.slow()
 def test_invalid_non_multiple_excess_blob_gas(
     blockchain_test: BlockchainTestFiller,
     env: Environment,
diff --git a/tests/cancun/eip4844_blobs/test_point_evaluation_precompile.py b/tests/cancun/eip4844_blobs/test_point_evaluation_precompile.py
@@ -449,6 +449,7 @@ def all_external_vectors() -> List:
 )
 @pytest.mark.parametrize("versioned_hash", [None])
 @pytest.mark.valid_from("Cancun")
+@pytest.mark.slow()
 def test_external_vectors(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/constantinople/eip1014_create2/test_create_returndata.py b/tests/constantinople/eip1014_create2/test_create_returndata.py
@@ -28,6 +28,7 @@
 @pytest.mark.parametrize("create_type", [Op.CREATE, Op.CREATE2])
 @pytest.mark.parametrize("return_type", [Op.RETURN, Op.REVERT])
 @pytest.mark.parametrize("return_type_in_create", [Op.RETURN, Op.REVERT])
+@pytest.mark.slow()
 def test_create2_return_data(
     call_return_size: int,
     create_type: Op,
diff --git a/tests/frontier/opcodes/test_calldatasize.py b/tests/frontier/opcodes/test_calldatasize.py
@@ -19,6 +19,7 @@
     [0, 2, 16, 33, 257],
 )
 @pytest.mark.parametrize("calldata_source", ["contract", "tx"])
+@pytest.mark.slow()
 def test_calldatasize(
     state_test: StateTestFiller,
     fork: Fork,
diff --git a/tests/frontier/opcodes/test_push.py b/tests/frontier/opcodes/test_push.py
@@ -91,6 +91,7 @@ def test_push(state_test: StateTestFiller, fork: Fork, pre: Alloc, push_opcode:
 )
 @pytest.mark.parametrize("stack_height", range(1024, 1026))
 @pytest.mark.valid_from("Frontier")
+@pytest.mark.slow()
 def test_stack_overflow(
     state_test: StateTestFiller, fork: Fork, pre: Alloc, push_opcode: Op, stack_height: int
 ):
diff --git a/tests/frontier/scenarios/test_scenarios.py b/tests/frontier/scenarios/test_scenarios.py
@@ -144,7 +144,7 @@ def scenarios(fork: Fork, pre: Alloc, test_program: ScenarioTestProgram) -> List
         ProgramTstoreTload(),
         ProgramLogs(),
         ProgramSuicide(),
-        pytest.param(ProgramInvalidOpcode(), marks=[pytest.mark.slow()]),
+        ProgramInvalidOpcode(),
         ProgramAddress(),
         ProgramBalance(),
         ProgramOrigin(),
@@ -159,7 +159,7 @@ def scenarios(fork: Fork, pre: Alloc, test_program: ScenarioTestProgram) -> List
         ProgramReturnDataSize(),
         ProgramReturnDataCopy(),
         ProgramExtCodehash(),
-        pytest.param(ProgramBlockhash(), marks=[pytest.mark.slow()]),
+        ProgramBlockhash(),
         ProgramCoinbase(),
         ProgramTimestamp(),
         ProgramNumber(),
@@ -176,6 +176,7 @@ def scenarios(fork: Fork, pre: Alloc, test_program: ScenarioTestProgram) -> List
         ProgramAllFrontierOpcodes(),
     ],
 )
+@pytest.mark.slow()
 def test_scenarios(
     blockchain_test: BlockchainTestFiller,
     fork: Fork,
diff --git a/tests/istanbul/eip152_blake2/test_blake2.py b/tests/istanbul/eip152_blake2/test_blake2.py
@@ -389,6 +389,7 @@
         ),
     ],
 )
+@pytest.mark.slow()
 def test_blake2b(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/osaka/eip7883_modexp_gas_increase/test_modexp_thresholds.py b/tests/osaka/eip7883_modexp_gas_increase/test_modexp_thresholds.py
@@ -39,6 +39,7 @@
 @EIPChecklist.Precompile.Test.InputLengths.Dynamic.Valid()
 @EIPChecklist.GasCostChanges.Test.GasUpdatesMeasurement()
 @pytest.mark.valid_from("Berlin")
+@pytest.mark.slow()
 def test_vectors_from_eip(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/prague/eip2537_bls_12_381_precompiles/test_bls12_g1add.py b/tests/prague/eip2537_bls_12_381_precompiles/test_bls12_g1add.py
@@ -386,6 +386,7 @@ def test_valid(
     ],
 )
 @pytest.mark.parametrize("expected_output", [Spec.INVALID], ids=[""])
+@pytest.mark.slow()
 def test_invalid(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/prague/eip2537_bls_12_381_precompiles/test_bls12_g1msm.py b/tests/prague/eip2537_bls_12_381_precompiles/test_bls12_g1msm.py
@@ -87,6 +87,7 @@
         ),
     ],
 )
+@pytest.mark.slow()
 def test_valid(
     state_test: StateTestFiller,
     pre: Alloc,
@@ -247,6 +248,7 @@ def test_invalid(
         ),
     ],
 )
+@pytest.mark.slow()
 def test_call_types(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/prague/eip2537_bls_12_381_precompiles/test_bls12_g2msm.py b/tests/prague/eip2537_bls_12_381_precompiles/test_bls12_g2msm.py
@@ -89,6 +89,7 @@
         ),
     ],
 )
+@pytest.mark.slow()
 def test_valid(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/prague/eip2537_bls_12_381_precompiles/test_bls12_variable_length_input_contracts.py b/tests/prague/eip2537_bls_12_381_precompiles/test_bls12_variable_length_input_contracts.py
@@ -564,6 +564,7 @@ def test_invalid_length_g2msm(
 @pytest.mark.parametrize("gas_modifier", [pytest.param(0, id="exact_gas")])
 @pytest.mark.parametrize("expected_output", [Spec.PAIRING_TRUE], ids=[""])
 @pytest.mark.parametrize("precompile_address", [Spec.PAIRING])
+@pytest.mark.slow()
 def test_valid_gas_pairing(
     state_test: StateTestFiller,
     env: Environment,
diff --git a/tests/prague/eip7702_set_code_tx/test_gas.py b/tests/prague/eip7702_set_code_tx/test_gas.py
@@ -753,6 +753,7 @@ def gas_test_parameter_args(
 @pytest.mark.parametrize(
     **gas_test_parameter_args(include_pre_authorized=False, execution_gas_allowance=True)
 )
+@pytest.mark.slow()
 def test_gas_cost(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/prague/eip7702_set_code_tx/test_set_code_txs.py b/tests/prague/eip7702_set_code_tx/test_set_code_txs.py
@@ -522,6 +522,7 @@ def test_set_code_to_self_destruct(
 
 
 @pytest.mark.with_all_create_opcodes
+@pytest.mark.slow()
 def test_set_code_to_contract_creator(
     state_test: StateTestFiller,
     pre: Alloc,
@@ -3562,6 +3563,7 @@ def test_creating_delegation_designation_contract(
         ),
     ],
 )
+@pytest.mark.slow()
 def test_many_delegations(
     state_test: StateTestFiller,
     fork: Fork,
@@ -3824,6 +3826,7 @@ def test_set_code_from_account_with_non_delegating_code(
     ],
 )
 @pytest.mark.exception_test
+@pytest.mark.slow()
 def test_set_code_transaction_fee_validations(
     state_test: StateTestFiller,
     pre: Alloc,
diff --git a/tests/shanghai/eip3860_initcode/test_initcode.py b/tests/shanghai/eip3860_initcode/test_initcode.py
@@ -339,6 +339,7 @@ def post(
             return Alloc({create_contract_address: Account(code=initcode.deploy_code)})
         return Alloc({create_contract_address: Account.NONEXISTENT})
 
+    @pytest.mark.slow()
     def test_gas_usage(
         self,
         state_test: StateTestFiller,
@@ -501,6 +502,7 @@ def create2_word_cost(self, opcode: Op, fork: Fork, initcode: Initcode) -> int:
         return ceiling_division(len(initcode), 32) * gas_costs.G_KECCAK_256_WORD
 
     @pytest.mark.xdist_group(name="bigmem")
+    @pytest.mark.slow()
     def test_create_opcode_initcode(
         self,
         state_test: StateTestFiller,
diff --git a/tests/shanghai/eip4895_withdrawals/test_withdrawals.py b/tests/shanghai/eip4895_withdrawals/test_withdrawals.py
@@ -694,6 +694,7 @@ def test_large_amount(
 @pytest.mark.xdist_group(name="bigmem")
 @pytest.mark.parametrize("amount", [0, 1])
 @pytest.mark.with_all_precompiles
+@pytest.mark.slow()
 def test_withdrawing_to_precompiles(
     blockchain_test: BlockchainTestFiller,
     pre: Alloc,

Original file line number	Diff line number	Diff line change
`@@ -79,6 +79,7 @@ def precompile_addresses_in_predecessor_successor(`
`79`	`79`	`)`
`80`	`80`	`@EIPChecklist.Precompile.Test.ForkTransition.Before.Cold(eip=[7951])`
`81`	`81`	`@EIPChecklist.Precompile.Test.ForkTransition.After.Warm(eip=[7951])`
	`82`	`+@pytest.mark.slow()`
`82`	`83`	`def test_precompile_warming(`
`83`	`84`	`blockchain_test: BlockchainTestFiller,`
`84`	`85`	`fork: Fork,`
Original file line number	Diff line number	Diff line change
`@@ -145,6 +145,7 @@`
`145`	`145`	`],`
`146`	`146`	`)`
`147`	`147`	`@pytest.mark.with_all_tx_types(selector=lambda tx_type: tx_type in [1, 2])`
	`148`	`+@pytest.mark.slow()`
`148`	`149`	`def test_tx_intrinsic_gas(`
`149`	`150`	`state_test: StateTestFiller,`
`150`	`151`	`tx_type: int,`
Original file line number	Diff line number	Diff line change
`@@ -449,6 +449,7 @@ def all_external_vectors() -> List:`
`449`	`449`	`)`
`450`	`450`	`@pytest.mark.parametrize("versioned_hash", [None])`
`451`	`451`	`@pytest.mark.valid_from("Cancun")`
	`452`	`+@pytest.mark.slow()`
`452`	`453`	`def test_external_vectors(`
`453`	`454`	`state_test: StateTestFiller,`
`454`	`455`	`pre: Alloc,`
Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,7 @@`
`19`	`19`	`[0, 2, 16, 33, 257],`
`20`	`20`	`)`
`21`	`21`	`@pytest.mark.parametrize("calldata_source", ["contract", "tx"])`
	`22`	`+@pytest.mark.slow()`
`22`	`23`	`def test_calldatasize(`
`23`	`24`	`state_test: StateTestFiller,`
`24`	`25`	`fork: Fork,`
Original file line number	Diff line number	Diff line change
`@@ -91,6 +91,7 @@ def test_push(state_test: StateTestFiller, fork: Fork, pre: Alloc, push_opcode:`
`91`	`91`	`)`
`92`	`92`	`@pytest.mark.parametrize("stack_height", range(1024, 1026))`
`93`	`93`	`@pytest.mark.valid_from("Frontier")`
	`94`	`+@pytest.mark.slow()`
`94`	`95`	`def test_stack_overflow(`
`95`	`96`	`state_test: StateTestFiller, fork: Fork, pre: Alloc, push_opcode: Op, stack_height: int`
`96`	`97`	`):`
Original file line number	Diff line number	Diff line change
`@@ -389,6 +389,7 @@`
`389`	`389`	`),`
`390`	`390`	`],`
`391`	`391`	`)`
	`392`	`+@pytest.mark.slow()`
`392`	`393`	`def test_blake2b(`
`393`	`394`	`state_test: StateTestFiller,`
`394`	`395`	`pre: Alloc,`