[qa] Fix merge conflicts and formatting issues

- Resolve remaining git conflict markers in admin.py
- Fix whitespace issue in serializers.py
- Format rest-api.rst documentation

Author: Eeshu-Yadav

docs/user/rest-api.rst

@@ -857,7 +857,7 @@ When using this strategy, in the response you can find the field
 credentials.
 
 Batch retrieve and update
-++++++++++++++++++++++++++
++++++++++++++++++++++++++
 
 .. code-block:: text
 
@@ -870,16 +870,16 @@ Used to retrieve or update a ``RadiusBatch`` instance.
 .. note::
 
     The ``organization`` field is **read-only** for existing batch objects
-    and cannot be changed via the API. This is intentional as changing
-    the organization after batch creation would be inconsistent.
+    and cannot be changed via the API. This is intentional as changing the
+    organization after batch creation would be inconsistent.
 
 Parameters for **GET**:
 
-===== ===========
+===== =================
 Param Description
-===== ===========
+===== =================
 id    UUID of the batch
-===== ===========
+===== =================
 
 Parameters for **PUT**/**PATCH** (only certain fields can be updated):
 

openwisp_radius/admin.py

@@ -463,9 +463,7 @@ def delete_selected_batches(self, request, queryset):
         )
 
     def get_readonly_fields(self, request, obj=None):
-        readonly_fields = super(RadiusBatchAdmin, self).get_readonly_fields(
-            request, obj
-        )
+        readonly_fields = super().get_readonly_fields(request, obj)
         if obj and obj.status != "pending":
             return (
                 "strategy",
@@ -476,7 +474,6 @@ def get_readonly_fields(self, request, obj=None):
                 "users",
                 "expiration_date",
                 "name",
-                "organization",
                 "status",
             ) + readonly_fields
         elif obj:

openwisp_radius/api/serializers.py

@@ -537,7 +537,7 @@ def validate(self, data):
         """
         # Remove organization_slug from data if provided (should not be changeable)
         data.pop("organization_slug", None)
-        
+
         strategy = data.get("strategy") or (self.instance and self.instance.strategy)
 
         if (

openwisp_radius/tests/test_admin.py

@@ -520,6 +520,10 @@ def test_organization_radsettings_freeradius_allowed_hosts(self):
                 "radius_settings-0-id": radsetting.pk,
                 "radius_settings-0-organization": org.pk,
                 "radius_settings-0-password_reset_url": PASSWORD_RESET_URL,
+                "notification_settings-TOTAL_FORMS": 0,
+                "notification_settings-INITIAL_FORMS": 0,
+                "notification_settings-MIN_NUM_FORMS": 0,
+                "notification_settings-MAX_NUM_FORMS": 1,
             }
         )
 
@@ -602,6 +606,10 @@ def test_organization_radsettings_password_reset_url(self):
                 "radius_settings-0-id": radsetting.pk,
                 "radius_settings-0-organization": org.pk,
                 "radius_settings-0-password_reset_url": PASSWORD_RESET_URL,
+                "notification_settings-TOTAL_FORMS": 0,
+                "notification_settings-INITIAL_FORMS": 0,
+                "notification_settings-MIN_NUM_FORMS": 0,
+                "notification_settings-MAX_NUM_FORMS": 1,
             }
         )
 
@@ -1212,6 +1220,10 @@ def test_organization_radsettings_allowed_mobile_prefixes(self):
                 "radius_settings-0-id": radsetting.pk,
                 "radius_settings-0-organization": org.pk,
                 "radius_settings-0-password_reset_url": PASSWORD_RESET_URL,
+                "notification_settings-TOTAL_FORMS": 0,
+                "notification_settings-INITIAL_FORMS": 0,
+                "notification_settings-MIN_NUM_FORMS": 0,
+                "notification_settings-MAX_NUM_FORMS": 1,
             }
         )
 
@@ -1289,6 +1301,10 @@ def test_organization_radsettings_sms_message(self):
                 "radius_settings-0-id": radsetting.pk,
                 "radius_settings-0-organization": org.pk,
                 "radius_settings-0-password_reset_url": PASSWORD_RESET_URL,
+                "notification_settings-TOTAL_FORMS": 0,
+                "notification_settings-INITIAL_FORMS": 0,
+                "notification_settings-MIN_NUM_FORMS": 0,
+                "notification_settings-MAX_NUM_FORMS": 1,
                 "_continue": True,
             }
         )
@@ -1495,30 +1511,56 @@ def test_admin_menu_groups(self):
             html = '<div class="mg-dropdown-label">RADIUS </div>'
             self.assertContains(response, html, html=True)
 
-    def test_radiusbatch_organization_readonly_for_existing_objects(self):
-        """
-        Test that organization field is readonly for existing RadiusBatch objects
-        """
-        batch = self._create_radius_batch(
-            name="test-batch", strategy="prefix", prefix="test-prefix"
-        )
-        url = reverse(f"admin:{self.app_label}_radiusbatch_change", args=[batch.pk])
-
-        response = self.client.get(url)
-        self.assertEqual(response.status_code, 200)
 
-        self.assertContains(response, "readonly")
-        self.assertContains(response, batch.organization.name)
+class TestRadiusGroupAdmin(BaseTestCase):
+    def setUp(self):
+        self.organization = self._create_org()
+        self.admin = self._create_admin()
+        self.organization.add_user(self.admin, is_admin=True)
+        self.client.force_login(self.admin)
+
+    def test_add_radiusgroup_with_inline_check_succeeds(self):
+        add_url = reverse("admin:openwisp_radius_radiusgroup_add")
+
+        post_data = {
+            # Main RadiusGroup form
+            "organization": self.organization.pk,
+            "name": "test-group-with-inline",
+            "description": "A test group created with an inline check",
+            # Inline RadiusGroupCheck formset
+            "radiusgroupcheck_set-TOTAL_FORMS": "1",
+            "radiusgroupcheck_set-INITIAL_FORMS": "0",
+            "radiusgroupcheck_set-0-attribute": "Max-Daily-Session",
+            "radiusgroupcheck_set-0-op": ":=",
+            "radiusgroupcheck_set-0-value": "3600",
+            # Inline RadiusGroupReply formset
+            "radiusgroupreply_set-TOTAL_FORMS": "1",
+            "radiusgroupreply_set-INITIAL_FORMS": "0",
+            "radiusgroupreply_set-0-attribute": "Session-Timeout",
+            "radiusgroupreply_set-0-op": "=",
+            "radiusgroupreply_set-0-value": "1800",
+        }
 
-    def test_radiusbatch_organization_editable_for_new_objects(self):
-        """
-        Test that organization field is editable for new RadiusBatch objects
-        """
-        url = reverse(f"admin:{self.app_label}_radiusbatch_add")
+        response = self.client.post(add_url, data=post_data, follow=True)
 
-        response = self.client.get(url)
         self.assertEqual(response.status_code, 200)
-
-        self.assertContains(response, 'name="organization"')
-        form_html = response.content.decode()
-        self.assertIn('name="organization"', form_html)
+        final_group_name = f"{self.organization.slug}-test-group-with-inline"
+
+        self.assertContains(response, "The group")
+        self.assertContains(response, f"{final_group_name}</a>")
+        self.assertContains(response, "was added successfully.")
+
+        self.assertTrue(RadiusGroup.objects.filter(name=final_group_name).exists())
+        group = RadiusGroup.objects.get(name=final_group_name)
+
+        self.assertEqual(group.radiusgroupcheck_set.count(), 1)
+        check = group.radiusgroupcheck_set.first()
+        self.assertEqual(check.attribute, "Max-Daily-Session")
+        self.assertEqual(check.value, "3600")
+        self.assertEqual(check.groupname, group.name)
+
+        self.assertEqual(group.radiusgroupreply_set.count(), 1)
+        reply = group.radiusgroupreply_set.first()
+        self.assertEqual(reply.attribute, "Session-Timeout")
+        self.assertEqual(reply.value, "1800")
+        self.assertEqual(reply.groupname, group.name)