Skip to content

Commit 57ad594

Browse files
committed
Revert not wanted
1 parent abfe543 commit 57ad594

5 files changed

Lines changed: 18 additions & 21 deletions

File tree

tests/client/test_base_client.py

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from typing import cast
21
from unittest.mock import AsyncMock, MagicMock, patch
32

43
import pytest
@@ -72,11 +71,11 @@ async def test_transport_async_context_manager() -> None:
7271
patch.object(ClientTransport, '__abstractmethods__', set()),
7372
patch.object(ClientTransport, 'close', new_callable=AsyncMock),
7473
):
75-
transport = ClientTransport() # type: ignore[abstract]
74+
transport = ClientTransport()
7675
async with transport as t:
7776
assert t is transport
78-
cast(AsyncMock, transport.close).assert_not_awaited()
79-
cast(AsyncMock, transport.close).assert_awaited_once()
77+
transport.close.assert_not_awaited()
78+
transport.close.assert_awaited_once()
8079

8180

8281
@pytest.mark.asyncio
@@ -85,11 +84,11 @@ async def test_transport_async_context_manager_on_exception() -> None:
8584
patch.object(ClientTransport, '__abstractmethods__', set()),
8685
patch.object(ClientTransport, 'close', new_callable=AsyncMock),
8786
):
88-
transport = ClientTransport() # type: ignore[abstract]
87+
transport = ClientTransport()
8988
with pytest.raises(RuntimeError, match='boom'):
9089
async with transport:
9190
raise RuntimeError('boom')
92-
cast(AsyncMock, transport.close).assert_awaited_once()
91+
transport.close.assert_awaited_once()
9392

9493

9594
@pytest.mark.asyncio

tests/integration/test_client_server_integration.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1129,7 +1129,7 @@ async def test_json_transport_get_signed_extended_card(
11291129
private_key = ec.generate_private_key(ec.SECP256R1())
11301130
public_key = private_key.public_key()
11311131
signer = create_agent_card_signer(
1132-
signing_key=private_key, # type: ignore[arg-type]
1132+
signing_key=private_key,
11331133
protected_header={
11341134
'alg': 'ES256',
11351135
'kid': 'testkey',
@@ -1194,7 +1194,7 @@ async def test_json_transport_get_signed_base_and_extended_cards(
11941194
private_key = ec.generate_private_key(ec.SECP256R1())
11951195
public_key = private_key.public_key()
11961196
signer = create_agent_card_signer(
1197-
signing_key=private_key, # type: ignore[arg-type]
1197+
signing_key=private_key,
11981198
protected_header={
11991199
'alg': 'ES256',
12001200
'kid': 'testkey',
@@ -1271,7 +1271,7 @@ async def test_rest_transport_get_signed_card(
12711271
private_key = ec.generate_private_key(ec.SECP256R1())
12721272
public_key = private_key.public_key()
12731273
signer = create_agent_card_signer(
1274-
signing_key=private_key, # type: ignore[arg-type]
1274+
signing_key=private_key,
12751275
protected_header={
12761276
'alg': 'ES256',
12771277
'kid': 'testkey',
@@ -1340,7 +1340,7 @@ async def test_grpc_transport_get_signed_card(
13401340
private_key = ec.generate_private_key(ec.SECP256R1())
13411341
public_key = private_key.public_key()
13421342
signer = create_agent_card_signer(
1343-
signing_key=private_key, # type: ignore[arg-type]
1343+
signing_key=private_key,
13441344
protected_header={
13451345
'alg': 'ES256',
13461346
'kid': 'testkey',

tests/integration/test_end_to_end.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
from collections.abc import AsyncGenerator
2-
from typing import NamedTuple, cast
2+
from typing import NamedTuple
33

44
import grpc
55
import httpx
@@ -74,8 +74,6 @@ def assert_events_match(events, expected_events):
7474

7575
class MockAgentExecutor(AgentExecutor):
7676
async def execute(self, context: RequestContext, event_queue: EventQueue):
77-
assert context.task_id is not None
78-
assert context.context_id is not None
7977
task_updater = TaskUpdater(
8078
event_queue,
8179
context.task_id,
@@ -183,7 +181,7 @@ def rest_setup(agent_card, base_e2e_setup) -> ClientSetup:
183181
supported_protocol_bindings=[TransportProtocol.HTTP_JSON],
184182
)
185183
)
186-
client = cast(BaseClient, factory.create(agent_card))
184+
client = factory.create(agent_card)
187185
return ClientSetup(
188186
client=client,
189187
task_store=task_store,
@@ -206,7 +204,7 @@ def jsonrpc_setup(agent_card, base_e2e_setup) -> ClientSetup:
206204
supported_protocol_bindings=[TransportProtocol.JSONRPC],
207205
)
208206
)
209-
client = cast(BaseClient, factory.create(agent_card))
207+
client = factory.create(agent_card)
210208
return ClientSetup(
211209
client=client,
212210
task_store=task_store,
@@ -243,7 +241,7 @@ async def grpc_setup(
243241
supported_protocol_bindings=[TransportProtocol.GRPC],
244242
)
245243
)
246-
client = cast(BaseClient, factory.create(grpc_agent_card))
244+
client = factory.create(grpc_agent_card)
247245
yield ClientSetup(
248246
client=client,
249247
task_store=task_store,

tests/server/apps/jsonrpc/test_jsonrpc_app.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ def test_create_jsonrpc_based_app_with_present_deps_succeeds(
141141
class MockJSONRPCApp(JSONRPCApplication):
142142
def build( # type: ignore[override]
143143
self,
144-
agent_card_url='/custom.json',
144+
agent_card_url='/.well-known/agent.json',
145145
rpc_url='/',
146146
**kwargs,
147147
):
@@ -162,7 +162,7 @@ def test_create_jsonrpc_based_app_with_missing_deps_raises_importerror(
162162
class MockJSONRPCApp(JSONRPCApplication):
163163
def build( # type: ignore[override]
164164
self,
165-
agent_card_url='/custom.json',
165+
agent_card_url='/.well-known/agent.json',
166166
rpc_url='/',
167167
**kwargs,
168168
):

tests/utils/test_signing.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def test_signer_and_verifier_symmetric(sample_agent_card: AgentCard):
5959
wrong_key = 'wrongkey'
6060

6161
agent_card_signer = signing.create_agent_card_signer(
62-
signing_key=key, # type: ignore[arg-type]
62+
signing_key=key,
6363
protected_header={
6464
'alg': 'HS384',
6565
'kid': 'key1',
@@ -109,7 +109,7 @@ def test_signer_and_verifier_symmetric_multiple_signatures(
109109
wrong_key = 'wrongkey'
110110

111111
agent_card_signer = signing.create_agent_card_signer(
112-
signing_key=key, # type: ignore[arg-type]
112+
signing_key=key,
113113
protected_header={
114114
'alg': 'HS384',
115115
'kid': 'key1',
@@ -148,7 +148,7 @@ def test_signer_and_verifier_asymmetric(sample_agent_card: AgentCard):
148148
public_key_error = private_key_error.public_key()
149149

150150
agent_card_signer = signing.create_agent_card_signer(
151-
signing_key=private_key, # type: ignore[arg-type]
151+
signing_key=private_key,
152152
protected_header={
153153
'alg': 'ES256',
154154
'kid': 'key2',

0 commit comments

Comments
 (0)