TEST: cover deprecated shim methods to satisfy diff-cover threshold

The unit-test-diff-cover check failed because six newly-added

print_deprecation_message(...) lines were not exercised by existing tests:

- pyrit/output/scenario_result/base.py:38 (print_summary_async)

- pyrit/output/scorer/base.py:68 (print_objective_scorer), :79 (print_harm_scorer)

- pyrit/prompt_converter/persuasion_converter.py:132 (send_persuasion_prompt_async)

- pyrit/prompt_converter/variation_converter.py:116 (send_variation_prompt_async)

- pyrit/prompt_target/common/prompt_chat_target.py:53 (__init__)

Add focused tests that invoke each shim, assert a DeprecationWarning is

emitted, and verify the shim delegates to the replacement API.

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

Author: romanlutz

tests/unit/output/scenario_result/test_base.py

@@ -0,0 +1,33 @@
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT license.
+
+from unittest.mock import AsyncMock, MagicMock
+
+import pytest
+
+from pyrit.models.scenario_result import ScenarioResult
+from pyrit.output.scenario_result.base import ScenarioResultPrinterBase
+
+
+def test_scenario_result_printer_cannot_be_instantiated():
+    with pytest.raises(TypeError, match="Can't instantiate abstract class"):
+        ScenarioResultPrinterBase()  # type: ignore[abstract]
+
+
+async def test_print_summary_async_emits_deprecation_warning_and_delegates():
+    """``print_summary_async`` is a deprecated shim that should warn and call ``write_async``."""
+
+    class _MinimalPrinter(ScenarioResultPrinterBase):
+        def __init__(self) -> None:
+            self.write_async = AsyncMock()
+
+        async def render_async(self, result: ScenarioResult) -> str:
+            return ""
+
+    printer = _MinimalPrinter()
+    result = MagicMock(spec=ScenarioResult)
+
+    with pytest.warns(DeprecationWarning, match="print_summary_async"):
+        await printer.print_summary_async(result)
+
+    printer.write_async.assert_awaited_once_with(result)

tests/unit/output/scorer/test_base.py

@@ -1,6 +1,8 @@
 # Copyright (c) Microsoft Corporation.
 # Licensed under the MIT license.
 
+from unittest.mock import AsyncMock
+
 import pytest
 
 from pyrit.identifiers import ComponentIdentifier
@@ -57,3 +59,44 @@ async def render_async(
 
     printer = CompletePrinter()
     assert isinstance(printer, ScorerPrinterBase)
+
+
+def _make_complete_printer() -> ScorerPrinterBase:
+    class CompletePrinter(ScorerPrinterBase):
+        def __init__(self) -> None:
+            self.write_async = AsyncMock()
+
+        def _get_objective_metrics(self, *, scorer_identifier: ComponentIdentifier):
+            return None
+
+        def _get_harm_metrics(self, *, scorer_identifier: ComponentIdentifier, harm_category: str):
+            return None
+
+        async def render_async(
+            self, *, scorer_identifier: ComponentIdentifier, harm_category: str | None = None
+        ) -> str:
+            return ""
+
+    return CompletePrinter()
+
+
+async def test_print_objective_scorer_emits_deprecation_warning_and_delegates():
+    """``print_objective_scorer`` is a deprecated shim that should warn and call ``write_async``."""
+    printer = _make_complete_printer()
+    scorer_identifier = ComponentIdentifier(class_name="TestScorer", class_module="tests")
+
+    with pytest.warns(DeprecationWarning, match="print_objective_scorer"):
+        await printer.print_objective_scorer(scorer_identifier=scorer_identifier)
+
+    printer.write_async.assert_awaited_once_with(scorer_identifier=scorer_identifier)
+
+
+async def test_print_harm_scorer_emits_deprecation_warning_and_delegates():
+    """``print_harm_scorer`` is a deprecated shim that should warn and call ``write_async``."""
+    printer = _make_complete_printer()
+    scorer_identifier = ComponentIdentifier(class_name="TestScorer", class_module="tests")
+
+    with pytest.warns(DeprecationWarning, match="print_harm_scorer"):
+        await printer.print_harm_scorer(scorer_identifier=scorer_identifier, harm_category="violence")
+
+    printer.write_async.assert_awaited_once_with(scorer_identifier=scorer_identifier, harm_category="violence")

tests/unit/prompt_converter/test_persuasion_converter.py

@@ -149,3 +149,32 @@ def test_persuasion_converter_identifier_includes_technique(sqlite_instance):
     prompt_persuasion = PersuasionConverter(converter_target=prompt_target, persuasion_technique="logical_appeal")
     identifier = prompt_persuasion.get_identifier()
     assert identifier.params["persuasion_technique"] == "logical_appeal"
+
+
+async def test_send_persuasion_prompt_async_emits_deprecation_warning_and_delegates(sqlite_instance):
+    """``send_persuasion_prompt_async`` is a deprecated shim that warns and delegates to the retry helper."""
+    prompt_target = MockPromptTarget()
+    prompt_persuasion = PersuasionConverter(
+        converter_target=prompt_target, persuasion_technique="authority_endorsement"
+    )
+
+    request = Message(
+        message_pieces=[
+            MessagePiece(
+                role="user",
+                conversation_id="conv-1",
+                original_value="test input",
+                original_value_data_type="text",
+                prompt_target_identifier=ComponentIdentifier(class_name="test", class_module="test"),
+            )
+        ]
+    )
+
+    with patch.object(
+        prompt_persuasion, "_send_with_retries_async", new=AsyncMock(return_value="shim response")
+    ) as mock_send:
+        with pytest.warns(DeprecationWarning, match="send_persuasion_prompt_async"):
+            result = await prompt_persuasion.send_persuasion_prompt_async(request)
+
+    assert result == "shim response"
+    mock_send.assert_awaited_once_with(request)

tests/unit/prompt_converter/test_variation_converter.py

@@ -115,3 +115,30 @@ def test_variation_converter_input_supported(sqlite_instance):
     converter = VariationConverter(converter_target=prompt_target)
     assert converter.input_supported("audio_path") is False
     assert converter.input_supported("text") is True
+
+
+async def test_send_variation_prompt_async_emits_deprecation_warning_and_delegates(sqlite_instance):
+    """``send_variation_prompt_async`` is a deprecated shim that warns and delegates to the retry helper."""
+    prompt_target = MockPromptTarget()
+    prompt_variation = VariationConverter(converter_target=prompt_target)
+
+    request = Message(
+        message_pieces=[
+            MessagePiece(
+                role="user",
+                conversation_id="conv-1",
+                original_value="test input",
+                original_value_data_type="text",
+                prompt_target_identifier=ComponentIdentifier(class_name="test", class_module="test"),
+            )
+        ]
+    )
+
+    with patch.object(
+        prompt_variation, "_send_with_retries_async", new=AsyncMock(return_value="shim response")
+    ) as mock_send:
+        with pytest.warns(DeprecationWarning, match="send_variation_prompt_async"):
+            result = await prompt_variation.send_variation_prompt_async(request)
+
+    assert result == "shim response"
+    mock_send.assert_awaited_once_with(request)

tests/unit/prompt_target/test_prompt_chat_target.py

@@ -78,6 +78,28 @@ async def _send_prompt_to_target_async(self, *, normalized_conversation: list[Me
     assert len(deprecation_warnings) >= 1
 
 
+@pytest.mark.usefixtures("patch_central_database")
+def test_instantiating_prompt_chat_target_subclass_emits_deprecation_warning():
+    """``PromptChatTarget.__init__`` is deprecated and must emit a warning when called."""
+
+    class _LegacyChatSubclassForInit(PromptChatTarget):
+        async def _send_prompt_to_target_async(self, *, normalized_conversation: list[Message]) -> list[Message]:
+            return []
+
+    with warnings.catch_warnings(record=True) as caught:
+        warnings.simplefilter("always")
+        _LegacyChatSubclassForInit()
+
+    deprecation_warnings = [
+        w
+        for w in caught
+        if issubclass(w.category, DeprecationWarning)
+        and "PromptChatTarget" in str(w.message)
+        and "0.16.0" in str(w.message)
+    ]
+    assert len(deprecation_warnings) >= 1
+
+
 @pytest.mark.usefixtures("patch_central_database")
 def test_set_system_prompt_available_on_prompt_target():
     """The set_system_prompt API now lives on PromptTarget directly."""