Skip to content

Commit b70d99d

Browse files
CopilotJesuTerraz
andauthored
Remove unnecessary Agent class patches from googleadk extension tests (#169)
* Initial plan * Remove unnecessary Agent patches from googleadk tests Co-authored-by: JesuTerraz <96103167+JesuTerraz@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: JesuTerraz <96103167+JesuTerraz@users.noreply.github.com>
1 parent dbd0358 commit b70d99d

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

tests/tooling/extensions/googleadk/test_mcp_tool_registration_service.py

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -116,9 +116,6 @@ async def test_add_tool_servers_exchanges_token_when_not_provided(
116116
patch(
117117
"microsoft_agents_a365.tooling.extensions.googleadk.services.mcp_tool_registration_service.get_mcp_platform_authentication_scope"
118118
) as mock_get_scope,
119-
patch(
120-
"microsoft_agents_a365.tooling.extensions.googleadk.services.mcp_tool_registration_service.Agent"
121-
) as mock_agent_class,
122119
):
123120
# Setup mocks
124121
mock_get_scope.return_value = ["https://test.scope/.default"]
@@ -129,8 +126,6 @@ async def test_add_tool_servers_exchanges_token_when_not_provided(
129126
mock_config_service.list_tool_servers = AsyncMock(return_value=[])
130127
mock_config_service_class.return_value = mock_config_service
131128

132-
mock_agent_class.return_value = mock_agent
133-
134129
from microsoft_agents_a365.tooling.extensions.googleadk import (
135130
McpToolRegistrationService,
136131
)
@@ -161,9 +156,6 @@ async def test_add_tool_servers_uses_provided_token(
161156
patch(
162157
"microsoft_agents_a365.tooling.extensions.googleadk.services.mcp_tool_registration_service.Utility"
163158
) as mock_utility,
164-
patch(
165-
"microsoft_agents_a365.tooling.extensions.googleadk.services.mcp_tool_registration_service.Agent"
166-
) as mock_agent_class,
167159
):
168160
# Setup mocks
169161
mock_utility.resolve_agent_identity.return_value = "agent-123"
@@ -173,8 +165,6 @@ async def test_add_tool_servers_uses_provided_token(
173165
mock_config_service.list_tool_servers = AsyncMock(return_value=[])
174166
mock_config_service_class.return_value = mock_config_service
175167

176-
mock_agent_class.return_value = mock_agent
177-
178168
from microsoft_agents_a365.tooling.extensions.googleadk import (
179169
McpToolRegistrationService,
180170
)

0 commit comments

Comments
 (0)