From a4fc38c5b1ce11c8a955eba27402ef7a41c5cb3f Mon Sep 17 00:00:00 2001 From: Teknium Date: Fri, 10 Apr 2026 16:47:16 -0700 Subject: [PATCH] test: remove dead TestResolveForcedProvider tests (function doesn't exist on main) --- tests/agent/test_auxiliary_client.py | 117 --------------------------- 1 file changed, 117 deletions(-) diff --git a/tests/agent/test_auxiliary_client.py b/tests/agent/test_auxiliary_client.py index 547224892..7038582ff 100644 --- a/tests/agent/test_auxiliary_client.py +++ b/tests/agent/test_auxiliary_client.py @@ -851,123 +851,6 @@ class TestGetAuxiliaryProvider: assert _get_auxiliary_provider("web_extract") == "main" -class TestResolveForcedProvider: - """Tests for _resolve_forced_provider with explicit provider selection.""" - - def test_forced_openrouter(self, monkeypatch): - monkeypatch.setenv("OPENROUTER_API_KEY", "or-key") - with patch("agent.auxiliary_client.OpenAI") as mock_openai: - client, model = _resolve_forced_provider("openrouter") - assert model == "google/gemini-3-flash-preview" - assert client is not None - - def test_forced_openrouter_no_key(self, monkeypatch): - with patch("agent.auxiliary_client._read_nous_auth", return_value=None): - client, model = _resolve_forced_provider("openrouter") - assert client is None - assert model is None - - def test_forced_nous(self, monkeypatch): - with patch("agent.auxiliary_client._read_nous_auth") as mock_nous, \ - patch("agent.auxiliary_client.OpenAI"): - mock_nous.return_value = {"access_token": "nous-tok"} - client, model = _resolve_forced_provider("nous") - assert model == "google/gemini-3-flash-preview" - assert client is not None - - def test_forced_nous_not_configured(self, monkeypatch): - with patch("agent.auxiliary_client._read_nous_auth", return_value=None): - client, model = _resolve_forced_provider("nous") - assert client is None - assert model is None - - def test_forced_main_uses_custom(self, monkeypatch): - config = { - "model": { - "provider": "custom", - "base_url": "http://local:8080/v1", - "default": "my-local-model", - } - } - monkeypatch.setenv("OPENAI_API_KEY", "local-key") - monkeypatch.setattr("hermes_cli.config.load_config", lambda: config) - monkeypatch.setattr("hermes_cli.runtime_provider.load_config", lambda: config) - with patch("agent.auxiliary_client._read_nous_auth", return_value=None), \ - patch("agent.auxiliary_client.OpenAI") as mock_openai: - client, model = _resolve_forced_provider("main") - assert model == "my-local-model" - - def test_forced_main_uses_config_saved_custom_endpoint(self, monkeypatch): - config = { - "model": { - "provider": "custom", - "base_url": "http://local:8080/v1", - "default": "my-local-model", - } - } - monkeypatch.setenv("OPENAI_API_KEY", "local-key") - monkeypatch.setattr("hermes_cli.config.load_config", lambda: config) - monkeypatch.setattr("hermes_cli.runtime_provider.load_config", lambda: config) - with patch("agent.auxiliary_client._read_nous_auth", return_value=None), \ - patch("agent.auxiliary_client._read_codex_access_token", return_value=None), \ - patch("agent.auxiliary_client._resolve_api_key_provider", return_value=(None, None)), \ - patch("agent.auxiliary_client.OpenAI") as mock_openai: - client, model = _resolve_forced_provider("main") - assert client is not None - assert model == "my-local-model" - call_kwargs = mock_openai.call_args - assert call_kwargs.kwargs["base_url"] == "http://local:8080/v1" - - def test_forced_main_skips_openrouter_nous(self, monkeypatch): - """Even if OpenRouter key is set, 'main' skips it.""" - config = { - "model": { - "provider": "custom", - "base_url": "http://local:8080/v1", - "default": "my-local-model", - } - } - monkeypatch.setenv("OPENROUTER_API_KEY", "or-key") - monkeypatch.setenv("OPENAI_API_KEY", "local-key") - monkeypatch.setattr("hermes_cli.config.load_config", lambda: config) - monkeypatch.setattr("hermes_cli.runtime_provider.load_config", lambda: config) - with patch("agent.auxiliary_client._read_nous_auth", return_value=None), \ - patch("agent.auxiliary_client.OpenAI") as mock_openai: - client, model = _resolve_forced_provider("main") - # Should use custom endpoint, not OpenRouter - assert model == "my-local-model" - - def test_forced_main_falls_to_codex(self, codex_auth_dir, monkeypatch): - with patch("agent.auxiliary_client._read_nous_auth", return_value=None), \ - patch("agent.auxiliary_client._resolve_custom_runtime", return_value=(None, None, None)), \ - patch("agent.auxiliary_client.OpenAI"): - client, model = _resolve_forced_provider("main") - from agent.auxiliary_client import CodexAuxiliaryClient - assert isinstance(client, CodexAuxiliaryClient) - assert model == "gpt-5.2-codex" - - def test_forced_codex(self, codex_auth_dir, monkeypatch): - with patch("agent.auxiliary_client._read_nous_auth", return_value=None), \ - patch("agent.auxiliary_client.OpenAI"): - client, model = _resolve_forced_provider("codex") - from agent.auxiliary_client import CodexAuxiliaryClient - assert isinstance(client, CodexAuxiliaryClient) - assert model == "gpt-5.2-codex" - - def test_forced_codex_no_token(self, monkeypatch): - with patch("agent.auxiliary_client._read_codex_access_token", return_value=None): - client, model = _resolve_forced_provider("codex") - assert client is None - assert model is None - - def test_forced_unknown_returns_none(self, monkeypatch): - with patch("agent.auxiliary_client._read_nous_auth", return_value=None), \ - patch("agent.auxiliary_client._read_codex_access_token", return_value=None): - client, model = _resolve_forced_provider("invalid-provider") - assert client is None - assert model is None - - class TestTaskSpecificOverrides: """Integration tests for per-task provider routing via get_text_auxiliary_client(task=...)."""