@leroysheep / Socializer / commits / 89454a60ed

merge-base: add tests for --is-ancestor

`git merge-base --is-ancestor A B` is used a lot in scripts but has no tests. Add some to t6010 covering its exit codes: 0 when A is an ancestor of B, 1 when it is not, and 128 (not 1) when given a bad argument. Also check that --is-ancestor and --all can't be combined, and that the resulting error names both options. Signed-off-by: Nikolaus Schuetz <nikolauspschuetz@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Nikolaus Schuetz committed Jul 30, 2026 at 06:19 UTC 89454a60ed3c668aabdc372327207096c9613e56
1 file changed +31
t/t6010-merge-base.sh
+31
index 44c726ea39..21f09a678f 100755 --- a/t/t6010-merge-base.sh +++ b/t/t6010-merge-base.sh @@ -305,4 +305,35 @@ test_expect_success 'merge-base --octopus --all for complex tree' ' test_cmp expected actual ' +test_expect_success '--is-ancestor with an ancestor and a descendant' ' + git merge-base --is-ancestor $E $A && + test_expect_code 1 git merge-base --is-ancestor $A $E +' + +test_expect_success '--is-ancestor treats a commit as its own ancestor' ' + git merge-base --is-ancestor $A $A +' + +test_expect_success '--is-ancestor with diverged commits' ' + test_expect_code 1 git merge-base --is-ancestor $G $H && + test_expect_code 1 git merge-base --is-ancestor $H $G +' + +test_expect_success '--is-ancestor exits 128 on a bad commit' ' + test_expect_code 128 git merge-base --is-ancestor $A no-such-commit && + test_expect_code 128 git merge-base --is-ancestor no-such-commit $A +' + +test_expect_success '--is-ancestor requires exactly two commits' ' + test_expect_code 129 git merge-base --is-ancestor && + test_expect_code 129 git merge-base --is-ancestor $A && + test_expect_code 128 git merge-base --is-ancestor $E $A $B 2>err && + test_grep ".--is-ancestor takes exactly two commits" err +' + +test_expect_success '--is-ancestor and --all cannot be used together' ' + test_expect_code 128 git merge-base --is-ancestor --all $E $A 2>err && + test_grep "options .--is-ancestor. and .--all. cannot be used together" err +' + test_done