Skip to content

chore(fill,execute): only collect test modules named test_*#1710

Merged
danceratopz merged 2 commits into
ethereum:forks/osakafrom
danceratopz:chore/update-pytest-ini
Oct 30, 2025
Merged

chore(fill,execute): only collect test modules named test_*#1710
danceratopz merged 2 commits into
ethereum:forks/osakafrom
danceratopz:chore/update-pytest-ini

Conversation

@danceratopz

Copy link
Copy Markdown
Member

🗒️ Description

Updates the fill and execute ini files to:

  1. Only collect modules named with a test_ prefix.
  2. Ignore tests/json_infra/, tests/evm_tools/.

🔗 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).
  • Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.
  • Tests: For PRs implementing a missed test case, update the post-mortem document to add an entry the list.
  • Ported Tests: All converted JSON/YML tests from ethereum/tests or tests/static have been assigned @ported_from marker.

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@danceratopz danceratopz added A-test-fill Area: execution_testing.cli.pytest_commands.plugins.filler A-test-execute Area: execution_testing.cli.pytest_commands.plugins.execute labels Oct 29, 2025
@danceratopz danceratopz requested a review from marioevz October 29, 2025 11:14
@spencer-tb

spencer-tb commented Oct 29, 2025

Copy link
Copy Markdown
Contributor

cc #1698 - I can rebase!

@spencer-tb spencer-tb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@danceratopz danceratopz merged commit fa17d51 into ethereum:forks/osaka Oct 30, 2025
10 of 11 checks passed
SamWilsn pushed a commit that referenced this pull request Dec 9, 2025
* fix(tests/zkevm): update jump seq for valid dest

* refactor: update worst jump test case

* refactor: remove unused fork parameter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-test-execute Area: execution_testing.cli.pytest_commands.plugins.execute A-test-fill Area: execution_testing.cli.pytest_commands.plugins.filler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants