We're working with community non-profits who have a Host Home or empty bedrooms initiative to develop a workflow management tool to make the process scalable (across all providers), reduce institutional bias, and effectively capture data.
Make admin_controller.remove_user function only attempt to delete a Cognito user when that operation is requested.
Formatted the code using yapf.
Rationale behind the changes?
A weak password test was failing because this function was throwing an AuthError because the remove_user function was attempting to remove a non-existing user from Cognito.
Testing done for these changes
I re-ran tox in a virtual environment
What did you learn or can share that is new?(optional)
The authorization/user management code could be redesigned.
The API code needs to follow consistent formatting and style.
Closes #759
What changes did you make?
admin_controller.remove_userfunction only attempt to delete a Cognito user when that operation is requested.yapf.Rationale behind the changes?
AuthErrorbecause theremove_userfunction was attempting to remove a non-existing user from Cognito.Testing done for these changes
toxin a virtual environmentWhat did you learn or can share that is new?(optional)