Skip to content

[User Registration][Testing] New Registration & Auth Flow Broke "Users" App Tests #192

Open
@BethanyG

Description

@BethanyG

The changes in PR #187 once merged will cause the tests under users/tests/ to mostly fail (6 out of 9).

Currently, these are marked skip, but need to be re-written to work (or the app needs to change to make them work).

Failing tests are:

 -   users/tests/test_models.py:7 test_user_get_absolute_url
 -   users/tests/test_urls.py:8 test_detail
 -   users/tests/test_urls.py:16 test_update
 -   users/tests/test_urls.py:21 test_redirect
 -   users/tests/test_views.py:19 TestUserUpdateView.test_get_success_url
 -   users/tests/test_views.py:43 TestUserRedirectView.test_get_redirect_url

Suspect they can mostly be fixed with some URL changes. Since the "users" app is largely unused due to DRF, these were seen as lower-priority tests, and I didn't hold #187 for them.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingneeds discussionThe fix for this issue needs discussionpinned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions