Full audit

[nerdstein]

1. identify anything that is broken
2. identify any gaps in automated tests
3. make general observations for improving UX

[nerdstein]

Prepping for next d.o release

[nerdstein]

please note prafful is working to resolve remaining automated test issues

[pbull]

Documenting my findings from the past couple days. I should be able to do some additional review/writing tests over the weekend and early next week:

Config files should not define UUID values

• Resolved: Remove hard-coded UUIDs from config yaml files: Remove hard-coded UUIDs from config yaml files #47

Second-level config (/admin/config/security) page returns a 404

• Resolved: Restores /admin/config/security page: Restores /admin/config/security page #48
• There is still a conflict when multiple modules define this new route.

ConstraintDelete form throws PHP warnings; Form title is missing tokenized constrain name

• Fixes errors in ConstraintDelete form validation and form title: Fixes errors in ConstraintDelete form validation and form title #49

Fixes bugs in PasswordReset behavior: #51
When resetting passwords by role, the following bugs were encountered:

• Users assigned to mutiple roles were being saved more than once (when selecting multiple roles that were assigned to those users)
• Password reset fails when selecting the "Authenticated users" role
• Confirmation message prints the array keys (including unselected values) instead of the role names.
• PasswordReset needs tests.

[nerdstein]

@pbull let me know if you are continuing to work on this

[pbull]

@nerdstein unassigning myself for now, I'll pick this back up if I have more time.