[deps] Added support for Django >=5.1,<5.3 and Python >=3.12, 3.14 #432

- Dropped support for Python < 3.9
- Dropped support for Django < 4.2

Closes #432

Author: pandafy

.github/workflows/ci.yml

@@ -24,13 +24,29 @@ jobs:
       fail-fast: false
       matrix:
         python-version:
-          - "3.8"
           - "3.9"
           - "3.10"
+          - "3.11"
+          - "3.12"
+          - "3.13"
         django-version:
-          - django~=3.2.0
-          - django~=4.1.0
           - django~=4.2.0
+          - django~=5.0.0
+          - django~=5.1.0
+          - django~=5.2rc0
+        exclude:
+          # Django 5.0+ requires Python >=3.10
+          - python-version: "3.9"
+            django-version: django~=5.0.0
+          - python-version: "3.9"
+            django-version: django~=5.1.0
+          - python-version: "3.9"
+            django-version: django~=5.2rc0
+          # Python 3.13 supported only in Django >=5.1.3
+          - python-version: "3.13"
+            django-version: django~=4.2.0
+          - python-version: "3.13"
+            django-version: django~=5.0.0
 
     steps:
       - uses: actions/checkout@v4
@@ -54,6 +70,7 @@ jobs:
           pip install -U -r requirements-test.txt
           sudo npm install -g prettier
           pip install -e .[rest]
+          pip install -IU "openwisp-utils[qa,selenium] @ https://github.com/openwisp/openwisp-utils/tarball/issues/439-deps"
           pip install -U ${{ matrix.django-version }}
 
       - name: QA checks

openwisp_users/base/models.py

@@ -257,7 +257,11 @@ class Meta:
         abstract = True
 
     def clean(self):
-        if self.user.is_owner(self.organization_id) and not self.is_admin:
+        if (
+            not self._state.adding
+            and self.user.is_owner(self.organization_id)
+            and not self.is_admin
+        ):
             raise ValidationError(
                 _(
                     f'{self.user.username} is the owner of the organization: '

tests/testapp/tests/test_selenium.py

@@ -64,7 +64,7 @@ def test_book_add_form_organization_field(self):
                 visible=Organization.objects.values_list('name', flat=True),
                 hidden=[],
             )
-        self.open(reverse('admin:logout'))
+        self.logout()
 
         with self.subTest('Test organization user: 1 org'):
             self._test_multitenant_autocomplete_org_field(
@@ -81,7 +81,7 @@ def test_book_add_form_organization_field(self):
             )
             self.assertEqual(len(org_select.all_selected_options), 1)
             self.assertEqual(org_select.first_selected_option.text, org1.name)
-        self.open(reverse('admin:logout'))
+        self.logout()
 
         with self.subTest('Test organization user: 2 orgs'):
             self._create_org_user(user=administrator, organization=org2, is_admin=True)
@@ -99,7 +99,7 @@ def test_book_add_form_organization_field(self):
                 self.web_driver.find_element(By.CSS_SELECTOR, '#id_organization')
             )
             self.assertEqual(len(org_select.all_selected_options), 0)
-        self.open(reverse('admin:logout'))
+        self.logout()
 
     def test_shelf_add_form_organization_field(self):
         path = reverse('admin:testapp_shelf_add')
@@ -122,7 +122,7 @@ def test_shelf_add_form_organization_field(self):
                 + ['Shared systemwide (no organization)'],
                 hidden=[],
             )
-        self.open(reverse('admin:logout'))
+        self.logout()
 
         with self.subTest('Test organization user'):
             self._test_multitenant_autocomplete_org_field(
@@ -142,4 +142,4 @@ def test_shelf_add_form_organization_field(self):
             )
             self.assertEqual(len(org_select.all_selected_options), 1)
             self.assertEqual(org_select.first_selected_option.text, org1.name)
-        self.open(reverse('admin:logout'))
+        self.logout()