Skip to content

Conversation

@didier-durand
Copy link
Contributor

Description

Hi,

adding 12 tests for InMemoryQueueManager. They all pass: see execution report below

Didier

test execution:

============================= test session starts ==============================
collecting ... collected 12 items

server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_init PASSED [  8%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_add_new_queue PASSED [ 16%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_add_existing_queue PASSED [ 25%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_get_existing_queue PASSED [ 33%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_get_nonexistent_queue PASSED [ 41%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_tap_existing_queue PASSED [ 50%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_tap_nonexistent_queue PASSED [ 58%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_close_existing_queue PASSED [ 66%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_close_nonexistent_queue PASSED [ 75%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_create_or_tap_new_queue PASSED [ 83%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_create_or_tap_existing_queue PASSED [ 91%]
server/events/test_inmemory_queue_manager.py::TestInMemoryQueueManager::test_concurrency PASSED [100%]

@didier-durand didier-durand requested a review from a team as a code owner May 20, 2025 07:05
@didier-durand didier-durand changed the title adding 12 tests for InMemoryQueueManager test: adding 12 tests for InMemoryQueueManager May 20, 2025
@holtskinner holtskinner enabled auto-merge (squash) May 20, 2025 14:11
@holtskinner holtskinner merged commit 16195d2 into a2aproject:main May 20, 2025
5 checks passed
martimfasantos pushed a commit to martimfasantos/a2a-python that referenced this pull request May 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants