fix: handle no models in response in read_latest_authorization_model

Author: ewanharris

openfga_sdk/client/client.py

@@ -321,7 +321,12 @@ async def read_latest_authorization_model(
         )
         options["page_size"] = 1
         api_response = await self.read_authorization_models(options)
-        return ReadAuthorizationModelResponse(api_response.authorization_models[0])
+        model = (
+            api_response.authorization_models[0]
+            if len(api_response.authorization_models) > 0
+            else None
+        )
+        return ReadAuthorizationModelResponse(model)
 
     #######################
     # Relationship Tuples

openfga_sdk/sync/client/client.py

@@ -319,7 +319,12 @@ def read_latest_authorization_model(self, options: dict[str, int | str] = None):
         )
         options["page_size"] = 1
         api_response = self.read_authorization_models(options)
-        return ReadAuthorizationModelResponse(api_response.authorization_models[0])
+        model = (
+            api_response.authorization_models[0]
+            if len(api_response.authorization_models) > 0
+            else None
+        )
+        return ReadAuthorizationModelResponse(model)
 
     #######################
     # Relationship Tuples

test/client/client_test.py

@@ -615,6 +615,36 @@ async def test_read_latest_authorization_model(self, mock_request):
                 _request_timeout=None,
             )
 
+    @patch.object(rest.RESTClientObject, "request")
+    async def test_read_latest_authorization_model_with_no_models(self, mock_request):
+        """Test case for read_latest_authorization_model when no models are in the store
+
+        Return the latest authorization models configured for the store
+        """
+        response_body = """
+{
+  "authorization_models": []
+}
+        """
+        mock_request.return_value = mock_response(response_body, 200)
+        configuration = self.configuration
+        configuration.store_id = store_id
+        # Enter a context with an instance of the API client
+        async with OpenFgaClient(configuration) as api_client:
+
+            # Return a particular version of an authorization model
+            api_response = await api_client.read_latest_authorization_model(options={})
+            self.assertIsInstance(api_response, ReadAuthorizationModelResponse)
+            self.assertIsNone(api_response.authorization_model)
+            mock_request.assert_called_once_with(
+                "GET",
+                "http://api.fga.example/stores/01YCP46JKYM8FJCQ37NMBYHE5X/authorization-models",
+                headers=ANY,
+                query_params=[("page_size", 1)],
+                _preload_content=ANY,
+                _request_timeout=None,
+            )
+
     @patch.object(rest.RESTClientObject, "request")
     async def test_read_changes(self, mock_request):
         """Test case for read_changes
@@ -2460,7 +2490,6 @@ async def test_list_users(self, mock_request):
                 relation="can_read",
                 user_filters=[
                     UserTypeFilter(type="user"),
-                    UserTypeFilter(type="team", relation="member"),
                 ],
                 context={},
                 contextual_tuples=[
@@ -2521,7 +2550,6 @@ async def test_list_users(self, mock_request):
                     "relation": "can_read",
                     "user_filters": [
                         {"type": "user"},
-                        {"type": "team", "relation": "member"},
                     ],
                     "contextual_tuples": [
                         {

test/sync/client/client_test.py

@@ -614,6 +614,36 @@ def test_read_latest_authorization_model(self, mock_request):
                 _request_timeout=None,
             )
 
+    @patch.object(rest.RESTClientObject, "request")
+    def test_read_latest_authorization_model_with_no_models(self, mock_request):
+        """Test case for read_latest_authorization_model when no models are in the store
+
+        Return the latest authorization models configured for the store
+        """
+        response_body = """
+{
+  "authorization_models": []
+}
+        """
+        mock_request.return_value = mock_response(response_body, 200)
+        configuration = self.configuration
+        configuration.store_id = store_id
+        # Enter a context with an instance of the API client
+        with OpenFgaClient(configuration) as api_client:
+
+            # Return a particular version of an authorization model
+            api_response = api_client.read_latest_authorization_model(options={})
+            self.assertIsInstance(api_response, ReadAuthorizationModelResponse)
+            self.assertIsNone(api_response.authorization_model)
+            mock_request.assert_called_once_with(
+                "GET",
+                "http://api.fga.example/stores/01YCP46JKYM8FJCQ37NMBYHE5X/authorization-models",
+                headers=ANY,
+                query_params=[("page_size", 1)],
+                _preload_content=ANY,
+                _request_timeout=None,
+            )
+
     @patch.object(rest.RESTClientObject, "request")
     def test_read_changes(self, mock_request):
         """Test case for read_changes
@@ -2464,7 +2494,6 @@ def test_list_users(self, mock_request):
             body.relation = "can_read"
             body.user_filters = [
                 UserTypeFilter(type="user"),
-                UserTypeFilter(type="team", relation="member"),
             ]
             body.context = {}
             body.contextual_tuples = [
@@ -2524,7 +2553,6 @@ def test_list_users(self, mock_request):
                     "relation": "can_read",
                     "user_filters": [
                         {"type": "user"},
-                        {"type": "team", "relation": "member"},
                     ],
                     "contextual_tuples": [
                         {