diff options
author | René Scharfe <l.s.r@web.de> | 2019-11-26 20:46:07 +0100 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2019-11-27 11:32:07 +0900 |
commit | 213dabf49dc53ae5808efbdebf0cd2bfc2c1bfa8 (patch) | |
tree | 81dc603474b3678c33834754529f1df186b7c89e /t/t6019-rev-list-ancestry-path.sh | |
parent | c93a5aaec859f93ff24277269e2fdef00c9c45e6 (diff) | |
download | git-213dabf49dc53ae5808efbdebf0cd2bfc2c1bfa8.tar.gz |
test: use test_must_be_empty F instead of test -z $(cat F)
Use test_must_be_empty instead of reading the file and comparing its
contents to an empty string. That's more efficient, as the function
only needs built-in meta-data only check in the usual case, and provides
nicer debug output otherwise.
Signed-off-by: René Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t6019-rev-list-ancestry-path.sh')
-rwxr-xr-x | t/t6019-rev-list-ancestry-path.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/t/t6019-rev-list-ancestry-path.sh b/t/t6019-rev-list-ancestry-path.sh index beadaf6cca..353f84313f 100755 --- a/t/t6019-rev-list-ancestry-path.sh +++ b/t/t6019-rev-list-ancestry-path.sh @@ -143,14 +143,14 @@ test_expect_success 'setup criss-cross' ' test_expect_success 'criss-cross: rev-list --ancestry-path cb..bc' ' (cd criss-cross && git rev-list --ancestry-path xcb..xbc > actual && - test -z "$(cat actual)") + test_must_be_empty actual) ' # no commits in repository descend from cb test_expect_success 'criss-cross: rev-list --ancestry-path --all ^cb' ' (cd criss-cross && git rev-list --ancestry-path --all ^xcb > actual && - test -z "$(cat actual)") + test_must_be_empty actual) ' test_done |