Skip to content

Commit 6f99fb2

Browse files
committed
Add type ignores to tests
1 parent c5e0600 commit 6f99fb2

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

pydantic_ai_slim/pydantic_ai/models/openrouter.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,7 @@ def _verify_response_is_not_error(response: ChatCompletion) -> ChatCompletion:
249249
else:
250250
choice = response.choices[0]
251251

252-
if choice.finish_reason == 'error':
252+
if choice.finish_reason == 'error': # type: ignore[reportUnnecessaryComparison]
253253
raise UnexpectedModelBehavior(
254254
'Invalid response from OpenRouter chat completions endpoint, error finish_reason without error data'
255255
)

tests/models/test_openrouter.py

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,16 @@
22

33
import pytest
44
from inline_snapshot import snapshot
5-
from openai.types.chat import ChatCompletion
6-
from openai.types.chat.chat_completion import Choice
75

86
from pydantic_ai import Agent, ModelHTTPError, ModelRequest, TextPart, ThinkingPart, UnexpectedModelBehavior
97
from pydantic_ai.direct import model_request
108

119
from ..conftest import try_import
1210

1311
with try_import() as imports_successful:
12+
from openai.types.chat import ChatCompletion
13+
from openai.types.chat.chat_completion import Choice
14+
1415
from pydantic_ai.models.openrouter import OpenRouterModel, OpenRouterModelSettings
1516
from pydantic_ai.providers.openrouter import OpenRouterProvider
1617

@@ -127,7 +128,7 @@ async def test_openrouter_validate_non_json_response(openrouter_api_key: str) ->
127128
model = OpenRouterModel('google/gemini-2.0-flash-exp:free', provider=provider)
128129

129130
with pytest.raises(UnexpectedModelBehavior) as exc_info:
130-
model._process_response('This is not JSON!')
131+
model._process_response('This is not JSON!') # type: ignore[reportPrivateUsage]
131132

132133
assert str(exc_info.value) == snapshot(
133134
'Invalid response from OpenRouter chat completions endpoint, expected JSON data'
@@ -139,10 +140,10 @@ async def test_openrouter_validate_error_response(openrouter_api_key: str) -> No
139140
model = OpenRouterModel('google/gemini-2.0-flash-exp:free', provider=provider)
140141

141142
response = ChatCompletion.model_construct(model='test')
142-
response.error = {'message': 'This response has an error attribute', 'code': 200}
143+
response.error = {'message': 'This response has an error attribute', 'code': 200} # type: ignore[reportAttributeAccessIssue]
143144

144145
with pytest.raises(ModelHTTPError) as exc_info:
145-
model._process_response(response)
146+
model._process_response(response) # type: ignore[reportPrivateUsage]
146147

147148
assert str(exc_info.value) == snapshot(
148149
'status_code: 200, model_name: test, body: This response has an error attribute'
@@ -157,7 +158,7 @@ async def test_openrouter_validate_error_finish_reason(openrouter_api_key: str)
157158
response = ChatCompletion.model_construct(choices=[choice])
158159

159160
with pytest.raises(UnexpectedModelBehavior) as exc_info:
160-
model._process_response(response)
161+
model._process_response(response) # type: ignore[reportPrivateUsage]
161162

162163
assert str(exc_info.value) == snapshot(
163164
'Invalid response from OpenRouter chat completions endpoint, error finish_reason without error data'

0 commit comments

Comments
 (0)