✨ feat(tests): EIP-7928 net zero balance transactions

[raxhvl]

🗒️ Description

Adds the net zero balance transaction as discussed in this PR to the spec.

✅ Checklist

• All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:

  uvx --with=tox-uv tox -e lint,typecheck,spellcheck,markdownlint

• All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
• All: Set appropriate labels for the changes (only maintainers can apply labels).

[fselmo]

Hey @raxhvl I added a changelog entry and I changed the test up a bit. I'd like to record net-zero transfers even when the starting balance is non-zero. Please take a look at the change here and let me know if you are good with it or let me know your thoughts. We also record the balance of the contract along the way so this can be captured as a sanity check.

I think this is good to go from my end but will wait for your feedback here before we merge.

[raxhvl]

This looks much better, thanks! One concern for the 0->0 net transfer can we still include the SELFDESTRUCT operation? It likely touches a different codepath, and was specifically called out in the draft PR to specs.

[fselmo]

for the 0->0 net transfer can we still include the SELFDESTRUCT operation?

We cover this in the self destruct test self_destruct_in_same_tx=False and pre_funded=False, parametrized case no? Or is this different?

[fselmo]

We cover this in the self destruct test self_destruct_in_same_tx=False and pre_funded=False, parametrized case no? Or is this different?

Ah no it's quite different. The self destruct has no balance change. Yeah, let's add this as a self destruct test variation of the 0 -> 0 case. Can you add this here?

[raxhvl]

I have added the SEFLDESTRUCT case as a parameter and updated the test description.

[fselmo]

lgtm! Thanks!