-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add staking module integration tests #34
Add staking module integration tests #34
Conversation
…o prathyusha/staking_module
…o prathyusha/staking_module
…o prathyusha/staking_module
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @PrathyushaLakkireddy 🎉, just small nits
internal/modules/staking/test.py
Outdated
if sender == validator: | ||
sys.exit( | ||
'Error: The values of arguments "sender" and "receiver" are equal make sure to set different values' | ||
) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this canot be true as there will be different prefixes
if sender == validator: | |
sys.exit( | |
'Error: The values of arguments "sender" and "receiver" are equal make sure to set different values' | |
) |
internal/modules/staking/test.py
Outdated
if sender == validator: | ||
sys.exit( | ||
'Error: The values of arguments "sender" and "receiver" are equal make sure to set different values' | ||
) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this canot be true as there will be different prefixes
if sender == validator: | |
sys.exit( | |
'Error: The values of arguments "sender" and "receiver" are equal make sure to set different values' | |
) |
internal/modules/staking/test.py
Outdated
# assign the arguments | ||
sender, amount_to_be_sent = delegator, 5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# assign the arguments | |
sender, amount_to_be_sent = delegator, 5 | |
# assign the arguments | |
amount = 5 |
ref: #20
This PR adds all the txs and query integration tests for x/staking module