Lint.

harvard-lil · Jan 17, 2025 · 45f914d · 45f914d
1 parent d341e9d
commit 45f914d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/perma_web/perma/tests/test_views_user_management.py b/perma_web/perma/tests/test_views_user_management.py
@@ -1152,7 +1152,7 @@ def test_can_remove_user_from_registrar(user_type, request, client, link_user):
     link_user.refresh_from_db()
     assert link_user.is_registrar_user()
 
-    response = submit_form(
+    submit_form(
         client,
         url=reverse('user_management_manage_single_registrar_user_remove', args=[link_user.id]),
         success_url=reverse('user_management_manage_registrar_user')
@@ -1177,7 +1177,7 @@ def test_registrar_cannot_remove_unrelated_user_from_registrar(client, registrar
 def test_can_remove_self_from_registrar(client, registrar_user):
     client.force_login(registrar_user)
 
-    response = submit_form(
+    submit_form(
         client,
         url=reverse('user_management_manage_single_registrar_user_remove', args=[registrar_user.id]),
         success_url=reverse('create_link')

diff --git a/perma_web/perma/views/user_management.py b/perma_web/perma/views/user_management.py
@@ -914,7 +914,7 @@ def target_user_valid(self):
                 return False, f"{self.object} belongs to organizations that are not controlled by your registrar. You cannot make them a registrar unless they leave those organizations."
 
         if self.object.registrar_id:
-            if not 'registrar' in self.get_form().changed_data:
+            if 'registrar' not in self.get_form().changed_data:
                 return False, f"{self.object} is already a registrar user for that registrar."
 
         if len(set(org.registrar_id for org in self.object.organizations.all())) > 1: