Merge lp://staging/~canonical-isd-hackers/canonical-identity-provider/password-db-entry-missing into lp://staging/canonical-identity-provider/release
Proposed by
Szilveszter Farkas
Status: | Merged |
---|---|
Approved by: | Danny Tamez |
Approved revision: | no longer in the source branch. |
Merged at revision: | 17 |
Proposed branch: | lp://staging/~canonical-isd-hackers/canonical-identity-provider/password-db-entry-missing |
Merge into: | lp://staging/canonical-identity-provider/release |
Diff against target: |
423 lines (+123/-122) 5 files modified
identityprovider/auth.py (+7/-2) identityprovider/models/account.py (+2/-1) identityprovider/tests/test_auth.py (+10/-0) identityprovider/tests/test_views_ui.py (+95/-116) identityprovider/views/ui.py (+9/-3) |
To merge this branch: | bzr merge lp://staging/~canonical-isd-hackers/canonical-identity-provider/password-db-entry-missing |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Łukasz Czyżykowski (community) | Approve | ||
Review via email:
|
To post a comment you must log in.
- why in test_authentica te_account_ no_password you're getting object again? looks like this was asserted in the beginning of the test
- in test_reset_ password_ when_account_ {active, active_no_password} I would rather see account.status = AccountStatus. ACTIVE; account.save() instead of the assert, as this more in line of what is checked in those tests
- I feel that the logic in views.ui for accessing account password should be moved to account model