From 263f6b03eb4c8778a86268e881968bc3ccdd0bf8 Mon Sep 17 00:00:00 2001 From: Pink Date: Sat, 13 Jun 2026 00:18:43 +0800 Subject: [PATCH] chore: rename test to reflect new semantics of resolve_resume_session_id --- tests/hermes_state/test_resolve_resume_session_id.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/hermes_state/test_resolve_resume_session_id.py b/tests/hermes_state/test_resolve_resume_session_id.py index 341adefde1d..2737415143d 100644 --- a/tests/hermes_state/test_resolve_resume_session_id.py +++ b/tests/hermes_state/test_resolve_resume_session_id.py @@ -48,9 +48,9 @@ def test_redirects_from_empty_head_to_descendant_with_messages(db): db.append_message("bulk", role="user", content=f"msg {i}") assert db.resolve_resume_session_id("head") == "bulk" - - -def test_returns_self_when_session_has_messages(db): +def test_returns_self_when_only_parent_has_messages(db): + # When a session already has messages AND no descendant has messages, + # it should still be returned. The chain walk finds no better candidate. _make_chain(db, [("root", None), ("child", "root")]) db.append_message("root", role="user", content="hi") assert db.resolve_resume_session_id("root") == "root"