From 3c668e4a5c0d77b1187c3087e80014ffe70fb937 Mon Sep 17 00:00:00 2001 From: -LAN- Date: Thu, 11 Sep 2025 16:41:10 +0800 Subject: [PATCH] fix: update test assertions for ToolProviderApiEntity validation - Fixed test_repack_provider_entity_no_dark_icon to use empty string instead of None for icon_dark field - Updated test_builtin_provider_to_user_provider_no_credentials assertion to match actual implementation behavior where masked_credentials always contains empty strings for schema fields --- .../services/tools/test_tools_transform_service.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/api/tests/test_containers_integration_tests/services/tools/test_tools_transform_service.py b/api/tests/test_containers_integration_tests/services/tools/test_tools_transform_service.py index bf25968100..827f9c010e 100644 --- a/api/tests/test_containers_integration_tests/services/tools/test_tools_transform_service.py +++ b/api/tests/test_containers_integration_tests/services/tools/test_tools_transform_service.py @@ -454,7 +454,7 @@ class TestToolTransformService: name=fake.company(), description=I18nObject(en_US=fake.text(max_nb_chars=100)), icon='{"background": "#FF6B6B", "content": "🔧"}', - icon_dark=None, + icon_dark="", label=I18nObject(en_US=fake.company()), type=ToolProviderType.API, masked_credentials={}, @@ -473,8 +473,8 @@ class TestToolTransformService: assert provider.icon["background"] == "#FF6B6B" assert provider.icon["content"] == "🔧" - # Verify dark icon remains None - assert provider.icon_dark is None + # Verify dark icon remains empty string + assert provider.icon_dark == "" def test_builtin_provider_to_user_provider_success( self, db_session_with_containers, mock_external_service_dependencies @@ -628,7 +628,7 @@ class TestToolTransformService: assert result is not None assert result.is_team_authorization is True assert result.allow_delete is False - assert result.masked_credentials == {} + assert result.masked_credentials == {"api_key": ""} def test_api_provider_to_controller_success(self, db_session_with_containers, mock_external_service_dependencies): """