-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
API test for removeSubAdmin #31390
API test for removeSubAdmin #31390
Conversation
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.
Let's also add a scenario where an ordinary user in the group tries to remove the subadmin of their group, and make sure it does not succeed.
And the HTTP status code should be "200" | ||
|
||
Scenario: subadmin tries to remove other subadmin in the group |
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.
indent problem
also needs fixing in line 10 (old problem)
4748b0d
to
f52cdeb
Compare
Codecov Report
@@ Coverage Diff @@
## master #31390 +/- ##
=========================================
Coverage 62.63% 62.63%
Complexity 18255 18255
=========================================
Files 1145 1145
Lines 68472 68472
Branches 1234 1234
=========================================
Hits 42890 42890
Misses 25221 25221
Partials 361 361
Continue to review full report at Codecov.
|
Backport on #31396 |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
API test for removeSubAdmin
This PR has one skipped scenario which will pass only after resolving issue #31276
Related Issue
#31533
How Has This Been Tested?
Locally
Types of changes
Checklist: