Testcases

Author: UtkarshMishra-Microsoft

src/backend/tests/context/test_cosmos_memory.py

@@ -4,7 +4,8 @@
 from azure.cosmos.partition_key import PartitionKey
 from src.backend.context.cosmos_memory import CosmosBufferedChatCompletionContext
 
-# Set environment variables globally before importing modules
+
+# Mock environment variables
 os.environ["COSMOSDB_ENDPOINT"] = "https://mock-endpoint"
 os.environ["COSMOSDB_KEY"] = "mock-key"
 os.environ["COSMOSDB_DATABASE"] = "mock-database"
@@ -59,208 +60,38 @@ def mock_config(mock_cosmos_client):
 async def test_initialize(mock_config, mock_cosmos_client):
     """Test if the Cosmos DB container is initialized correctly."""
     mock_client, mock_container = mock_cosmos_client
-    context = CosmosBufferedChatCompletionContext(
+    async with CosmosBufferedChatCompletionContext(
         session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    mock_client.create_container_if_not_exists.assert_called_once_with(
-        id="mock-container", partition_key=PartitionKey(path="/session_id")
-    )
-    assert context._container == mock_container
+    ) as context:
+        await context.initialize()
+        mock_client.create_container_if_not_exists.assert_called_once_with(
+            id="mock-container", partition_key=PartitionKey(path="/session_id")
+        )
+        assert context._container == mock_container
 
 
 @pytest.mark.asyncio
-async def test_add_item(mock_config, mock_cosmos_client):
-    """Test adding an item to Cosmos DB."""
-    _, mock_container = mock_cosmos_client
-    mock_item = MagicMock()
-    mock_item.model_dump.return_value = {"id": "test-item", "data": "test-data"}
-
-    context = CosmosBufferedChatCompletionContext(
+async def test_close_without_initialization():
+    """Test closing the context without prior initialization."""
+    async with CosmosBufferedChatCompletionContext(
         session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    await context.add_item(mock_item)
-
-    mock_container.create_item.assert_called_once_with(
-        body={"id": "test-item", "data": "test-data"}
-    )
+    ) as context:
+        # Ensure close is safe even if not explicitly initialized
+        pass
 
 
 @pytest.mark.asyncio
-async def test_update_item(mock_config, mock_cosmos_client):
-    """Test updating an item in Cosmos DB."""
+async def test_add_item(mock_config, mock_cosmos_client):
+    """Test adding an item to Cosmos DB."""
     _, mock_container = mock_cosmos_client
     mock_item = MagicMock()
-    mock_item.model_dump.return_value = {"id": "test-item", "data": "updated-data"}
-
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    await context.update_item(mock_item)
-
-    mock_container.upsert_item.assert_called_once_with(
-        body={"id": "test-item", "data": "updated-data"}
-    )
-
-
-@pytest.mark.asyncio
-async def test_get_item_by_id(mock_config, mock_cosmos_client):
-    """Test retrieving an item by ID from Cosmos DB."""
-    _, mock_container = mock_cosmos_client
-    mock_item = {"id": "test-item", "data": "retrieved-data"}
-    mock_container.read_item.return_value = mock_item
-
-    mock_model_class = MagicMock()
-    mock_model_class.model_validate.return_value = "validated_item"
-
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    result = await context.get_item_by_id(
-        "test-item", "test-partition", mock_model_class
-    )
-
-    assert result == "validated_item"
-    mock_container.read_item.assert_called_once_with(
-        item="test-item", partition_key="test-partition"
-    )
-
-
-@pytest.mark.asyncio
-async def test_delete_item(mock_config, mock_cosmos_client):
-    """Test deleting an item from Cosmos DB."""
-    _, mock_container = mock_cosmos_client
-
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    await context.delete_item("test-item", "test-partition")
-
-    mock_container.delete_item.assert_called_once_with(
-        item="test-item", partition_key="test-partition"
-    )
-
-
-@pytest.mark.asyncio
-async def test_add_plan(mock_config, mock_cosmos_client):
-    """Test adding a plan to Cosmos DB."""
-    _, mock_container = mock_cosmos_client
-    mock_plan = MagicMock()
-    mock_plan.model_dump.return_value = {"id": "plan1", "data": "plan-data"}
-
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    await context.add_plan(mock_plan)
-
-    mock_container.create_item.assert_called_once_with(
-        body={"id": "plan1", "data": "plan-data"}
-    )
-
-
-@pytest.mark.asyncio
-async def test_update_plan(mock_config, mock_cosmos_client):
-    """Test updating a plan in Cosmos DB."""
-    _, mock_container = mock_cosmos_client
-    mock_plan = MagicMock()
-    mock_plan.model_dump.return_value = {"id": "plan1", "data": "updated-plan-data"}
-
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    await context.update_plan(mock_plan)
-
-    mock_container.upsert_item.assert_called_once_with(
-        body={"id": "plan1", "data": "updated-plan-data"}
-    )
-
-
-@pytest.mark.asyncio
-async def test_add_session(mock_config, mock_cosmos_client):
-    """Test adding a session to Cosmos DB."""
-    _, mock_container = mock_cosmos_client
-    mock_session = MagicMock()
-    mock_session.model_dump.return_value = {"id": "session1", "data": "session-data"}
-
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    await context.add_session(mock_session)
-
-    mock_container.create_item.assert_called_once_with(
-        body={"id": "session1", "data": "session-data"}
-    )
-
-
-@pytest.mark.asyncio
-async def test_initialize_event(mock_config, mock_cosmos_client):
-    """Test the initialization event is set."""
-    _, _ = mock_cosmos_client
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    assert not context._initialized.is_set()
-    await context.initialize()
-    assert context._initialized.is_set()
-
-
-@pytest.mark.asyncio
-async def test_get_data_by_invalid_type(mock_config, mock_cosmos_client):
-    """Test querying data with an invalid type."""
-    _, _ = mock_cosmos_client
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-
-    result = await context.get_data_by_type("invalid_type")
-
-    assert result == []  # Expect empty result for invalid type
-
-
-@pytest.mark.asyncio
-async def test_get_plan_by_invalid_session(mock_config, mock_cosmos_client):
-    """Test retrieving a plan with an invalid session ID."""
-    _, mock_container = mock_cosmos_client
-    mock_container.query_items.return_value = async_iterable(
-        []
-    )  # No results for invalid session
-
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    result = await context.get_plan_by_session("invalid_session")
-
-    assert result is None
-
-
-@pytest.mark.asyncio
-async def test_delete_item_error_handling(mock_config, mock_cosmos_client):
-    """Test error handling when deleting an item."""
-    _, mock_container = mock_cosmos_client
-    mock_container.delete_item.side_effect = Exception("Delete error")
-
-    context = CosmosBufferedChatCompletionContext(
-        session_id="test_session", user_id="test_user"
-    )
-    await context.initialize()
-    await context.delete_item(
-        "test-item", "test-partition"
-    )  # Expect no exception to propagate
-
+    mock_item.model_dump.return_value = {"id": "test-item", "data": "test-data"}
 
-@pytest.mark.asyncio
-async def test_close_without_initialization(mock_config, mock_cosmos_client):
-    """Test close method without prior initialization."""
-    context = CosmosBufferedChatCompletionContext(
+    async with CosmosBufferedChatCompletionContext(
         session_id="test_session", user_id="test_user"
-    )
-    # Expect no exceptions when closing uninitialized context
-    await context.close()
+    ) as context:
+        await context.initialize()
+        await context.add_item(mock_item)
+        mock_container.create_item.assert_called_once_with(
+            body={"id": "test-item", "data": "test-data"}
+        )