- Issue created by @plopesc
- First commit to issue fork.
- Merge request !16issue/3441321: Created users with perms remove superuser. → (Open) created by pooja_sharma
I have added the permission for createUser() & removed the unrelated code for all test.
Please review, moving NR.
- Status changed to Needs review
6 months ago 3:29pm 11 September 2024 - Status changed to Needs work
6 months ago 8:09am 13 September 2024 - 🇪🇸Spain plopesc Valladolid
Thank you for your work on this one @pooja_sharma.
Checked the MR and might be that the issue summary is not explaining the needs properly.
We are not expecting to refactor the way tests work with user and roles, that could be discussed in a different issue, though.
What we need for this issue is to create a new test for scenarios where the Super user access policy is turned off and confirm that module is still working as expected. See change record → .
- Status changed to Needs review
6 months ago 10:20am 13 September 2024 Tried to add test coverage with super user policy disabled, tried to test the functionality of some piece of code of module , apart from it nothing seems to left , correct me if I 'm missing anything.
Please review, moving NR.