feat: enhance user filtering and add lowercase field serializer for API

Author: BryanttV

openedx_authz/rest_api/utils.py

@@ -92,8 +92,8 @@ def filter_users(users: list[dict], search: str | None, roles: list[str] | None)
     Returns:
         list[dict]: The filtered users, preserving the original order.
     """
-    if search:
-        search = search.lower()
+    if not search and not roles:
+        return users
 
     filtered_users = []
     for user in users:

openedx_authz/rest_api/v1/fields.py

@@ -8,8 +8,20 @@ class CommaSeparatedListField(serializers.Field):
 
     def to_internal_value(self, data):
         """Convert string separated by commas to list"""
-        return [item.strip() for item in data.split(",") if item.strip()]
+        return [item.strip().lower() for item in data.split(",") if item.strip()]
 
     def to_representation(self, value):
         """Convert list to string separated by commas"""
-        return ",".join(value)
+        return ",".join(value).lower()
+
+
+class LowercaseCharField(serializers.CharField):
+    """Serializer for a lowercase string."""
+
+    def to_internal_value(self, data):
+        """Convert string to lowercase"""
+        return data.strip().lower()
+
+    def to_representation(self, value):
+        """Convert string to lowercase"""
+        return value.strip().lower()

openedx_authz/rest_api/v1/serializers.py

@@ -4,7 +4,7 @@
 
 from openedx_authz.rest_api.enums import SortField, SortOrder
 from openedx_authz.rest_api.utils import get_user_by_username_or_email
-from openedx_authz.rest_api.v1.fields import CommaSeparatedListField
+from openedx_authz.rest_api.v1.fields import CommaSeparatedListField, LowercaseCharField
 
 
 class ScopeMixin(serializers.Serializer):  # pylint: disable=abstract-method
@@ -53,7 +53,7 @@ class ListUsersInRoleWithScopeSerializer(ScopeMixin):  # pylint: disable=abstrac
     roles = CommaSeparatedListField(required=False)
     sort_by = serializers.ChoiceField(required=False, choices=[(e.value, e.name) for e in SortField])
     order = serializers.ChoiceField(required=False, choices=[(e.value, e.name) for e in SortOrder])
-    search = serializers.CharField(required=False)
+    search = LowercaseCharField(required=False)
 
 
 class ListRolesWithScopeSerializer(ScopeMixin):  # pylint: disable=abstract-method