benchmark: fix calldatacopy in low gas fillings (#1889)

Signed-off-by: Ignacio Hagopian <[email protected]>

Author: jsign

tests/benchmark/test_worst_memory.py

@@ -77,6 +77,15 @@ def test_worst_calldatacopy(
     if size == 0 and non_zero_data:
         pytest.skip("Non-zero data with size 0 is not applicable.")
 
+    # If `non_zero_data` is True, we fill the calldata with deterministic random data.
+    # Note that if `size == 0` and `non_zero_data` is a skipped case.
+    data = Bytes([i % 256 for i in range(size)]) if non_zero_data else Bytes()
+
+    intrinsic_gas_calculator = fork.transaction_intrinsic_cost_calculator()
+    min_gas = intrinsic_gas_calculator(calldata=data)
+    if min_gas > env.gas_limit:
+        pytest.skip("Minimum gas required for calldata ({min_gas}) is greater than the gas limit")
+
     # We create the contract that will be doing the CALLDATACOPY multiple times.
     #
     # If `non_zero_data` is True, we leverage CALLDATASIZE for the copy length. Otherwise, since we
@@ -105,10 +114,6 @@ def test_worst_calldatacopy(
         )
         tx_target = pre.deploy_contract(code=code)
 
-    # If `non_zero_data` is True, we fill the calldata with deterministic random data.
-    # Note that if `size == 0` and `non_zero_data` is a skipped case.
-    data = Bytes([i % 256 for i in range(size)]) if non_zero_data else Bytes()
-
     tx = Transaction(
         to=tx_target,
         gas_limit=env.gas_limit,