Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions tests/test_llm_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,14 @@ class TestLoadLLMConfigValidConfigs:

def test_minimal_valid_config(self, tmp_path: Path) -> None:
"""Minimal config with only required 'model' field loads correctly."""
config = {"model": "gpt-4"}
config = {"model": "gpt-4o"}
config_path = tmp_path / "config.json"
config_path.write_text(json.dumps(config))

llm = load_llm_config(config_path)

assert isinstance(llm, LLM)
assert llm.model == "gpt-4"
assert llm.model == "gpt-4o"

def test_full_valid_config(self, tmp_path: Path) -> None:
"""Config with all common fields loads correctly."""
Expand Down Expand Up @@ -211,7 +211,7 @@ def test_unreadable_file_raises_permission_error(self, tmp_path: Path) -> None:
def test_config_with_extra_fields_loads(self, tmp_path: Path) -> None:
"""Config with unknown extra fields should still load (pydantic default)."""
config = {
"model": "gpt-4",
"model": "gpt-4o",
"unknown_field": "value",
"another_unknown": 123,
}
Expand All @@ -220,11 +220,11 @@ def test_config_with_extra_fields_loads(self, tmp_path: Path) -> None:

# Should not raise - pydantic by default ignores extra fields
llm = load_llm_config(config_path)
assert llm.model == "gpt-4"
assert llm.model == "gpt-4o"

def test_unicode_in_config(self, tmp_path: Path) -> None:
"""Config with unicode characters loads correctly."""
config = {"model": "gpt-4", "api_key": "key-with-émojis-🔑"}
config = {"model": "gpt-4o", "api_key": "key-with-émojis-🔑"}
config_path = tmp_path / "config.json"
config_path.write_text(json.dumps(config, ensure_ascii=False))

Expand Down
2 changes: 1 addition & 1 deletion vendor/software-agent-sdk
Loading