squash!: Apply suggestions

Author: rodmgwgu

openedx_authz/rest_api/v1/serializers.py

@@ -169,14 +169,6 @@ class ListRolesWithScopeResponseSerializer(serializers.Serializer):  # pylint: d
     user_count = serializers.IntegerField()
 
 
-class OrganizationSerializer(serializers.Serializer):  # pylint: disable=abstract-method
-    """Serializer for Organization model."""
-
-    id = serializers.IntegerField()
-    name = serializers.CharField()
-    short_name = serializers.CharField()
-
-
 class UserRoleAssignmentSerializer(serializers.Serializer):  # pylint: disable=abstract-method
     """Serializer for a user role assignment."""
 

openedx_authz/rest_api/v1/urls.py

@@ -12,5 +12,5 @@
     ),
     path("roles/", views.RoleListView.as_view(), name="role-list"),
     path("roles/users/", views.RoleUserAPIView.as_view(), name="role-user-list"),
-    path("orgs/", views.OrgsAPIView.as_view(), name="orgs-view"),
+    path("orgs/", views.OrgsAPIView.as_view(), name="orgs-list"),
 ]

openedx_authz/rest_api/v1/views.py

@@ -12,6 +12,7 @@
 from django.http import HttpRequest
 from edx_api_doc_tools import schema_for
 from organizations.models import Organization
+from organizations.serializers import OrganizationSerializer
 from rest_framework import filters, generics, status
 from rest_framework.response import Response
 from rest_framework.views import APIView
@@ -34,7 +35,6 @@
     ListRolesWithScopeResponseSerializer,
     ListRolesWithScopeSerializer,
     ListUsersInRoleWithScopeSerializer,
-    OrganizationSerializer,
     PermissionValidationResponseSerializer,
     PermissionValidationSerializer,
     RemoveUsersFromRoleWithScopeSerializer,
@@ -507,14 +507,19 @@ class OrgsAPIView(generics.ListAPIView):
             "results": [
                 {
                     "id": 1,
-                    "name": "edX",
-                    "short_name": "edx"
+                    "created": "2026-04-02T19:30:36.779095Z",
+                    "modified": "2026-04-02T19:30:36.779095Z",
+                    "name": "OpenedX",
+                    "short_name": "OpenedX",
+                    "description": "",
+                    "logo": null,
+                    "active": true
                 }
             ]
         }
     """
 
-    queryset = Organization.objects.order_by("name")
+    queryset = Organization.objects.filter(active=True).order_by("name")
     serializer_class = OrganizationSerializer
     pagination_class = AuthZAPIViewPagination
     filter_backends = [filters.SearchFilter]

openedx_authz/tests/rest_api/test_views.py

@@ -874,7 +874,7 @@ def setUpTestData(cls):
     def setUp(self):
         """Set up test fixtures."""
         super().setUp()
-        self.url = reverse("openedx_authz:orgs-view")
+        self.url = reverse("openedx_authz:orgs-list")
 
     def test_get_orgs_returns_all(self):
         """Test that all orgs are returned when no search param is provided.