fix(test-tests): Use ZeroPaddedHexNumber instead of HexNumber for BALs

[fselmo]

🗒️ Description

Use ZeroPaddedHexNumber for consistency with other fields in test vectors (e.g. 0x0c instead of 0xc)

🔗 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: Set appropriate labels for the changes (only maintainers can apply labels).

Cute Animal Picture

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (eips/amsterdam/eip-7928@ba0eec1). Learn more about missing BASE report.

Additional details and impacted files

@@                    Coverage Diff                     @@
##             eips/amsterdam/eip-7928    #1922   +/-   ##
==========================================================
  Coverage                           ?   86.25%           
==========================================================
  Files                              ?      538           
  Lines                              ?    34561           
  Branches                           ?     3224           
==========================================================
  Hits                               ?    29809           
  Misses                             ?     4165           
  Partials                           ?      587           

Flag	Coverage Δ
unittests	86.25% <ø> (?)

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.

[danceratopz]

Nice, this def looks like it addresses the issue at hand.

Added a dedicated BALs serialization roundtrip test that checks that values are zero-padded correctly here:

• fselmo#33

Did you intentionally leave HexNumber in test_block_access_list_t8n.py? I think we should update them there, too:

https://github.com/fselmo/execution-specs/blob/ab17e0642769c0c9bc4b3a4a34ff21dfa09e7d3d/packages/testing/src/execution_testing/test_types/tests/test_block_access_list_t8n.py

Otherwise, a couple of small comments below.

[fselmo]

Thanks @danceratopz, can you run back through and make sure your comments are addressed and re-review? 🙏🏼

[danceratopz]

Thanks @danceratopz, can you run back through and make sure your comments are addressed and re-review? 🙏🏼

There was just this remaining consistency nit:

Did you intentionally leave HexNumber in test_block_access_list_t8n.py? I think we should update them there, too:
https://github.com/fselmo/execution-specs/blob/ab17e0642769c0c9bc4b3a4a34ff21dfa09e7d3d/packages/testing/src/execution_testing/test_types/tests/test_block_access_list_t8n.py

But approving, as I don't want to block this PR any longer!