Opened 9 years ago

Closed 9 years ago

#26882 closed Cleanup/optimization (fixed)

Add test(s) for django.contrib.auth.views.logout_then_login

Reported by: Tim Graham Owned by: Andrew Nester
Component: contrib.auth Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

This view is untested according to the coverage report. Tests likely go in tests/auth_tests/test_views.py.

Change History (6)

comment:1 by Anderson Resende, 9 years ago

Owner: changed from nobody to Anderson Resende
Status: new → assigned

comment:2 by Anderson Resende, 9 years ago

The warning is breaking my test :/
How to remove the warnings at test RemovedInDjango21Warning: The logout() view is superseded by the class-based LogoutView().

comment:3 by Tim Graham, 9 years ago

Replace return logout(request, login_url, extra_context=extra_context) with return LogoutView.as_view(login_url, extra_context=extra_context)(request).

comment:4 by Anderson Resende, 9 years ago

Thx!

comment:5 by Andrew Nester, 9 years ago

Owner: changed from Anderson Resende to Andrew Nester

comment:6 by Tim Graham <timograham@…>, 9 years ago

Resolution: → fixed
Status: assigned → closed

In dde6288f:

Fixed #26882 -- Added tests for auth.views.logout_then_login().

Note: See TracTickets for help on using tickets.
Back to Top