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

Add test for GetValidatorPerformance #5652

Merged
merged 9 commits into from
Apr 27, 2020
Merged

Add test for GetValidatorPerformance #5652

merged 9 commits into from
Apr 27, 2020

Conversation

0xKiwi
Copy link
Contributor

@0xKiwi 0xKiwi commented Apr 27, 2020

What type of PR is this?
Test addition

What does this PR do? Why is it needed?
This PR adds a test for GetValidatorPerformance which was previously untested and is currently used in the validator client for metrics logging.

@0xKiwi 0xKiwi requested a review from a team as a code owner April 27, 2020 15:57
@0xKiwi 0xKiwi added the Ready For Review A pull request ready for code review label Apr 27, 2020
terencechain
terencechain previously approved these changes Apr 27, 2020
rauljordan
rauljordan previously approved these changes Apr 27, 2020
@0xKiwi 0xKiwi dismissed stale reviews from rauljordan and terencechain via 8b30bd8 April 27, 2020 16:34
@rauljordan rauljordan merged commit 8327c9d into master Apr 27, 2020
@rauljordan rauljordan deleted the test-val-perf branch April 27, 2020 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants