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

tests(app): 🍖 compare new and existing validator rates #4059

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

cratelyn
Copy link
Contributor

addresses §7.b in #3995.

this introduces new assertions so that the
mock_consensus_can_define_and_delegate_to_a_validator test exercises that the staking component is calculating different rates for validators that were created at different times.

addresses §7.b in #3995.

this introduces new assertions so that the
`mock_consensus_can_define_and_delegate_to_a_validator` test exercises
that the staking component is calculating different rates for validators
that were created at different times.
@cratelyn cratelyn added C-enhancement Category: an enhancement to the codebase A-testing Area: Relates to testing of Penumbra A-mock-consensus Area: Relates to the mock consensus engine labels Mar 20, 2024
@cratelyn cratelyn added this to the Sprint 2 milestone Mar 20, 2024
@cratelyn cratelyn self-assigned this Mar 20, 2024
@cratelyn cratelyn marked this pull request as ready for review March 20, 2024 16:55
@cratelyn cratelyn merged commit 8d09a75 into main Mar 20, 2024
6 checks passed
@cratelyn cratelyn deleted the kate/check-validator-exchange-rates branch March 20, 2024 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-mock-consensus Area: Relates to the mock consensus engine A-testing Area: Relates to testing of Penumbra C-enhancement Category: an enhancement to the codebase
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant