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.

Sign up for GitHub

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

Jump to bottom

add tests that checks if a table is registered when used #2883

Open

holic opened this issue May 31, 2024 · 0 comments

Member

holic commented May 31, 2024

from #2877 (comment)

we could set up our tests to track storage access, then check that all tables accessed were registered:
https://github.com/foundry-rs/forge-std/blob/52715a217dc51d0de15877878ab8213f6cbbbab5/src/Vm.sol#L406-L407

not sure how hard this will be with our storage slots being hashes of table ID + key, but maybe there's something smart we can do here?

holic mentioned this issue

fix(world-modules): register total supply table in erc20 module #2877

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment