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(x/gamm): add liquidity event tests (manual backport #2141) #2275

Merged
merged 1 commit into from
Aug 2, 2022

Conversation

p0mvn
Copy link
Member

@p0mvn p0mvn commented Aug 2, 2022

Closes: #XXX

What is the purpose of the change

Manually backporting #2141 since Mergify is not working ref: #2273

There are no state-breaking changes. There are no risks related to the recently discovered gas problem

Testing and Verifying

This change is a trivial rework / code cleanup without any test coverage.

Documentation and Release Note

  • Does this pull request introduce a new feature or user-facing behavior changes? no
  • Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
  • How is the feature or change documented? not applicable

@github-actions github-actions bot added C:x/gamm Changes, features and bugs related to the gamm module. C:x/epochs labels Aug 2, 2022
* test(gamm): add liquidity events

* Apply suggestions from code review

* Update x/gamm/keeper/msg_server_test.go

Co-authored-by: Matt, Park <45252226+mattverse@users.noreply.github.com>

Co-authored-by: Matt, Park <45252226+mattverse@users.noreply.github.com>
@p0mvn p0mvn force-pushed the roman/add-liq-events-backport branch from 09ec894 to f333a67 Compare August 2, 2022 17:58
@p0mvn p0mvn marked this pull request as ready for review August 2, 2022 18:02
@p0mvn p0mvn requested a review from a team August 2, 2022 18:02
Copy link
Member

@czarcas7ic czarcas7ic left a comment

Choose a reason for hiding this comment

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

ACK

@p0mvn p0mvn merged commit 18937bc into v11.x Aug 2, 2022
@p0mvn p0mvn deleted the roman/add-liq-events-backport branch August 2, 2022 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:x/gamm Changes, features and bugs related to the gamm module.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants