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.

Sign up for GitHub

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

Jump to bottom

test: add tests for x/liquidity/types #111

Merged
merged 29 commits into from
Feb 4, 2022
Merged

test: add tests for x/liquidity/types #111

merged 29 commits into from
Feb 4, 2022

Conversation

hallazzang
Copy link
Contributor

@hallazzang hallazzang commented Jan 25, 2022 •
edited
Loading

Description

Add tests for types package.

Tasks

  • Integration test
  • match.go
  • keys.go
  • msgs.go
  • order.go
  • orderbook.go
  • ordersource.go
  • pair.go
  • params.go
  • pool.go
  • request.go

@hallazzang hallazzang self-assigned this Jan 25, 2022
@hallazzang
style: sort imports
f489f32
@hallazzang
chore: Merge branch 'main' of github.com:cosmosquad-labs/squad into t…
f463ec8 
…esting-1
@hallazzang
chore: Merge branch '93-addr-naming' of github.com:cosmosquad-labs/sq…
587612e 
…uad into testing-1
@hallazzang
chore: Merge branch 'main' of github.com:cosmosquad-labs/squad into t…
ac881bb 
…esting-1
@hallazzang
chore: Merge branch 'main' of github.com:cosmosquad-labs/squad into t…
3d8b4e8 
…esting-1
@hallazzang
chore: Merge branch 'testing-1' of github.com:cosmosquad-labs/squad i…
36d343e 
…nto testing-1
@hallazzang
test: add and update keys tests
7b25173
@hallazzang
chore: Merge branch '114-market-order' of github.com:cosmosquad-labs/…
06cb758 
…squad into testing-1
@hallazzang
chore: Merge branch 'main' of github.com:cosmosquad-labs/squad into t…
7e1ef51 
…esting-1
@hallazzang
test: update params test
19760f6
@hallazzang
test: update tests for requests
37fde95
@hallazzang
test: update tests
70e83df
@hallazzang
chore: Merge branch 'main' of github.com:cosmosquad-labs/squad into t…
5312cbb 
…esting-1
@hallazzang
chore: Merge branch 'main' of github.com:cosmosquad-labs/squad into t…
29cc4b0 
…esting-1
@hallazzang
test: remove outdated test
65f476f
@hallazzang
chore: Merge branch 'main' of github.com:cosmosquad-labs/squad into t…
c4033cc 
…esting-1
@hallazzang
style: sort imports
4ec2813
@hallazzang
test: add integration test skeleton
42a5840
@hallazzang
fix: rename PairEscrowAddr to PairEscrowAddress
cfb6621 
see #93 for the reason
@hallazzang
test: add tests for pair.go
0d2d572
@hallazzang
test: update msg tests
3de6ccb
@hallazzang
fix: fix bug in UserOrder and PoolOrder
9fce5b3
@hallazzang
fix: fix bug in Orders.Sort and add tests
221f5c1
@hallazzang
chore: rename test file
cd15eaf
@hallazzang
Copy link
Contributor Author

Let's add missing tests in a separate PR.

@hallazzang hallazzang marked this pull request as ready for review February 4, 2022 08:04
@hallazzang hallazzang merged commit 9318023 into main Feb 4, 2022
@hallazzang hallazzang deleted the testing-1 branch February 4, 2022 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dongsam dongsam dongsam approved these changes

@jaybxyz jaybxyz Awaiting requested review from jaybxyz

Assignees

@hallazzang hallazzang

Labels
tests x/liquidity
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hallazzang @dongsam