Skip to content

Comments

feat(inspector): make TestInspector public for testing EVM variants#3282

Merged
rakita merged 2 commits intobluealloy:mainfrom
RyanBGG:TestInspector
Jan 8, 2026
Merged

feat(inspector): make TestInspector public for testing EVM variants#3282
rakita merged 2 commits intobluealloy:mainfrom
RyanBGG:TestInspector

Conversation

@RyanBGG
Copy link
Contributor

@RyanBGG RyanBGG commented Jan 6, 2026

Closes #3275

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 6, 2026

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing RyanBGG:TestInspector (7cf0e91) with main (191e2e3)

Summary

✅ 173 untouched benchmarks

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

This PR just makes it public

main task for the issue is to create a function that would take a EVM (or maybe have a closure that returns evm with InspectEvm trait or something similar). It needs to settup the bytecode that is going to be executed, execute it and then track if all fields are called.

Bytecode needs to cover at least CALL/CREATE/LOG/JUMP and precompile call to test how they behave.

@rakita rakita merged commit 40845b2 into bluealloy:main Jan 8, 2026
31 checks passed
@github-actions github-actions bot mentioned this pull request Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Inspector testing bootstrap for evm variants that extend revm

2 participants