[fix] Made requested changes by coderabbitai

Author: pandafy

docs/user/rest-api.rst

@@ -1020,6 +1020,18 @@ GET (detail)
 
 Returns a single RADIUS user group assignment by its UUID.
 
+PUT
+^^^
+
+Fully updates the RADIUS user group assignment.
+
+======== ==============================================
+Param    Description
+======== ==============================================
+group    UUID of the RADIUS group to assign (optional)
+priority Priority of the assignment (optional, integer)
+======== ==============================================
+
 PATCH
 ^^^^^
 

openwisp_radius/api/serializers.py

@@ -373,8 +373,10 @@ class Meta:
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        if self.context.get("view") and getattr(
-            self.context["view"], "get_parent_queryset", None
+        if (
+            self.context.get("view")
+            and getattr(self.context["view"], "get_parent_queryset", None)
+            and not getattr(self.context["view"], "swagger_fake_view", False)
         ):
             self._user = self.context["view"].get_parent_queryset().first()
         else:

openwisp_radius/api/views.py

@@ -961,9 +961,9 @@ class BaseRadiusUserGroupView(ProtectedAPIMixin, FilterByParentManaged):
     )
 
     def get_queryset(self):
-        qs = super().get_queryset()
         if getattr(self, "swagger_fake_view", False):
-            return qs
+            return self.queryset.none()
+        qs = super().get_queryset()
         return qs.filter(user_id=self.kwargs["user_pk"])
 
     def get_parent_queryset(self):

openwisp_radius/tests/test_api/test_api.py

@@ -26,6 +26,7 @@
 
 from openwisp_radius import settings as app_settings
 from openwisp_radius.api.serializers import (
+    RadiusUserGroupSerializer,
     RadiusUserSerializer,
     UserGroupCheckSerializer,
 )
@@ -1397,7 +1398,7 @@ def test_radius_user_group_list(self):
             OrganizationUser.objects.filter(
                 user=target_user, organization=org2
             ).delete()
-            self._create_org_user(user=target_user, organization=org2, is_admin=True)
+            self._create_org_user(user=admin_user, organization=org2, is_admin=True)
             response = self.client.post(
                 url,
                 {"group": str(org2_group.pk)},
@@ -1519,6 +1520,11 @@ def test_radius_user_group_detail(self):
             org2_rug.refresh_from_db()
             self.assertEqual(org2_rug.priority, 7)
 
+    def test_radius_user_group_serializer_without_view_context(self):
+        serializer = RadiusUserGroupSerializer(context={})
+        self.assertEqual(serializer._user, None)
+        self.assertEqual(serializer.fields["group"].queryset.count(), 0)
+
 
 class TestTransactionApi(AcctMixin, ApiTokenMixin, BaseTransactionTestCase):
     def test_user_radius_usage_view(self):