diff options
| author |  Jeff King <peff@peff.net> | 2015-03-20 06:13:25 -0400 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2015-03-20 11:35:56 -0700 |
| commit | bfe998fc9b5a02271941c7c28f55f6dd000fe76f (patch) | |
| tree | 9e89c80186cfe742ebdb3dc0ecf44ea16a5b8bed /t/t0050-filesystem.sh | |
| parent | t9001: use test_when_finished (diff) | |
| download | tgif-bfe998fc9b5a02271941c7c28f55f6dd000fe76f.tar.xz | |
t0050: appease --chain-lint
Some of the symlink tests check an either-or case using the
"||". This is not wrong, but fools --chain-lint into
thinking the &&-chain is broken (in fact, there is no &&
chain here).
We can solve this by wrapping the "||" inside a {} block.
This is a bit more verbose, but this construct is rare, and
the {} block helps call attention to it.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t0050-filesystem.sh')
| -rwxr-xr-x | t/t0050-filesystem.sh | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/t/t0050-filesystem.sh b/t/t0050-filesystem.sh index 988c3925d5..b29d749bb7 100755 --- a/t/t0050-filesystem.sh +++ b/t/t0050-filesystem.sh @@ -33,16 +33,20 @@ test_expect_success "detection of case insensitive filesystem during repo init" ' else test_expect_success "detection of case insensitive filesystem during repo init" ' - test_must_fail git config --bool core.ignorecase >/dev/null || - test $(git config --bool core.ignorecase) = false + { + test_must_fail git config --bool core.ignorecase >/dev/null || + test $(git config --bool core.ignorecase) = false + } ' fi if test_have_prereq SYMLINKS then test_expect_success "detection of filesystem w/o symlink support during repo init" ' - test_must_fail git config --bool core.symlinks || - test "$(git config --bool core.symlinks)" = true + { + test_must_fail git config --bool core.symlinks || + test "$(git config --bool core.symlinks)" = true + } ' else test_expect_success "detection of filesystem w/o symlink support during repo init" ' |