Repository navigation

test: update SSE tests to use StreamingASGITransport instead of uvicorn for server testing - #1566

Closed

bjoaquinc wants to merge 24 commits into

modelcontextprotocol:mainfrom

bjoaquinc:streaming-asgi-sse

Commits on Nov 2, 2025
- clean all remaining stream ends on close
  bjoaquinc
  committed
  38f307c
- add disconnect_event to handle closure from client/task_group
  bjoaquinc
  committed
  7f241e7
- create context_app using StreamingASGITransport and update test_request_context_propogation to apply this methodology
  bjoaquinc
  committed
  7250289
- add back await content_send_channel.aclose()
  bjoaquinc
  committed
  8a8e14f
- revert spaces
  bjoaquinc
  committed
  7a83024
Commits on Nov 3, 2025
- remove try finally block for testing and just add sse_stream_reader.aclose()
  bjoaquinc
  committed
  c880cc1
- fix ruff errors
  bjoaquinc
  committed
  0aa5a45
- run precommit
  bjoaquinc
  committed
  b489d30
Commits on Nov 6, 2025
- update all sse tests that use uvicorn to use StreamingASGITransport instead. DRY implementation
  bjoaquinc
  committed
  bd70a51
- run precommit
  bjoaquinc
  committed
  5cc10fa
- remove taskgroup fixture due to premature closing
  bjoaquinc
  committed
  e1745f8
- prevent sse_client from cancelling external task groups
  bjoaquinc
  committed
  8fc6473
- run precommit
  bjoaquinc
  committed
  fabf6c5
- revert sse_client and add cleanup to outer task group in tests
  bjoaquinc
  committed
  e8a3b0e
- remove timeout for sse_client
  bjoaquinc
  committed
  8188bf2
- add reset_sse_app_status workaround for sse_starlette quirk
  bjoaquinc
  committed
  86a377f
- improve workaround
  bjoaquinc
  committed
  f1748a0
Commits on Nov 7, 2025
- move workaround to conftest for other test files
  bjoaquinc
  committed
  749d506
Commits on Nov 8, 2025
- add version check into reset_sse_app_status
  bjoaquinc
  committed
  044728b
- revert duplicate http.response.start handling
  bjoaquinc
  committed
  2ec8907
- add NoopASGI
  bjoaquinc
  committed
  b5b4e6f
- add NoopASGI and clean tests
  bjoaquinc
  committed
  6857129
- remove uvicorn and DRY
  bjoaquinc
  committed
  3a9e576
- add short comment explaining NoopASGI usage
  bjoaquinc
  committed
  3d2fea9

Navigation Menu

Repository navigation

test: update SSE tests to use StreamingASGITransport instead of uvicorn for server testing - #1566

test: update SSE tests to use StreamingASGITransport instead of uvicorn for server testing#1566
bjoaquinc wants to merge 24 commits into
modelcontextprotocol:mainfrom
bjoaquinc:streaming-asgi-sse

Commits on Nov 2, 2025

clean all remaining stream ends on close

add disconnect_event to handle closure from client/task_group

create context_app using StreamingASGITransport and update test_request_context_propogation to apply this methodology

add back await content_send_channel.aclose()

revert spaces

Commits on Nov 3, 2025

remove try finally block for testing and just add sse_stream_reader.aclose()

fix ruff errors

run precommit

Commits on Nov 6, 2025

update all sse tests that use uvicorn to use StreamingASGITransport instead. DRY implementation

run precommit

remove taskgroup fixture due to premature closing

prevent sse_client from cancelling external task groups

run precommit

revert sse_client and add cleanup to outer task group in tests

remove timeout for sse_client

add reset_sse_app_status workaround for sse_starlette quirk

improve workaround

Commits on Nov 7, 2025

move workaround to conftest for other test files

Commits on Nov 8, 2025

add version check into reset_sse_app_status

revert duplicate http.response.start handling

add NoopASGI

add NoopASGI and clean tests

remove uvicorn and DRY

add short comment explaining NoopASGI usage