| """Tests for hermes tools disable/enable/list command (backend).""" |
| from argparse import Namespace |
| from unittest.mock import patch |
|
|
| from hermes_cli.tools_config import tools_disable_enable_command |
|
|
|
|
| |
|
|
|
|
| class TestToolsDisableBuiltin: |
|
|
| def test_disable_removes_toolset_from_platform(self): |
| config = {"platform_toolsets": {"cli": ["web", "memory", "terminal"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command(Namespace(tools_action="disable", names=["web"], platform="cli")) |
| saved = mock_save.call_args[0][0] |
| assert "web" not in saved["platform_toolsets"]["cli"] |
| assert "memory" in saved["platform_toolsets"]["cli"] |
|
|
| def test_disable_multiple_toolsets(self): |
| config = {"platform_toolsets": {"cli": ["web", "memory", "terminal"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command(Namespace(tools_action="disable", names=["web", "memory"], platform="cli")) |
| saved = mock_save.call_args[0][0] |
| assert "web" not in saved["platform_toolsets"]["cli"] |
| assert "memory" not in saved["platform_toolsets"]["cli"] |
| assert "terminal" in saved["platform_toolsets"]["cli"] |
|
|
| def test_disable_already_absent_is_idempotent(self): |
| config = {"platform_toolsets": {"cli": ["memory"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command(Namespace(tools_action="disable", names=["web"], platform="cli")) |
| saved = mock_save.call_args[0][0] |
| assert "web" not in saved["platform_toolsets"]["cli"] |
|
|
|
|
| |
|
|
|
|
| class TestToolsEnableBuiltin: |
|
|
| def test_enable_adds_toolset_to_platform(self): |
| config = {"platform_toolsets": {"cli": ["memory"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command(Namespace(tools_action="enable", names=["web"], platform="cli")) |
| saved = mock_save.call_args[0][0] |
| assert "web" in saved["platform_toolsets"]["cli"] |
|
|
| def test_enable_already_present_is_idempotent(self): |
| config = {"platform_toolsets": {"cli": ["web"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command(Namespace(tools_action="enable", names=["web"], platform="cli")) |
| saved = mock_save.call_args[0][0] |
| assert saved["platform_toolsets"]["cli"].count("web") == 1 |
|
|
|
|
| |
|
|
|
|
| class TestToolsDisableMcp: |
|
|
| def test_disable_adds_to_exclude_list(self): |
| config = {"mcp_servers": {"github": {"command": "npx"}}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command( |
| Namespace(tools_action="disable", names=["github:create_issue"], platform="cli") |
| ) |
| saved = mock_save.call_args[0][0] |
| assert "create_issue" in saved["mcp_servers"]["github"]["tools"]["exclude"] |
|
|
| def test_disable_already_excluded_is_idempotent(self): |
| config = {"mcp_servers": {"github": {"tools": {"exclude": ["create_issue"]}}}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command( |
| Namespace(tools_action="disable", names=["github:create_issue"], platform="cli") |
| ) |
| saved = mock_save.call_args[0][0] |
| assert saved["mcp_servers"]["github"]["tools"]["exclude"].count("create_issue") == 1 |
|
|
| def test_disable_unknown_server_prints_error(self, capsys): |
| config = {"mcp_servers": {}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config"): |
| tools_disable_enable_command( |
| Namespace(tools_action="disable", names=["unknown:tool"], platform="cli") |
| ) |
| out = capsys.readouterr().out |
| assert "MCP server 'unknown' not found in config" in out |
|
|
|
|
| |
|
|
|
|
| class TestToolsEnableMcp: |
|
|
| def test_enable_removes_from_exclude_list(self): |
| config = {"mcp_servers": {"github": {"tools": {"exclude": ["create_issue", "delete_branch"]}}}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command( |
| Namespace(tools_action="enable", names=["github:create_issue"], platform="cli") |
| ) |
| saved = mock_save.call_args[0][0] |
| assert "create_issue" not in saved["mcp_servers"]["github"]["tools"]["exclude"] |
| assert "delete_branch" in saved["mcp_servers"]["github"]["tools"]["exclude"] |
|
|
|
|
| |
|
|
|
|
| class TestToolsMixedTargets: |
|
|
| def test_disable_builtin_and_mcp_together(self): |
| config = { |
| "platform_toolsets": {"cli": ["web", "memory"]}, |
| "mcp_servers": {"github": {"command": "npx"}}, |
| } |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command(Namespace( |
| tools_action="disable", |
| names=["web", "github:create_issue"], |
| platform="cli", |
| )) |
| saved = mock_save.call_args[0][0] |
| assert "web" not in saved["platform_toolsets"]["cli"] |
| assert "create_issue" in saved["mcp_servers"]["github"]["tools"]["exclude"] |
|
|
|
|
| |
|
|
|
|
| class TestToolsList: |
|
|
| def test_list_shows_enabled_toolsets(self, capsys): |
| config = {"platform_toolsets": {"cli": ["web", "memory"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config): |
| tools_disable_enable_command(Namespace(tools_action="list", platform="cli")) |
| out = capsys.readouterr().out |
| assert "web" in out |
| assert "memory" in out |
|
|
| def test_list_shows_mcp_excluded_tools(self, capsys): |
| config = { |
| "mcp_servers": {"github": {"tools": {"exclude": ["create_issue"]}}}, |
| } |
| with patch("hermes_cli.tools_config.load_config", return_value=config): |
| tools_disable_enable_command(Namespace(tools_action="list", platform="cli")) |
| out = capsys.readouterr().out |
| assert "github" in out |
| assert "create_issue" in out |
|
|
|
|
| |
|
|
|
|
| class TestToolsValidation: |
|
|
| def test_unknown_platform_prints_error(self, capsys): |
| config = {} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config"): |
| tools_disable_enable_command( |
| Namespace(tools_action="disable", names=["web"], platform="invalid_platform") |
| ) |
| out = capsys.readouterr().out |
| assert "Unknown platform 'invalid_platform'" in out |
|
|
| def test_unknown_toolset_prints_error(self, capsys): |
| config = {"platform_toolsets": {"cli": ["web"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config"): |
| tools_disable_enable_command( |
| Namespace(tools_action="disable", names=["nonexistent_toolset"], platform="cli") |
| ) |
| out = capsys.readouterr().out |
| assert "Unknown toolset 'nonexistent_toolset'" in out |
|
|
| def test_unknown_toolset_does_not_corrupt_config(self): |
| config = {"platform_toolsets": {"cli": ["web", "memory"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command( |
| Namespace(tools_action="disable", names=["nonexistent_toolset"], platform="cli") |
| ) |
| saved = mock_save.call_args[0][0] |
| assert "web" in saved["platform_toolsets"]["cli"] |
| assert "memory" in saved["platform_toolsets"]["cli"] |
|
|
| def test_mixed_valid_and_invalid_applies_valid_only(self): |
| config = {"platform_toolsets": {"cli": ["web", "memory"]}} |
| with patch("hermes_cli.tools_config.load_config", return_value=config), \ |
| patch("hermes_cli.tools_config.save_config") as mock_save: |
| tools_disable_enable_command( |
| Namespace(tools_action="disable", names=["web", "bad_toolset"], platform="cli") |
| ) |
| saved = mock_save.call_args[0][0] |
| assert "web" not in saved["platform_toolsets"]["cli"] |
| assert "memory" in saved["platform_toolsets"]["cli"] |
|
|
