-
Notifications
You must be signed in to change notification settings - Fork 58
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 Configurable Verbosity Levels for Execution Traces #601
Conversation
Closes #204 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The core functionality looks great! My only gripe is the complexity of the smart contract test case. A simpler test is always better as long as it covers the same code paths as the test case you have written (which I believe it can).
Additionally, can you make sure that the feature works as expected through the CLI? Thus, build the binary and run medusa fuzz -v
, medusa fuzz -vv
, and medusa fuzz -vvv
on the same piece of test code. Maybe a small stress test of medusa fuzz -verbosity -v
would be good to see how that would work? You can run these commands on the same contract that you wrote the unit test for.
Finally, one of the linters is complaining so make sure to fix that as well.
fuzzing/testdata/contracts/execution_tracing/verbosity_levels.sol
Outdated
Show resolved
Hide resolved
fuzzing/testdata/contracts/execution_tracing/verbosity_levels.sol
Outdated
Show resolved
Hide resolved
…nto dev/add-verbosity-levels
Summary
This PR introduces three distinct verbosity levels for execution traces in Medusa, giving users more control over the level of detail shown in trace outputs. This is especially beneficial for complex systems with multiple call frames, where the current trace output can be overwhelming.
Changes