search

vyperlang / vyper

Pythonic Smart Contract Language for the EVM
https://vyperlang.org
Other
4.81k stars 788 forks source link

fix[test]: fix call graph stability fuzzer #4064

Closed charles-cooper closed 1 month ago

charles-cooper commented 1 month ago

the call graph stability fuzzer could generate an internal function named foo(), which is the same name as the entry function in the test contract.

What I did

fix https://github.com/vyperlang/vyper/issues/4061

How I did it

How to verify it

Commit message

Commit message for the final, squashed PR. (Optional, but reviewers will appreciate it! Please see our commit message style guide for what we would ideally like to see in a commit message.)

Description for the changelog

Cute Animal Picture

![Put a link to a cute animal picture inside the parenthesis-->]()

cyberthirst commented 1 month ago

maybe we rather care about the selector than the name?

charles-cooper commented 1 month ago

maybe we rather care about the selector than the name?

addressed offline but to recap -- call graph stability fuzzer is concerned with internal function calls, not selector table