-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
feat(cmd/influx): update password in cli #16537
Conversation
639d8db
to
b5d14f9
Compare
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.
Can you add some tests for this? This small body of change is a good candidate for the refactoring that is needed to make things testable 👍
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.
looks reallllly good, super excited to see this part of the codebase become testable 😄 . Would be great if we can add some simple test cases to cover the env vars where they are possible.
a29ccea
to
1d87b87
Compare
1d87b87
to
107f71a
Compare
feat(cmd/influx): add user tests fix(cmd/influx): make golint happier
107f71a
to
cf5daee
Compare
Closes #16440
Add the feature, so cli can update user's password. manually tested.