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: add unit test for dynamic link trampoline #185

Merged
merged 5 commits into from
May 9, 2022

Conversation

brew0722
Copy link
Contributor

@brew0722 brew0722 commented Apr 27, 2022

Description

Closes : #186

  • add unit test for dynamic link trampoline
  • remove available gas parameter by contract_call
    it's a refactoring for wasmvm side(fix: add gas cost of consumed by callee instance wasmvm#63).
    It is appropriate to merge into separate PRs, but it is included in this PR because of too complex of changing the cosmwasm rev in wasmvm each time it is merged.

Types of changes

  • Bug fix (changes which fixes an issue)
  • New feature (changes which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • ETC (build, ci, docs, perf, refactor, style, test)

Checklist

@brew0722 brew0722 added the dynamic_link relate the dynamic link call feature label Apr 27, 2022
@brew0722 brew0722 self-assigned this Apr 27, 2022
@brew0722 brew0722 changed the base branch from main to dynamic_link April 27, 2022 05:05
@brew0722 brew0722 marked this pull request as ready for review April 28, 2022 09:33
@brew0722 brew0722 requested a review from loloicci April 29, 2022 07:26
@brew0722 brew0722 merged commit 141c988 into Finschia:dynamic_link May 9, 2022
@brew0722 brew0722 deleted the improve_gas_and_test_code branch May 9, 2022 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dynamic_link relate the dynamic link call feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants