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
Voluntary exit CLI/Client integration #7162
Conversation
# Conflicts: # validator/accounts/v2/accounts_exit.go
@@ -81,3 +81,5 @@ func TestExitAccountsCli_EmptyWalletReturnsError(t *testing.T) { | |||
err = ExitAccountsCli(cliCtx, os.Stdin) | |||
assert.ErrorContains(t, "wallet is empty, no accounts to perform voluntary exit", err) | |||
} | |||
|
|||
// TODO: Write an integration test? |
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.
yes, please add test
prysm/validator/accounts/v2/accounts_exit.go Lines 168 to 169 in c3eef97
I am New ing the clients and because of this I cannot use gomock to mock them. Is there any standard pattern that should be used here? Initialize them inside main() and pass into ExitAccountsCli() so that I can pass a mock implementation in tests?
|
# Conflicts: # validator/client/propose.go
Codecov Report
@@ Coverage Diff @@
## master #7162 +/- ##
==========================================
- Coverage 61.64% 61.53% -0.11%
==========================================
Files 411 410 -1
Lines 32628 32657 +29
==========================================
- Hits 20112 20095 -17
- Misses 9647 9693 +46
Partials 2869 2869 |
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.
Nice work on this one!
What type of PR is this?
Feature
What does this PR do? Why is it needed?
The aim of this PR is to provide full integration between the CLI command for performing an exit, the validator and the gRPC validator client.
Which issues(s) does this PR fix?
Part of #6882
Other notes for review
N/A