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

test: failing test for imt tree root #135

Conversation

alrevuelta
Copy link
Contributor

Description

This PR adds a failing test for the LazyIMT tree that would be expected to pass. When creating a tree with more than 128 elements inserted, when checking its root() the transaction reverts with the following error. With 128 it works, with 129 (and more) it fails. Note that in both cases, the depth of the tree is enough to fit all elements.

 Error: call revert exception; VM Exception while processing transaction: reverted with panic code 17 [ See: https://links.ethers.org/v5-errors-CALL_EXCEPTION ] (method="root()", data="0x4e487b710000000000000000000000000000000000000000000000000000000000000011", errorArgs=[{"type":"BigNumber","hex":"0x11"}], errorName="Panic", errorSignature="Panic(uint256)", reason=null, code=CALL_EXCEPTION, version=abi/5.7.0)

It this expected?

@alrevuelta
Copy link
Contributor Author

Does anyone have some spare time to look into this?

@vimwitch vimwitch mentioned this pull request Feb 7, 2024
2 tasks
@vimwitch
Copy link
Collaborator

vimwitch commented Feb 7, 2024

Hi @alrevuelta, thanks for submitting this. I've opened a pr with a fix and it should be merged soon.

@vimwitch vimwitch closed this Feb 9, 2024
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.

None yet

2 participants