23036e20a6
Squashed integration of the user-facing message audit for this surface set. Full per-finding receipts: /tmp/ux-audit/lanes/*-receipt.md (campaign artifacts).
51 lines
1.8 KiB
Python
51 lines
1.8 KiB
Python
"""Regression tests: `hermes sessions` error paths return non-zero (SES-04).
|
|
|
|
Before this, delete/rename not-found, prune bad-arg, blank rename, and import
|
|
of a missing file all printed an error and returned exit 0 — a scripting/CI
|
|
hazard (a script pinning a bad id failed loudly via `pin` but deleting a bad
|
|
id "succeeded" silently). The subcommand dispatcher already maps an int
|
|
handler return to the process exit code; these tests pin the returns.
|
|
"""
|
|
|
|
from argparse import Namespace
|
|
|
|
import pytest
|
|
|
|
import hermes_cli.sessions_cmd as sc
|
|
|
|
|
|
def _args(action, **kw):
|
|
base = dict(
|
|
sessions_action=action,
|
|
session_id=None, title=None, yes=True, source=None, path=None,
|
|
from_source=None, dry_run=False, older_than=None, newer_than=None,
|
|
before=None, after=None, limit=50,
|
|
)
|
|
base.update(kw)
|
|
return Namespace(**base)
|
|
|
|
|
|
def test_delete_missing_returns_1(tmp_path, monkeypatch, capsys):
|
|
monkeypatch.setenv("HERMES_HOME", str(tmp_path))
|
|
from hermes_state import SessionDB
|
|
SessionDB(tmp_path / "state.db") # initialize an empty store
|
|
rc = sc.cmd_sessions(_args("delete", session_id="nope_xyz"))
|
|
assert rc == 1
|
|
out = capsys.readouterr().out
|
|
assert "No session 'nope_xyz'" in out and "hermes sessions list" in out
|
|
|
|
|
|
def test_rename_missing_returns_1(tmp_path, monkeypatch):
|
|
monkeypatch.setenv("HERMES_HOME", str(tmp_path))
|
|
from hermes_state import SessionDB
|
|
SessionDB(tmp_path / "state.db")
|
|
rc = sc.cmd_sessions(_args("rename", session_id="nope_xyz", title=["New"]))
|
|
assert rc == 1
|
|
|
|
|
|
def test_import_missing_file_returns_1(tmp_path, monkeypatch, capsys):
|
|
monkeypatch.setenv("HERMES_HOME", str(tmp_path))
|
|
rc = sc.cmd_sessions(_args("import", path=str(tmp_path / "nope.jsonl")))
|
|
assert rc == 1
|
|
assert "file not found" in capsys.readouterr().out.lower()
|