d09dacf66a
Change-detectors, tautologies, source-reading tests, redundant duplicates, mock-echo tests and dead/unrunnable tests. Per-test rationale in the lane ledger (category + reason for every removal).
52 lines
1.8 KiB
Python
52 lines
1.8 KiB
Python
"""Cross-surface contract for the persistent /approvals mode command."""
|
|
|
|
|
|
from hermes_cli.commands import GATEWAY_KNOWN_COMMANDS, SUBCOMMANDS, gateway_help_lines, resolve_command
|
|
from hermes_cli.commands_completion import SlashCommandCompleter
|
|
from hermes_cli.commands_platforms import telegram_bot_commands
|
|
from prompt_toolkit.completion import CompleteEvent
|
|
from prompt_toolkit.document import Document
|
|
|
|
|
|
def _completions(text: str) -> set[str]:
|
|
return {
|
|
item.text
|
|
for item in SlashCommandCompleter().get_completions(
|
|
Document(text=text), CompleteEvent(completion_requested=True)
|
|
)
|
|
}
|
|
|
|
|
|
def test_approvals_registry_drives_help_menu_and_autocomplete():
|
|
command = resolve_command("approvals")
|
|
assert command is not None
|
|
assert SUBCOMMANDS["/approvals"] == ["manual", "smart", "off"]
|
|
assert "approvals" in GATEWAY_KNOWN_COMMANDS
|
|
assert any("/approvals" in line for line in gateway_help_lines())
|
|
assert "approvals" in {name for name, _ in telegram_bot_commands()}
|
|
assert _completions("/approvals ") == {"manual", "smart", "off"}
|
|
|
|
|
|
def test_shared_command_refuses_managed_mode_override(tmp_path, monkeypatch):
|
|
from hermes_cli import managed_scope
|
|
from hermes_cli.approval_mode import run_approval_mode_command
|
|
|
|
home = tmp_path / "home"
|
|
managed = tmp_path / "managed"
|
|
home.mkdir()
|
|
managed.mkdir()
|
|
monkeypatch.setenv("HERMES_HOME", str(home))
|
|
monkeypatch.setenv("HERMES_MANAGED_DIR", str(managed))
|
|
(managed / "config.yaml").write_text("approvals:\n mode: manual\n", encoding="utf-8")
|
|
managed_scope.invalidate_managed_cache()
|
|
|
|
result = run_approval_mode_command("off")
|
|
|
|
assert result.ok is False
|
|
assert result.mode == "manual"
|
|
assert result.changed is False
|
|
assert "managed" in result.message.lower()
|
|
assert not (home / "config.yaml").exists()
|
|
|
|
|