feat(tests): eip7623 - parametrize test_transaction_validity_type_* with a tx to an eoa

[danceratopz]

🗒️ Description

Additionally parametrize the test functions:

1. execution-spec-tests/tests/prague/eip7623_increase_calldata_cost/test_transaction_validity.py

   Line 79 in dcdbfb2

   def test_transaction_validity_type_0(

2. execution-spec-tests/tests/prague/eip7623_increase_calldata_cost/test_transaction_validity.py

   Line 141 in dcdbfb2

   def test_transaction_validity_type_1_type_2(

with a "to" value that corresponds to an EOA.

This adds an additional 96 test cases, due to the existing parametrization these tests. All mainnet clients were tested using consume-engine, no fails were found.

Coverage analysis of the tests themselves revealed that L47 in the to fixture was not hit:

execution-spec-tests/tests/prague/eip7623_increase_calldata_cost/conftest.py

Lines 33 to 51 in dcdbfb2

	@pytest.fixture
	def to(
	request: pytest.FixtureRequest,
	pre: Alloc,
	) -> Address | None:
	"""Create the sender account."""
	if hasattr(request, "param"):
	param = request.param
	else:
	param = Op.STOP
	if param is None:
	return None
	if isinstance(param, Address):
	return param
	if isinstance(param, Bytecode):
	return pre.deploy_contract(param)
	raise ValueError(f"Invalid value for `to` fixture: {param}")

Generate a coverage report locally with:

fill --until Prague tests/prague/eip7623_increase_calldata_cost/test_transaction_validity.py --cov tests --cov-report=html:7623-coverage

🔗 Related Issues

None

✅ Checklist

• All: Set appropriate labels for the changes.
• All: Considered squashing commits to improve commit history.
• All: Added an entry to CHANGELOG.md.

[danceratopz]

@marioevz feel free to close if these bring no value, but then I'd favor updating the to fixture to remove Address as a valid request.param.

[danceratopz]

@marioevz Ran the tests against all clients via consume-engine with no failures.

[spencer-tb]

LGTM! Thanks