fix: mutation Module Permission bug

Author: delcroip

core/models/user.py

@@ -189,6 +189,7 @@ class Meta:
 
 
 class InteractiveUser(VersionedModel):
+    UNIQUE_FIELDS = {"pk", "uuid", "id", "login_name"}
     USE_CACHE = not settings.IS_TESTING
     id = models.AutoField(db_column="UserID", primary_key=True)
     uuid = models.CharField(

core/models/versioned_model.py

@@ -21,7 +21,7 @@ class BaseVersionedModel(CachedModelMixin, models.Model):
 
     @staticmethod
     def filter_validity(validity=None, prefix="", **kwargs):
-        return filter_validity(validity, prefix, **kwargs)
+        return filter_validity(validity=validity, prefix=prefix, **kwargs)
 
     def update(self, *args, **kwargs):
         """

core/schema.py

@@ -1312,7 +1312,7 @@ def resolve_modules_permissions(self, info, **kwargs):
                     ModulePermissionGQLType(module_name=app, permissions=permissions)
                 )
 
-        return ModulePermissionsListGQLType(permissions=config)
+        return ModulePermissionsListGQLType(module_perms_list=config)
 
     def resolve_custom_filters(self, info, **kwargs):
         user = info.context.user

core/tests/test_graphql.py

@@ -202,3 +202,22 @@ def test_fetch_claimadmin(self):
             query, headers={"HTTP_AUTHORIZATION": f"Bearer {self.admin_token}"}
         )
         self.assertResponseNoErrors(response)
+
+    def test_authority_picker(self):
+        query = """
+            query AuthorityPicker {
+                modulesPermissions  {
+                    modulePermsList {
+                        moduleName
+                        permissions {
+                            permsName
+                            permsValue
+                        }
+                    }
+                }
+            }
+        """
+        response = self.query(
+            query, headers={"HTTP_AUTHORIZATION": f"Bearer {self.admin_token}"}
+        )
+        self.assertResponseNoErrors(response)

core/utils.py

@@ -239,7 +239,7 @@ def get(self, *args, **kwargs):
                     "Cache hit for get() with key: %s",
                     get_cache_key(self.model, self._normalize_value(value)),
                 )
-                return cached_qs.first()  # Use first() to get single instance
+                return list(cached_qs)[0]  # Use first() to get single instance
 
         # Fallback to default get() for non-simple queries or cache miss
         instance = super().get(*args, **kwargs)