Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add testing framework for ExEx #8545

Closed
bowenli86 opened this issue Jun 2, 2024 · 0 comments · Fixed by #8636
Closed

Add testing framework for ExEx #8545

bowenli86 opened this issue Jun 2, 2024 · 0 comments · Fixed by #8636
Assignees
Labels
A-exex Execution Extensions C-test A change that impacts how or what we test

Comments

@bowenli86
Copy link

Describe the feature

in order for developers to develop/test exex efficiently, there should be an easy to use testing framework

i've started to look at reth codebase, but found it's hard to start. The largest friction now is reth's lack of testing framework of exex. Eg. there seems to be no testing in reth codebase for anything under /crates/exex, which making it extremely hard to build some integration tests with a local kafka for #8402

Additional context

No response

@bowenli86 bowenli86 added C-enhancement New feature or request S-needs-triage This issue needs to be labelled labels Jun 2, 2024
@shekhirin shekhirin added A-exex Execution Extensions and removed S-needs-triage This issue needs to be labelled labels Jun 3, 2024
@shekhirin shekhirin self-assigned this Jun 6, 2024
@shekhirin shekhirin added C-test A change that impacts how or what we test and removed C-enhancement New feature or request labels Jun 6, 2024
@shekhirin shekhirin changed the title add testing framework for exex Add testing framework for ExEx Jun 6, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in Reth Tracker Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-exex Execution Extensions C-test A change that impacts how or what we test
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants