[chore] Use full_clean instead of saving directly

openwisp · Sep 20, 2024 · ad8ff56 · ad8ff56
1 parent c3f01e5
commit ad8ff56
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/openwisp_notifications/tests/test_admin.py b/openwisp_notifications/tests/test_admin.py
@@ -179,6 +179,7 @@ def test_notification_preferences_button_staff_user(self):
         # Button does not appear for non-staff user
         with self.subTest("Button should not appear for non-staff user"):
             user.is_staff = False
+            user.full_clean()
             user.save()
             response = self.client.get(user_admin_page)
             self.assertNotContains(response, expected_html, html=True)

diff --git a/openwisp_notifications/tests/test_notification_setting.py b/openwisp_notifications/tests/test_notification_setting.py
@@ -287,6 +287,7 @@ def test_global_notification_setting_update(self):
         # Update global settings
         global_setting.email = False
         global_setting.web = False
+        global_setting.full_clean()
         global_setting.save()
 
         with self.subTest(
@@ -298,6 +299,7 @@ def test_global_notification_setting_update(self):
             ).update(email=True)
 
             global_setting.web = True
+            global_setting.full_clean()
             global_setting.save()
 
             self.assertTrue(