diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-10-14 23:37:14 (GMT) |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-10-15 16:21:30 (GMT) |
commit | 3247919a758710692e316a13c209e13933936d51 (patch) | |
tree | 5848df0ec554474bb5f572ff5cc16b8dd89abe67 | |
parent | af6d1d602a8f64164b266364339c4e936d5bbc33 (diff) | |
download | git-3247919a758710692e316a13c209e13933936d51.zip git-3247919a758710692e316a13c209e13933936d51.tar.gz git-3247919a758710692e316a13c209e13933936d51.tar.bz2 |
commit-graph tests: fix error-hiding graph_git_two_modes() helper
The graph_git_two_modes() helper added in 177722b3442 (commit:
integrate commit graph with commit parsing, 2018-04-10) didn't
&&-chain its "git commit-graph" invocations, which as can be seen with
SANITIZE=leak will happily mark tests as passing if both of these
commands die, since test_cmp() will be comparing two empty files.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rwxr-xr-x | t/t5318-commit-graph.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/t/t5318-commit-graph.sh b/t/t5318-commit-graph.sh index af88f80..5790672 100755 --- a/t/t5318-commit-graph.sh +++ b/t/t5318-commit-graph.sh @@ -51,8 +51,8 @@ test_expect_success 'create commits and repack' ' ' graph_git_two_modes() { - git -c core.commitGraph=true $1 >output - git -c core.commitGraph=false $1 >expect + git -c core.commitGraph=true $1 >output && + git -c core.commitGraph=false $1 >expect && test_cmp expect output } |