summaryrefslogtreecommitdiff
path: root/builtin/submodule--helper.c
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2016-05-13 16:47:28 -0400
committerLibravatar Junio C Hamano <gitster@pobox.com>2016-05-14 10:37:29 -0700
commit2a86cb6dc4fbc894df1e858d94cdd45c26ccf9d0 (patch)
treec10a25b68edae582a6f0dadfbd5aea6a5c72f079 /builtin/submodule--helper.c
parentt9107: switch inverted single/double quotes in test (diff)
downloadtgif-2a86cb6dc4fbc894df1e858d94cdd45c26ccf9d0.tar.xz
t9103: modernize test style
The main goal here was to avoid double-quotes for surrounding the test snippet, since it makes the code hard to read (and to grep for common problems). But while we're here, we can fix a few other things: - use test_path_* helpers, which are more robust and give better error messages - only "cd" inside a subshell, which leaves the environment pristine if further tests are added - consistently quote shell arguments. These aren't wrong if we assume find-rev output doesn't have any whitespace, but it doesn't hurt to be careful. - replace the old-style 'test x$foo = x' with 'test -z "$foo"'. Besides the quoting fix, this is the form we generally use in our test suite. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/submodule--helper.c')
0 files changed, 0 insertions, 0 deletions