diff --git a/tests/test_generator_profile_agent.py b/tests/test_generator_profile_agent.py index 7102fd9a..1472f4d4 100644 --- a/tests/test_generator_profile_agent.py +++ b/tests/test_generator_profile_agent.py @@ -1,3 +1,5 @@ +import pytest + from taosmd import agents @@ -10,3 +12,22 @@ def test_agent_generator_profile_roundtrip(tmp_path): assert agents.get_agent_generator_profile("alice", data_dir=tmp_path) == "factual-recall" agents.set_agent_generator_profile("alice", None, data_dir=tmp_path) assert agents.get_agent_generator_profile("alice", data_dir=tmp_path) is None + + +def test_set_agent_generator_profile_unknown_agent_raises(tmp_path): + agents.AgentRegistry(tmp_path).register_agent("real") + with pytest.raises(agents.AgentNotFoundError): + agents.set_agent_generator_profile("ghost", "factual-recall", data_dir=tmp_path) + + +def test_get_agent_generator_profile_unknown_agent_raises(tmp_path): + agents.AgentRegistry(tmp_path).register_agent("real") + with pytest.raises(agents.AgentNotFoundError): + agents.get_agent_generator_profile("ghost", data_dir=tmp_path) + + +def test_set_agent_generator_profile_empty_string_clears(tmp_path): + agents.AgentRegistry(tmp_path).register_agent("bob") + agents.set_agent_generator_profile("bob", "factual-recall", data_dir=tmp_path) + agents.set_agent_generator_profile("bob", "", data_dir=tmp_path) + assert agents.get_agent_generator_profile("bob", data_dir=tmp_path) is None diff --git a/tests/test_generator_profile_config.py b/tests/test_generator_profile_config.py index 1a173f9b..fabf37e3 100644 --- a/tests/test_generator_profile_config.py +++ b/tests/test_generator_profile_config.py @@ -1,3 +1,5 @@ +import pytest + from taosmd import config @@ -7,3 +9,11 @@ def test_generator_profile_roundtrip(tmp_path): assert config.get_generator_profile(data_dir=tmp_path) == "factual-recall" config.set_generator_profile("", clear=True, data_dir=tmp_path) assert config.get_generator_profile(data_dir=tmp_path) is None + + +def test_set_generator_profile_rejects_blank_and_nonstr(tmp_path): + # clear=False with a blank or non-string id must raise and persist nothing. + for bad in ("", " ", 123, None): + with pytest.raises(ValueError): + config.set_generator_profile(bad, data_dir=tmp_path) + assert config.get_generator_profile(data_dir=tmp_path) is None