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(gamm): CalcJoinPoolNoSwapShares, CalcJoinPoolShares, CalcExitPoolCoinsFromShares state alteration tests #3378

Merged
merged 1 commit into from
Nov 16, 2022

Conversation

p0mvn
Copy link
Member

@p0mvn p0mvn commented Nov 14, 2022

Closes: #XXX

What is the purpose of the change

Add state alteration tests for the newly to be whitelisted queries: #3217

@github-actions github-actions bot added C:CLI C:x/gamm Changes, features and bugs related to the gamm module. labels Nov 14, 2022
@p0mvn p0mvn added V:state/compatible/backport State machine compatible PR, should be backported A:backport/v13.x backport patches to v13.x branch labels Nov 14, 2022
@p0mvn p0mvn marked this pull request as ready for review November 14, 2022 15:57
@p0mvn p0mvn requested a review from a team November 14, 2022 15:57
@ValarDragon ValarDragon merged commit c24b1e1 into main Nov 16, 2022
@ValarDragon ValarDragon deleted the roman/query-tests branch November 16, 2022 07:47
mergify bot pushed a commit that referenced this pull request Nov 16, 2022
…lCoinsFromShares state alteration tests (#3378)

(cherry picked from commit c24b1e1)
p0mvn pushed a commit that referenced this pull request Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:backport/v13.x backport patches to v13.x branch C:CLI C:x/gamm Changes, features and bugs related to the gamm module. V:state/compatible/backport State machine compatible PR, should be backported
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants