Spaces:
Sleeping
Sleeping
Add tests for model overrides in manual input and file upload controllers
Browse files
tests/test_model_overrides_cross_mode.py
ADDED
|
@@ -0,0 +1,75 @@
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1 |
+
import types
|
| 2 |
+
|
| 3 |
+
import pytest
|
| 4 |
+
|
| 5 |
+
from src.interface.manual_input_interface import ManualInputController
|
| 6 |
+
from src.interface.file_upload_interface import FileUploadInterfaceController
|
| 7 |
+
|
| 8 |
+
|
| 9 |
+
def test_manual_input_controller_applies_overrides_to_ai_client():
|
| 10 |
+
controller = ManualInputController()
|
| 11 |
+
|
| 12 |
+
controller.set_model_overrides({"SpiritualDistressAnalyzer": "gemini-test-model"})
|
| 13 |
+
|
| 14 |
+
assert controller.model_overrides["SpiritualDistressAnalyzer"] == "gemini-test-model"
|
| 15 |
+
assert controller.ai_client.model_overrides["SpiritualDistressAnalyzer"] == "gemini-test-model"
|
| 16 |
+
|
| 17 |
+
|
| 18 |
+
def test_file_upload_controller_applies_overrides_to_ai_client():
|
| 19 |
+
controller = FileUploadInterfaceController()
|
| 20 |
+
|
| 21 |
+
controller.set_model_overrides({"SpiritualDistressAnalyzer": "gemini-test-model"})
|
| 22 |
+
|
| 23 |
+
assert controller.model_overrides["SpiritualDistressAnalyzer"] == "gemini-test-model"
|
| 24 |
+
assert controller.ai_client.model_overrides["SpiritualDistressAnalyzer"] == "gemini-test-model"
|
| 25 |
+
|
| 26 |
+
|
| 27 |
+
def test_file_upload_batch_passes_override_into_call(monkeypatch):
|
| 28 |
+
controller = FileUploadInterfaceController()
|
| 29 |
+
controller.set_model_overrides({"SpiritualDistressAnalyzer": "gemini-batch-override"})
|
| 30 |
+
|
| 31 |
+
# Minimal session + dataset state so the batch method can run without UI.
|
| 32 |
+
controller.current_session = types.SimpleNamespace(
|
| 33 |
+
verifications=[],
|
| 34 |
+
verified_count=0,
|
| 35 |
+
correct_count=0,
|
| 36 |
+
incorrect_count=0,
|
| 37 |
+
verified_message_ids=[],
|
| 38 |
+
current_queue_index=0,
|
| 39 |
+
is_complete=False,
|
| 40 |
+
completed_at=None,
|
| 41 |
+
)
|
| 42 |
+
|
| 43 |
+
controller.current_file_result = types.SimpleNamespace(
|
| 44 |
+
parsed_test_cases=[
|
| 45 |
+
types.SimpleNamespace(
|
| 46 |
+
message_id="m1",
|
| 47 |
+
text="msg1",
|
| 48 |
+
pre_classified_label="green",
|
| 49 |
+
)
|
| 50 |
+
]
|
| 51 |
+
)
|
| 52 |
+
|
| 53 |
+
# Avoid touching file processing / exporters.
|
| 54 |
+
controller._initialize_progress_tracker = lambda total_messages: None
|
| 55 |
+
controller._create_session = lambda file_result, file_name: None
|
| 56 |
+
controller._update_progress_display_text = lambda *args, **kwargs: ""
|
| 57 |
+
controller.setup_progress_tracking = lambda total_messages: None
|
| 58 |
+
controller.record_verification_with_timing = lambda *args, **kwargs: None
|
| 59 |
+
controller.store.save_session = lambda session: None
|
| 60 |
+
|
| 61 |
+
captured = {}
|
| 62 |
+
|
| 63 |
+
def fake_call_spiritual_api(*, system_prompt=None, user_prompt=None, model_override=None, **kwargs):
|
| 64 |
+
captured["system_prompt"] = system_prompt
|
| 65 |
+
captured["user_prompt"] = user_prompt
|
| 66 |
+
captured["model_override"] = model_override
|
| 67 |
+
return "green"
|
| 68 |
+
|
| 69 |
+
monkeypatch.setattr(controller.ai_client, "call_spiritual_api", fake_call_spiritual_api)
|
| 70 |
+
|
| 71 |
+
# Run the simplest possible classification path.
|
| 72 |
+
result = controller.run_batch_classification()
|
| 73 |
+
|
| 74 |
+
assert captured["model_override"] == "gemini-batch-override"
|
| 75 |
+
assert result is not None
|