Unverified Commit 87ac404e authored by Graham King's avatar Graham King Committed by GitHub
Browse files

test: Fix cancellation test (#7296)


Signed-off-by: default avatarGraham King <grahamk@nvidia.com>
parent bb43fada
...@@ -7,7 +7,11 @@ import pytest ...@@ -7,7 +7,11 @@ import pytest
from dynamo.runtime import Context from dynamo.runtime import Context
pytestmark = pytest.mark.pre_merge pytestmark = [
pytest.mark.gpu_0,
pytest.mark.pre_merge,
pytest.mark.unit,
]
class MockServer: class MockServer:
...@@ -247,7 +251,9 @@ async def test_server_context_cancel(temp_file_store, server, client): ...@@ -247,7 +251,9 @@ async def test_server_context_cancel(temp_file_store, server, client):
except ValueError as e: except ValueError as e:
# Verify the expected cancellation exception is received # Verify the expected cancellation exception is received
# TODO: Should this be a asyncio.CancelledError? # TODO: Should this be a asyncio.CancelledError?
assert str(e).startswith("Stream ended before generation completed") assert str(e).startswith(
"Disconnected: Stream ended before generation completed"
)
# Verify server context cancellation status # Verify server context cancellation status
assert handler.context_is_stopped assert handler.context_is_stopped
...@@ -272,9 +278,8 @@ async def test_server_raise_cancelled(temp_file_store, server, client): ...@@ -272,9 +278,8 @@ async def test_server_raise_cancelled(temp_file_store, server, client):
except ValueError as e: except ValueError as e:
# Verify the expected cancellation exception is received # Verify the expected cancellation exception is received
# TODO: Should this be a asyncio.CancelledError? # TODO: Should this be a asyncio.CancelledError?
assert ( assert str(e).endswith(
str(e) "a python exception was caught while processing the async generator: CancelledError: "
== "a python exception was caught while processing the async generator: CancelledError: "
) )
# Verify server context cancellation status # Verify server context cancellation status
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment