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>
Jeff King committed
Mar 20, 2015 at 06:13 UTC
bfe998fc9b5a02271941c7c28f55f6dd000fe76f
1 file changed
+8
-4
t/t0050-filesystem.sh
+8
-4
@@ -33,16 +33,20 @@ test_expect_success "detection of case insensitive filesystem during repo init"
33
'
34
else
35
test_expect_success "detection of case insensitive filesystem during repo init" '
36
- test_must_fail git config --bool core.ignorecase >/dev/null ||
37
- test $(git config --bool core.ignorecase) = false
36
+ {
37
+ test_must_fail git config --bool core.ignorecase >/dev/null ||
38
+ test $(git config --bool core.ignorecase) = false
39
+ }
40
'
41
fi
42
43
if test_have_prereq SYMLINKS
44
then
45
test_expect_success "detection of filesystem w/o symlink support during repo init" '
44
- test_must_fail git config --bool core.symlinks ||
45
- test "$(git config --bool core.symlinks)" = true
46
+ {
47
+ test_must_fail git config --bool core.symlinks ||
48
+ test "$(git config --bool core.symlinks)" = true
49
+ }
50
'
51
else
52
test_expect_success "detection of filesystem w/o symlink support during repo init" '