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

Integration tests fail with "VM error" #508

Closed
maurolacy opened this issue Aug 24, 2020 · 3 comments · Fixed by #511
Closed

Integration tests fail with "VM error" #508

maurolacy opened this issue Aug 24, 2020 · 3 comments · Fixed by #511

Comments

@maurolacy
Copy link
Contributor

maurolacy commented Aug 24, 2020

An integration test that queries a contract for a list of ids is failing with "VM error:".

Tried it in cosmwasm-plus (see branch https://github.com/CosmWasm/cosmwasm-plus/tree/atomic-swap-integ_test_query for PoC).

Reproduced also in coswasm, under the queue contract.

The errors are of the form:

thread 'test_query' panicked at 'VM error: CommunicationErr { source: InvalidRegion { source: LengthExceedsCapacity { length: 1931622946, capacity: 1919247470, backtrace: Backtrace(()) } } }', ...

,

thread 'query_list' panicked at 'VM error: CommunicationErr { source: RegionLengthTooBig { length: 1663187490, max_length: 65536, backtrace: Backtrace(()) } }', ...

The problem seems to be related to the use of cosmwasm-storage. Or, maybe, adding cosmwasm-storage as new dependency changed the versions of some packages, and that is what is triggering the error.

Creating this issue for tracking and documentation.

@ethanfrey
Copy link
Member

ethanfrey commented Aug 24, 2020

I just gave you permission to push a branch. Can you please make a PR with the test showing the failure?

Good find and I hope to fix it shortly

@maurolacy
Copy link
Contributor Author

maurolacy commented Aug 24, 2020

PR: #509.

Tomorrow I'll continue testing. If I find something else I'll publish it here.

@webmaster128
Copy link
Member

Nice find! This is most likely a bug in cosmwasm-std.

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 a pull request may close this issue.

3 participants