Ignore type error

api/forms/session.py

@@ -27,6 +27,16 @@ def get_invalid_login_error_message(self):
         return "The code returned was invalid or expired."
 
     def clean(self):
+        """Authenticates a contributor.
+
+        Raises:
+            ValidationError: If there are form errors.
+            ValidationError: If the contributor's credentials were incorrect.
+            ValidationError: If the contributor's instance is incorrect.
+
+        Returns:
+            The cleaned form data.
+        """
         if self.errors:
             raise ValidationError(
                 "Found form errors. Skipping authentication.",

api/views/session.py

@@ -33,9 +33,10 @@ def get_form_kwargs(self):
     def form_valid(self, form: GitHubLoginForm):  # type: ignore
         contributor = form.contributor
 
-        self.request.session.clear_expired(contributor_id=contributor.pk)
+        # pylint: disable-next=line-too-long
+        self.request.session.clear_expired(contributor_id=contributor.pk)  # type: ignore
 
-        login(self.request, contributor)
+        login(self.request, contributor)  # type: ignore
 
         self.request.session.save()
 

api/views/session_test.py

@@ -20,15 +20,23 @@ def setUp(self):
         self.client = APIClient()
 
     def test_login__invalid_code(self):
-        """Login a user as an existing github user."""
+        """Provided code must be valid and not expired."""
         self.client.post(
             reverse("session-login"),
             data={"code": "7f06468085765cdc1578"},
             format="json",
         )
 
-    def test_login(self):
+    def test_login__form_errors(self):
         """Login a user as an existing github user."""
+        self.client.post(
+            reverse("session-login"),
+            data={"code": ""},
+            format="json",
+        )
+
+    def test_login(self):
+        """User can login with their existing github account."""
         code = "7f06468085765cdc1578"
 
         response_post = requests.Response()