feat(tests): EIP-7928 OOG edge cases for EIP-7702 delegations

[raxhvl]

🗒️ Description

The PR adds two test cases targeting OOG scenarios when accessing 7702 delegation:

1. Case 1: OOG at Authority access; Neither authority or delegation target is expected in BAL.
2. Case 2: OOG at Delegation access; Authority MUST be present but Delegation target is NOT expected in BAL.

The overall goal is to ensure when an account access runs out of gas, it does not make it to BAL.

🔗 Related Issues or PRs

N/A.

✅ Checklist

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

  uvx tox -e static

• All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
• All: Considered adding an entry to CHANGELOG.md.
• All: Considered updating the online docs in the ./docs/ directory.
• All: Set appropriate labels for the changes (only maintainers can apply labels).

Cute Animal Picture

    |
    \\ A _.-''.-""
    /`''`    /
   |__  @)   |
    '-....--'\
              \
              |

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.85%. Comparing base (6044b67) to head (3c37b64).
⚠️ Report is 5 commits behind head on eips/amsterdam/eip-7928.

Additional details and impacted files

@@                   Coverage Diff                    @@
##           eips/amsterdam/eip-7928    #1856   +/-   ##
========================================================
  Coverage                    83.85%   83.85%           
========================================================
  Files                          402      402           
  Lines                        25105    25105           
  Branches                      2287     2287           
========================================================
  Hits                         21051    21051           
  Misses                        3615     3615           
  Partials                       439      439           

Flag	Coverage Δ
unittests	83.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[jochem-brouwer]

Two comments 😄 👍

[jochem-brouwer]

I'm not sure how the test framework translates this, but if it tries to deploy a contract with the designated 7702 delegation pointer then this will not work (contract deployment will fail because it attempts to deploy 0xEF-prefixed code (see EIP-3541)

[raxhvl]

The deployment goes through (we've done the same for the other tests). My understanding that this just adds to the pre-state before invoking EVM so its fine.

cc: @fselmo

[jochem-brouwer]

Ah right, yes this would work to generate the fixtures but it would fail on a live network (uv run execute remote)

[jochem-brouwer]

This test is almost similar to the previous one, right? The only difference is the gas limit of the tx and the expected BAL 🤔 I think those should be merged in one test (and it should be parametrized)

[fselmo]

@raxhvl, we had a lot of changes this time that came from forks/osaka and I rebased here. Just letting you know :)

[fselmo]

I believe all these are covered by PR #1882 that was merged. Can you confirm @raxhvl? Closing for now