Added tests

see #3350

.mergify.yml

@@ -1,43 +1,43 @@
 queue_rules:
   - name: duplicated default from automatic merge on CI success and review
     queue_conditions:
-      - check-success=test (postgres, 3.9)
+      - check-success=test (postgres, 3.12)
       - "#approved-reviews-by>=1"
       - "#review-requested=0"
       - "#changes-requested-reviews-by=0"
       - "#commented-reviews-by=0"
       - label=ready-to-merge
     merge_conditions:
-      - check-success=test (postgres, 3.9)
+      - check-success=test (postgres, 3.12)
     merge_method: squash
   - name: duplicated default from automatic merge for pyUp pull requests
     queue_conditions:
       - author=pyup-bot
-      - check-success=test (postgres, 3.9)
+      - check-success=test (postgres, 3.12)
       - label=dependencies
       - "#approved-reviews-by>=1"
     merge_conditions:
-      - check-success=test (postgres, 3.9)
+      - check-success=test (postgres, 3.12)
     merge_method: squash
   - name: duplicated default from automatic merge for dependabot pull requests
     queue_conditions:
       - author~=^dependabot(|-preview)\[bot\]$
-      - check-success=test (postgres, 3.9)
+      - check-success=test (postgres, 3.12)
       - label=dependencies
       - "#approved-reviews-by>=1"
     merge_conditions:
-      - check-success=test (postgres, 3.9)
+      - check-success=test (postgres, 3.12)
     merge_method: squash
   - name: duplicated default from automatic merge for transifex pull requests
     queue_conditions:
       - author~=^transifex-integration\[bot\]$
       - "#approved-reviews-by>=1"
     merge_conditions:
-      - check-success=test (postgres, 3.9)
+      - check-success=test (postgres, 3.12)
     merge_method: squash
   - name: default
     conditions:
-      - check-success=test (postgres, 3.9)
+      - check-success=test (postgres, 3.12)
 
 pull_request_rules:
   - name: delete head branch after merge

modoboa/core/api/v2/tests.py

@@ -498,6 +498,22 @@ class ComponentAPITestCase(ModoAPITestCase):
         self.assertEqual(resp.status_code, 200)
 
 
+class NotificationAPITestCase(ModoAPITestCase):
+
+    def test_get_notifications(self):
+        url = reverse("v2:notifications")
+        resp = self.client.get(url)
+        self.assertEqual(resp.status_code, 200)
+        data = resp.json()
+        self.assertEqual(len(data), 0)
+
+        self.set_global_parameter("password_scheme", "crypt")
+        resp = self.client.get(url)
+        self.assertEqual(resp.status_code, 200)
+        data = resp.json()
+        self.assertEqual(len(data), 1)
+
+
 class AuthenticatorData(bytes):
 
     credential_data = b"RESPONSE"

modoboa/core/models.py

@@ -48,7 +48,7 @@ class UserManager(BaseUserManager):
         """Check if given password scheme is still in use."""
         return (
             self.get_queryset()
-            .filter(password__startswith=password_hasher.scheme)
+            .filter(password__startswith=password_hasher().scheme)
             .exists()
         )
 

modoboa/core/tests/test_authentication.py

@@ -256,6 +256,16 @@ class AuthenticationTestCase(ModoTestCase):
         hasher = get_password_hasher("crypt")
         self.assertEqual(hasher.label, "crypt (weak, deprecated)")
 
+    def test_is_password_scheme_in_use(self):
+        hasher = get_password_hasher("plain")
+        self.assertTrue(
+            models.User.objects.is_password_scheme_in_use(hasher),
+        )
+        hasher = get_password_hasher("crypt")
+        self.assertFalse(
+            models.User.objects.is_password_scheme_in_use(hasher),
+        )
+
     def test_fido_auth_begin(self):
         url = reverse("core:fido_auth_begin")
         resp = self.client.post(url)

modoboa/core/tests/test_core.py

@@ -8,7 +8,6 @@ from oauth2_provider.models import get_application_model
 
 from django.core import mail
 from django.core import management
-from django.test import TestCase
 from django.urls import reverse
 from django.utils import timezone
 

test_project/test_project/settings.py

@@ -391,6 +391,7 @@ LOGGING = {
 
 SILENCED_SYSTEM_CHECKS = [
     "security.W019",  # modoboa uses iframes to display e-mails
+    "ckeditor.W001",  # CKEditor 4.22.1 warning
 ]
 
 DISABLE_DASHBOARD_EXTERNAL_QUERIES = False