PPHA-475: Add logout endpoint

Author: Themitchell

lung_cancer_screening/core/middleware/session_timeout_middleware.py

@@ -3,11 +3,6 @@
 from django.contrib.auth import logout
 from django.shortcuts import redirect
 
-try:
-    from django.utils.deprecation import MiddlewareMixin
-except ImportError:
-    MiddlewareMixin = object
-
 
 SESSION_START_KEY = "_session_start_"
 LAST_ACTIVITY_KEY = "_last_activity_"

lung_cancer_screening/questions/tests/unit/views/test_logout.py

@@ -0,0 +1,25 @@
+from django.test import TestCase
+from django.urls import reverse
+from django.conf import settings
+
+from .helpers.authentication import login_user
+
+
+class TestLogout(TestCase):
+
+    def setUp(self):
+        self.user = login_user(self.client)
+
+
+    def test_logout_redirects_to_logout_redirect_url(self):
+        response = self.client.get(reverse("questions:logout"))
+
+        self.assertEqual(response.status_code, 302)
+        self.assertEqual(response.url, settings.LOGOUT_REDIRECT_URL)
+
+
+    def test_logout_destroys_session(self):
+        self.client.get(reverse("questions:logout"))
+
+        session = self.client.session
+        self.assertTrue(session.is_empty())

lung_cancer_screening/questions/urls.py

@@ -33,6 +33,7 @@
 from .views.asbestos_exposure import AsbestosExposureView
 from .views.cancer_diagnosis import CancerDiagnosisView
 from .views.family_history_lung_cancer import FamilyHistoryLungCancerView
+from .views.logout import logout_view
 
 
 urlpatterns = [
@@ -54,4 +55,5 @@
     path('age-range-exit', AgeRangeExitView.as_view(), name='age_range_exit'),
     path('non-smoker-exit', NonSmokerExitView.as_view(), name='non_smoker_exit'),
     path('your-results', YourResultsView.as_view(), name='your_results'),
+    path('logout', logout_view, name='logout'),
 ]

lung_cancer_screening/questions/views/logout.py

@@ -0,0 +1,8 @@
+from django.contrib.auth import logout
+from django.shortcuts import redirect
+from django.conf import settings
+
+
+def logout_view(request):
+    logout(request)
+    return redirect(settings.LOGOUT_REDIRECT_URL)