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

concentrated-liquidity: add test to check (de)serialization of cl's messages #3929

Merged
merged 12 commits into from
Jan 9, 2023

Conversation

pysel
Copy link
Member

@pysel pysel commented Jan 6, 2023

  • go work sum
  • MsgCreateConcentratedPool (de)ser check
  • test for MsgCreateConcentratedPool
  • MsgWithdrawPosition check
  • MsgWithdrawPosition test case
  • MsgCreatePosition check
  • MsgCreatePosition test case
  • add check to make sure result is expected and add else branch for msgs not yet checked

Closes: #3914

What is the purpose of the change

adds test that makes sure serialization/deserialization of CL's messages work as expected

Documentation and Release Note

no

x/concentrated-liquidity/types/msgs_test.go Outdated Show resolved Hide resolved
app/apptesting/test_suite.go Outdated Show resolved Hide resolved
@pysel pysel marked this pull request as draft January 7, 2023 16:05
@pysel pysel marked this pull request as ready for review January 9, 2023 04:14
@p0mvn p0mvn added the V:state/breaking State machine breaking PR label Jan 9, 2023
@p0mvn p0mvn merged commit 51ffef5 into main Jan 9, 2023
@p0mvn p0mvn deleted the ruslan/cl-test branch January 9, 2023 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:app-wiring Changes to the app folder C:x/concentrated-liquidity V:state/breaking State machine breaking PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test(CL): TestAuthzMsg for CL module
2 participants