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: refactor interchaintest and add test for using ibc token as fee #193

Merged
merged 21 commits into from
May 28, 2024

Conversation

tuantran1702
Copy link
Collaborator

@tuantran1702 tuantran1702 commented May 22, 2024

Description

closes #188

Need to bring osmosis files to interchaintest folder due to dependencies conflicts

Checklist

  • Query TWAP by ICQ
  • Proposal
  • IBC transfer
  • IBC transfer with ibc fee

@tuantran1702 tuantran1702 requested a review from tnv1 May 26, 2024 18:15
Copy link
Collaborator

@tnv1 tnv1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks

@tnv1 tnv1 merged commit 16adadb into main May 28, 2024
14 checks passed
tuantran1702 added a commit that referenced this pull request May 28, 2024
tnv1 pushed a commit that referenced this pull request May 29, 2024
* bring changes from #193

* add MsgUpdateParams to x/feeabs

* add test for param.String()

* add keeper test

* update params

* fix and add custom fee test

* fix failing test due to adjust Default Params

* linting
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.

Add interchaintest for using ibc denom fee instead of native
2 participants