-
Notifications
You must be signed in to change notification settings - Fork 182
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-only][full-ci] added test for updating admin password via cli #9593
Conversation
aa0e7ec
to
d65b61f
Compare
d65b61f
to
b876bba
Compare
tests/acceptance/features/cliCommands/resetUserPassword.feature
Outdated
Show resolved
Hide resolved
f0c832e
to
1b6817d
Compare
1b6817d
to
16cc5f2
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.
👍
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.
@S-Panta Could you squash the commits or reword to meaningful one?
16cc5f2
to
52a0c34
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.
LGTM
Quality Gate passedIssues Measures |
[tests-only][full-ci] added test for updating admin password via cli
Description
Added cli test for updating administrator credentials.
Previously, I directly changed the credential of administration but doing so fails the CI on ociswrapper as the admin credential is hardcoded.
Reference: https://drone.owncloud.com/owncloud/ocis/37023/43/5
due to
ocis/.drone.star
Line 2130 in ce199d5
Therefore, creating a new admin user to cover this scenario.
Related Issue
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: