Remove unnecessary Agent class patches from googleadk extension tests

[Copilot]

Two test methods were patching the Agent class despite the implementation never instantiating or directly referencing it—only using the passed-in agent parameter.

Changes

• Removed Agent patch from test_add_tool_servers_exchanges_token_when_not_provided
• Removed Agent patch from test_add_tool_servers_uses_provided_token

The Agent class is imported for type hints but not used as a symbol in the implementation:

# Implementation only uses the agent parameter, never Agent symbol
async def add_tool_servers_to_agent(
    self,
    agent: Agent,  # Type hint only
    auth: Authorization,
    ...
) -> None:
    # Uses agent.tools, never instantiates Agent
    agent.tools = list(agent.tools) + mcp_servers_info

This reduces unnecessary mocking surface and clarifies actual test dependencies.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• astral.sh
  • Triggering command: /usr/bin/curl curl -LsSf REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)