From e5cfada7e2b9b00724c6d8c77fcfaa5c9bd6cfed Mon Sep 17 00:00:00 2001 From: Ilya Grigoriev Date: Sun, 19 Nov 2023 15:38:04 -0800 Subject: [PATCH] test_abandon_command: remove obsolete TODO This was fixed in 7c8f66ab0, I believe. --- cli/tests/test_abandon_command.rs | 5 ----- 1 file changed, 5 deletions(-) diff --git a/cli/tests/test_abandon_command.rs b/cli/tests/test_abandon_command.rs index 28710105b..9bc6f068d 100644 --- a/cli/tests/test_abandon_command.rs +++ b/cli/tests/test_abandon_command.rs @@ -97,11 +97,6 @@ fn test_rebase_branch_with_merge() { test_env.jj_cmd_ok(&repo_path, &["undo"]); let (stdout, stderr) = test_env.jj_cmd_ok(&repo_path, &["abandon", "descendants(c)"]); - // TODO(ilyagr): Minor Bug: The branch `e` should be shown next - // to the commit with description `e` below. This is because the commits are - // printed in the state *after* abandonment. This will be fixed together with - // adding (hidden) to the commit template, which causes a more obvious version - // of the same problem. insta::assert_snapshot!(stdout, @""); insta::assert_snapshot!(stderr, @r###" Abandoned the following commits: