MAINT: Silence deprecation warnings in unit tests (microsoft#2108)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: romanlutz

tests/unit/backend/test_api_routes.py

@@ -648,7 +648,7 @@ def test_list_attacks_rejects_invalid_converter_types_match(self, client: TestCl
 
             response = client.get("/api/attacks?converter_types_match=garbage")
 
-            assert response.status_code == status.HTTP_422_UNPROCESSABLE_ENTITY
+            assert response.status_code == status.HTTP_422_UNPROCESSABLE_CONTENT
 
     def test_get_conversations_success(self, client: TestClient) -> None:
         """Test getting attack conversations returns service response."""

tests/unit/backend/test_initializer_service.py

@@ -392,7 +392,7 @@ def test_post_returns_422_for_invalid_name(
         response = client_with_custom_initializers_enabled.post(
             "/api/initializers", json={"name": bad_name, "script_content": _SAMPLE_SCRIPT}
         )
-        assert response.status_code == status.HTTP_422_UNPROCESSABLE_ENTITY
+        assert response.status_code == status.HTTP_422_UNPROCESSABLE_CONTENT
 
     def test_post_returns_201_with_registered_initializer(
         self, client_with_custom_initializers_enabled: TestClient

tests/unit/backend/test_scenario_run_routes.py

@@ -90,7 +90,7 @@ def test_start_run_invalid_scenario_returns_400(self, client: TestClient) -> Non
     def test_start_run_missing_required_fields_returns_422(self, client: TestClient) -> None:
         """Test that missing required fields returns 422."""
         response = client.post("/api/scenarios/runs", json={})
-        assert response.status_code == status.HTTP_422_UNPROCESSABLE_ENTITY
+        assert response.status_code == status.HTTP_422_UNPROCESSABLE_CONTENT
 
     def test_start_run_with_all_options(self, client: TestClient) -> None:
         """Test that all optional fields are accepted."""

tests/unit/prompt_target/target/test_openai_chat_target.py

@@ -711,8 +711,9 @@ def mock_token_provider():
     assert callable(target._api_key)
     # Since sync provider is wrapped, _api_key is now async
     import asyncio
+    import inspect
 
-    assert asyncio.iscoroutinefunction(target._api_key)
+    assert inspect.iscoroutinefunction(target._api_key)
     assert asyncio.run(target._api_key()) == "mock-entra-token"
 
 

tests/unit/prompt_target/target/test_openai_target_auth.py

@@ -2,6 +2,7 @@
 # Licensed under the MIT license.
 
 import asyncio
+import inspect
 import os
 from collections.abc import Callable
 from unittest.mock import AsyncMock, MagicMock, patch
@@ -101,7 +102,7 @@ def sync_provider() -> str:
             return "sync-token"
 
         target = _build_target(api_key=sync_provider)
-        assert asyncio.iscoroutinefunction(target._api_key)
+        assert inspect.iscoroutinefunction(target._api_key)
         # Verify the wrapper actually calls through
         token = asyncio.run(target._api_key())
         assert token == "sync-token"
@@ -142,7 +143,7 @@ def provider() -> str:
             return "sync-token"
 
         result = ensure_async_token_provider(provider)
-        assert asyncio.iscoroutinefunction(result)
+        assert inspect.iscoroutinefunction(result)
         assert asyncio.run(result()) == "sync-token"
 
     def test_non_callable_non_string_returned_as_is(self):

tests/unit/prompt_target/target/test_token_provider_wrapping.py

@@ -1,7 +1,7 @@
 # Copyright (c) Microsoft Corporation.
 # Licensed under the MIT license.
 
-import asyncio
+import inspect
 from unittest.mock import AsyncMock, patch
 
 import pytest
@@ -32,7 +32,7 @@ async def async_token_provider():
 
         result = ensure_async_token_provider(async_token_provider)
         assert result is async_token_provider
-        assert asyncio.iscoroutinefunction(result)
+        assert inspect.iscoroutinefunction(result)
 
     def test_sync_token_provider_wrapped(self):
         """Test that synchronous token providers are automatically wrapped."""
@@ -45,7 +45,7 @@ def sync_token_provider():
         # Should return a different callable (the wrapper)
         assert result is not sync_token_provider
         assert callable(result)
-        assert asyncio.iscoroutinefunction(result)
+        assert inspect.iscoroutinefunction(result)
 
     async def test_wrapped_sync_provider_returns_correct_token(self):
         """Test that wrapped synchronous token provider returns the correct token."""
@@ -147,7 +147,7 @@ def sync_token_provider():
             # The api_key should be a callable
             api_key_arg = call_kwargs["api_key"]
             assert callable(api_key_arg)
-            assert asyncio.iscoroutinefunction(api_key_arg)
+            assert inspect.iscoroutinefunction(api_key_arg)
 
             # Verify the wrapped token provider returns correct value
             token = await api_key_arg()
@@ -234,7 +234,7 @@ def mock_sync_bearer_token_provider():
             call_kwargs = mock_openai.call_args[1]
             wrapped_provider = call_kwargs["api_key"]
 
-            assert asyncio.iscoroutinefunction(wrapped_provider)
+            assert inspect.iscoroutinefunction(wrapped_provider)
 
             # Verify it returns the correct token
             token = await wrapped_provider()