Skip to content

Conversation

@chfast
Copy link
Member

@chfast chfast commented Dec 8, 2025

πŸ—’οΈ Description

This brings the copy-paste BytesConcatenation from tests/ to ethereum_testing.base_types.

πŸ”— 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).

@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 87.31%. Comparing base (9523e75) to head (0cbb3f3).
⚠️ Report is 11 commits behind head on forks/osaka.

Additional details and impacted files
@@             Coverage Diff              @@
##           forks/osaka    #1858   +/-   ##
============================================
  Coverage        87.31%   87.31%           
============================================
  Files              541      541           
  Lines            32832    32832           
  Branches          3015     3015           
============================================
  Hits             28668    28668           
  Misses            3557     3557           
  Partials           607      607           
Flag Coverage Ξ”
unittests 87.31% <ΓΈ> (ΓΈ)

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.

@spencer-tb spencer-tb added C-refactor Category: refactor A-test-tests Area: tests for packages/testing A-test-types Area: execution_testing.base_types and execution_testing.test_types labels Dec 8, 2025
@spencer-tb spencer-tb changed the title enhance(testing): add common BytesConcatenation to execution_testing refactor(test-types): add common BytesConcatenation to execution_testing Dec 8, 2025
Copy link
Contributor

@spencer-tb spencer-tb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very welcome change from my side. LGTM

Copy link
Member

@marioevz marioevz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@marioevz marioevz merged commit 55d6595 into ethereum:forks/osaka Dec 8, 2025
14 checks passed
SamWilsn pushed a commit that referenced this pull request Dec 9, 2025
* feat(tests): add codecopy and extcodecopy for clz opcode

* fix: resolve rebase issue

* refactor: combine codecopy operation test

* feat(tests): load memory for clz opcode parameter

* fix(tests): correct clz result storage in opcode tests

* fix(tests): add prestate storage for CLZ opcode tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-test-tests Area: tests for packages/testing A-test-types Area: execution_testing.base_types and execution_testing.test_types C-refactor Category: refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants