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 some keeper and type tests #118

Merged
merged 23 commits into from
Jan 26, 2024
Merged

!test: add some keeper and type tests #118

merged 23 commits into from
Jan 26, 2024

Conversation

neitdung
Copy link
Collaborator

  • Add tests for lock
  • Add tests for unlock
  • Add tests for multi staking coin
  • Add tests for adjust cancel unbond amount and adjust unbond amount

neitdung added 22 commits January 18, 2024 22:04
@neitdung
add some initial tests
55edb3c
@neitdung
Recover cancel unbonding logic
62df77c
@neitdung
Fix lint and remove unused func
60db969
@neitdung
Merge branch 'neitdung/msg-server-bugs' into msg-server-test
b019f7b
@neitdung
Redelegate after cancel unbonding
e49cf92
@neitdung
Merge branch 'neitdung/msg-server-bugs' into msg-server-test
0043dd9
@neitdung
Add some msg server tests
369181a
@neitdung
Fix lint error
ca4784f
@neitdung
Fix redelegate test error
b93efe8
@neitdung
Refactor cancel unbonding msg function
794786a
@neitdung
Merge branch 'neitdung/msg-server-bugs' into msg-server-test
7707559
@neitdung
fix-lint
654fc87
@neitdung
Merge branch 'neitdung/msg-server-bugs' into msg-server-test
a6718e3
@neitdung
Merge branch 'main' of https://github.com/realiotech/multi-staking in…
aa4f840 
…to msg-server-test
@neitdung
Add more checks to staking module
ba4d17b
@neitdung
Merge branch 'msg-server-test' of https://github.com/decentrio/multi-…
f0b8312 
…staking into msg-server-test
@neitdung
Fix linter error
9e03fee
@neitdung
Merge branch 'msg-server-test' into calculation-tests
b3bbd49
@neitdung
Merge branch 'main' of https://github.com/realiotech/multi-staking in…
386e268 
…to calculation-tests
@neitdung
Add some type tests and fix bug truncate int
526af6e
@neitdung
Fix linter errors
01b5731
@neitdung
Add keeper tests
8188f4a
GNaD13
GNaD13 reviewed Jan 25, 2024
View reviewed changes
Copy link
Collaborator

@GNaD13 GNaD13 left a comment

Choose a reason for hiding this comment

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

Hi @neitdung, I've left some comment

x/multi-staking/types/lock_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/lock_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/lock_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/lock_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/lock_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/lock_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/lock_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/multi_staking_coin_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/multi_staking_coin_test.go Outdated Show resolved Hide resolved
x/multi-staking/types/multi_staking_coin_test.go Outdated Show resolved Hide resolved
@catShaark catShaark merged commit 619dbf6 into realiotech:main Jan 26, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GNaD13 GNaD13 GNaD13 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@neitdung @GNaD13 @catShaark