fix: method/type fixes

Author: DMedina559

src/bsm_api_client/client/_content_methods.py

@@ -376,7 +376,7 @@ async def async_prune_server_backups(self, server_name: str) -> ActionResponse:
             server_name,
         )
         response = await self._request(
-            "POST",
+            "PUT",
             f"/server/{server_name}/backups/prune",
             json_data=None,
             authenticated=True,

src/bsm_api_client/client/_manager_methods.py

@@ -25,6 +25,7 @@
     PruneDownloadsResponse,
     SettingItemResponse,
     SettingsResponse,
+    ThemeListResponse,
 )
 
 _LOGGER = logging.getLogger(__name__.split(".")[0] + ".client.manager")
@@ -105,15 +106,17 @@ async def async_get_custom_zips(self) -> CustomZipsResponse:
         )
         return cast(CustomZipsResponse, CustomZipsResponse.model_validate(response))
 
-    async def async_get_themes(self) -> Dict[str, Any]:
+    async def async_get_themes(self) -> ThemeListResponse:
         """Retrieves a list of available themes.
 
         Returns:
-            A dictionary containing the list of themes.
+            A `ThemeListResponse` containing the list of themes.
         """
         _LOGGER.info("Fetching list of available themes.")
-        result = await self._request(method="GET", path="/themes", authenticated=True)
-        return dict(result)
+        result = await self._request(
+            method="GET", path="/info/themes", authenticated=True
+        )
+        return cast(ThemeListResponse, ThemeListResponse.model_validate(result))
 
     async def async_get_all_settings(self) -> SettingsResponse:
         """Retrieve all global application settings.
@@ -123,7 +126,7 @@ async def async_get_all_settings(self) -> SettingsResponse:
         """
         _LOGGER.info("Fetching all global application settings.")
         response = await self._request(
-            method="GET", path="/settings", authenticated=True
+            method="GET", path="/settings/get", authenticated=True
         )
         return cast(SettingsResponse, SettingsResponse.model_validate(response))
 
@@ -141,7 +144,7 @@ async def async_set_setting(self, payload: SettingItemResponse) -> SettingsRespo
         )
         response = await self._request(
             method="POST",
-            path="/settings",
+            path="/settings/set",
             json_data=payload.model_dump(),
             authenticated=True,
         )
@@ -228,7 +231,7 @@ async def async_prune_downloads(
         )
 
         response = await self._request(
-            method="POST",
+            method="PUT",
             path="/downloads/prune",
             json_data=payload.model_dump(),
             authenticated=True,

src/bsm_api_client/client/_users_methods.py

@@ -120,7 +120,7 @@ async def async_enable_user(self, user_id: int) -> BaseApiResponse:
         )
         return cast(BaseApiResponse, BaseApiResponse.model_validate(response))
 
-    async def async_generate_invite_token(self, role: str) -> Dict[str, Any]:
+    async def async_generate_invite_token(self, role: str) -> ActionResponse:
         """Generates an invite token.
 
         :param role: The role for the new user.

tests/test_api_client.py

@@ -40,15 +40,15 @@ async def test_get_themes(client):
     """Test get_themes method."""
     with patch.object(client, "_request", new_callable=AsyncMock) as mock_request:
         mock_request.return_value = {
-            "dark": "dark.css",
+            "status": "success",
+            "themes": ["dark"],
         }
         result = await client.async_get_themes()
         mock_request.assert_called_once_with(
-            method="GET", path="/themes", authenticated=True
+            method="GET", path="/info/themes", authenticated=True
         )
-        assert getattr(result, "dark", None) == "dark.css" or result == {
-            "dark": "dark.css"
-        }
+        assert result.status == "success"
+        assert result.themes == ["dark"]
 
 
 @pytest.mark.asyncio
@@ -61,7 +61,7 @@ async def test_prune_server_backups(client):
         }
         result = await client.async_prune_server_backups("test-server")
         mock_request.assert_called_once_with(
-            "POST",
+            "PUT",
             "/server/test-server/backups/prune",
             json_data=None,
             authenticated=True,
@@ -88,7 +88,7 @@ async def test_set_server_permissions(client):
         }
         result = await client.async_set_server_permissions("test-server", payload)
         mock_request.assert_called_once_with(
-            "PUT",
+            "POST",
             "/server/test-server/permissions/set",
             json_data=payload.model_dump(),
             authenticated=True,

tests/test_content_methods.py

@@ -266,7 +266,7 @@ async def test_uninstall_server_addon(client):
         mock_request.return_value = {"status": "success", "message": "Uninstalled"}
         result = await client.async_uninstall_server_addon("test-server", payload)
         mock_request.assert_called_once_with(
-            "POST",
+            "DELETE",
             "/server/test-server/addon/uninstall",
             json_data=payload.model_dump(),
             authenticated=True,
@@ -350,7 +350,7 @@ async def test_prune_server_backups(client):
         }
         result = await client.async_prune_server_backups("test-server")
         mock_request.assert_called_once_with(
-            "POST",
+            "PUT",
             "/server/test-server/backups/prune",
             json_data=None,
             authenticated=True,

tests/test_manager_methods.py

@@ -40,7 +40,7 @@ async def test_scan_players(client):
         mock_request.return_value = {"status": "success", "message": "Scan complete."}
         result = await client.async_scan_players()
         mock_request.assert_called_once_with(
-            method="POST", path="/players/scan", authenticated=True
+            method="PUT", path="/players/scan", authenticated=True
         )
         assert result.status == "success"
 
@@ -90,7 +90,7 @@ async def test_get_all_settings(client):
         }
         result = await client.async_get_all_settings()
         mock_request.assert_called_once_with(
-            method="GET", path="/settings", authenticated=True
+            method="GET", path="/settings/get", authenticated=True
         )
         assert result.settings["web"]["port"] == 8080
 
@@ -104,7 +104,7 @@ async def test_set_setting(client):
         result = await client.async_set_setting(payload)
         mock_request.assert_called_once_with(
             method="POST",
-            path="/settings",
+            path="/settings/set",
             json_data=payload.model_dump(),
             authenticated=True,
         )
@@ -121,7 +121,7 @@ async def test_reload_settings(client):
         }
         result = await client.async_reload_settings()
         mock_request.assert_called_once_with(
-            method="POST", path="/settings/reload", authenticated=True
+            method="PUT", path="/settings/reload", authenticated=True
         )
         assert result.status == "success"
 
@@ -137,7 +137,7 @@ async def test_prune_downloads(client):
         }
         result = await client.async_prune_downloads(payload)
         mock_request.assert_called_once_with(
-            method="POST",
+            method="PUT",
             path="/downloads/prune",
             json_data=payload.model_dump(),
             authenticated=True,