fix: logout route was being served by wrong handler and tests

accounts/tests.py

@@ -99,6 +99,16 @@ class LoginTest(TestCase):
         self.assertEqual(resp.status_code, 302)
         self.assertEqual(resp.headers["location"], reverse("dash.home"))
 
+    def test_logout_works(self):
+        """
+        Logout view tests
+        """
+        c = Client()
+        login_util(t=self, c=c, redirect_to="accounts.home")
+        resp = c.get(reverse("accounts.logout"))
+        self.assertEqual(resp.status_code, 302)
+        self.assertEqual(resp.headers["location"], reverse("accounts.login"))
+
     def test_default_login_uri_works(self):
         """
         /accounts/login should redirect_to /login

accounts/urls.py

@@ -29,7 +29,7 @@ from .views import (
 urlpatterns = [
     path("login/", login_view, name="accounts.login"),
     path("register/", register_view, name="accounts.register"),
-    path("logout/", login_view, name="accounts.logout"),
+    path("logout/", logout_view, name="accounts.logout"),
     path("accounts/login/", default_login_url, name="accounts.default_login_url"),
     path(
         "accounts/verify/pending/<str:public_ref>/",