From 14a4f6f30c0a39606cced957f85b1df56361882d Mon Sep 17 00:00:00 2001 From: Tristan Simas Date: Fri, 23 Oct 2026 18:32:29 -0400 Subject: [PATCH] Build the session details test notifications from Core's declared state The fixture passed "owner", "Responded", "" positionally into MessageNotification, so a str sat in the DeliveryProgress state field and ThreadStatusModel.known raised on .notification_label. Core declared these fields typed on 10-16 (83ee78593); only this test kept the old shape. No toad src constructs MessageNotification; Core builds it at assignment_states.py:161 with a typed state. Co-Authored-By: Claude Opus 5.5 --- tests/textual/test_session_details.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/textual/test_session_details.py b/tests/textual/test_session_details.py index 8078009f0..9d195ae70 100644 --- a/tests/textual/test_session_details.py +++ b/tests/textual/test_session_details.py @@ -5,7 +5,9 @@ from textual.screen import Screen from textual.widgets import Input +from agent_comms.audience_manifest import FrozenRecipient from agent_comms.messages import InfoKind, Message +from agent_comms.notification_progress import AcknowledgedProgress from agent_comms.presentation import MessageNotification from agent_comms.thread_presentation import ThreadPresentation from agent_comms.ui_model.delivery import InputDelivery @@ -20,7 +22,7 @@ @pytest.mark.parametrize("ansi", [False, True], ids=["rgb", "ansi"]) async def test_details_are_one_row_until_opened_and_then_bounded(workspace, ansi): notifications = tuple(MessageNotification( - "owner", "Responded", "", message=Message("peer", "#comms", f"Notice {index}: " + "body " * 30, + FrozenRecipient("owner", "owner"), "peer", AcknowledgedProgress(), message=Message("peer", "#comms", f"Notice {index}: " + "body " * 30, InfoKind, seq=index + 1), ) for index in range(30)) state = ThreadPresentation("owner", "●", "Ready", notifications=notifications)