Skip to content

Commit 9dd8080

Browse files
committed
test: remove stale no-cover pragmas
1 parent 7f5957d commit 9dd8080

5 files changed

Lines changed: 10 additions & 10 deletions

File tree

src/mcp/server/streamable_http.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,7 @@ def _create_session_message(
245245
async def close_stream_callback() -> None:
246246
self.close_sse_stream(request_id)
247247

248-
async def close_standalone_stream_callback() -> None: # pragma: no cover
248+
async def close_standalone_stream_callback() -> None:
249249
self.close_standalone_sse_stream()
250250

251251
metadata = ServerMessageMetadata(
@@ -421,7 +421,7 @@ async def _validate_accept_header(self, request: Request, scope: Scope, send: Se
421421
has_json, has_sse = self._check_accept_headers(request)
422422
if self.is_json_response_enabled:
423423
# For JSON-only responses, only require application/json
424-
if not has_json: # pragma: no cover
424+
if not has_json:
425425
response = self._create_error_response(
426426
"Not Acceptable: Client must accept application/json",
427427
HTTPStatus.NOT_ACCEPTABLE,
@@ -669,7 +669,7 @@ async def _handle_get_request(self, request: Request, send: Send) -> None:
669669
await response(request.scope, request.receive, send)
670670
return
671671

672-
if not await self._validate_request_headers(request, send): # pragma: no cover
672+
if not await self._validate_request_headers(request, send):
673673
return
674674

675675
# Handle resumability: check for Last-Event-ID header
@@ -1017,7 +1017,7 @@ async def message_router():
10171017
try:
10181018
# Send both the message and the event ID
10191019
await self._request_streams[request_stream_id][0].send(EventMessage(message, event_id))
1020-
except (anyio.BrokenResourceError, anyio.ClosedResourceError): # pragma: no cover
1020+
except (anyio.BrokenResourceError, anyio.ClosedResourceError):
10211021
# Stream might be closed, remove from registry
10221022
self._request_streams.pop(request_stream_id, None)
10231023
else:

tests/server/test_sse_security.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ def reset_sse_starlette_exit_event() -> None:
3838
app_status.should_exit_event = None
3939

4040

41-
class SecurityTestServer(Server): # pragma: no cover
41+
class SecurityTestServer(Server):
4242
def __init__(self):
4343
super().__init__(SERVER_NAME)
4444

tests/server/test_streamable_http_security.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
SERVER_NAME = "test_streamable_http_security_server"
1919

2020

21-
class SecurityTestServer(Server): # pragma: no cover
21+
class SecurityTestServer(Server):
2222
def __init__(self):
2323
super().__init__(SERVER_NAME)
2424

tests/shared/test_sse.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ async def _handle_call_tool( # pragma: no cover
7878
return CallToolResult(content=[TextContent(type="text", text=f"Called {params.name}")])
7979

8080

81-
def _create_server() -> Server: # pragma: no cover
81+
def _create_server() -> Server:
8282
return Server(
8383
SERVER_NAME,
8484
on_read_resource=_handle_read_resource,
@@ -266,7 +266,7 @@ async def test_sse_client_timeout( # pragma: no cover
266266
pytest.fail("the client should have timed out and returned an error already")
267267

268268

269-
def make_mounted_server_app() -> Starlette: # pragma: no cover
269+
def make_mounted_server_app() -> Starlette:
270270
app = make_server_app()
271271
main_app = Starlette(routes=[Mount("/mounted_app", app=app)])
272272
return main_app

tests/shared/test_streamable_http.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ class ServerState:
142142

143143

144144
@asynccontextmanager
145-
async def _server_lifespan(_server: Server[ServerState]) -> AsyncIterator[ServerState]: # pragma: no cover
145+
async def _server_lifespan(_server: Server[ServerState]) -> AsyncIterator[ServerState]:
146146
yield ServerState()
147147

148148

@@ -380,7 +380,7 @@ async def _handle_call_tool( # pragma: no cover
380380
return CallToolResult(content=[TextContent(type="text", text=f"Called {name}")])
381381

382382

383-
def _create_server() -> Server[ServerState]: # pragma: no cover
383+
def _create_server() -> Server[ServerState]:
384384
return Server(
385385
SERVER_NAME,
386386
lifespan=_server_lifespan,

0 commit comments

Comments
 (0)