From ec8f412cca677ff2378b8862d975c4ca7fc650a0 Mon Sep 17 00:00:00 2001 From: "Babak K. Shandiz" Date: Tue, 9 Sep 2025 12:24:09 +0100 Subject: [PATCH] docs(agent-task view): explain test case rationale Signed-off-by: Babak K. Shandiz --- pkg/cmd/agent-task/view/view_test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pkg/cmd/agent-task/view/view_test.go b/pkg/cmd/agent-task/view/view_test.go index 4fcb4c66d..64b63df5b 100644 --- a/pkg/cmd/agent-task/view/view_test.go +++ b/pkg/cmd/agent-task/view/view_test.go @@ -191,6 +191,7 @@ func Test_viewRun(t *testing.T) { `), }, { + // The user data should always be there, but we need to cover the code path. name: "with session id, success, without user data (tty)", tty: true, opts: ViewOptions{ @@ -224,6 +225,7 @@ func Test_viewRun(t *testing.T) { `), }, { + // This can happen when the session is just created and a PR is not yet available for it. name: "with session id, success, without pr data (tty)", tty: true, opts: ViewOptions{ @@ -249,6 +251,7 @@ func Test_viewRun(t *testing.T) { `), }, { + // The user data should always be there, but we need to cover the code path. name: "with session id, success, without pr nor user data (tty)", tty: true, opts: ViewOptions{