diff options
author | Jeff King <peff@peff.net> | 2016-05-13 16:47:28 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-05-14 10:37:29 -0700 |
commit | 2a86cb6dc4fbc894df1e858d94cdd45c26ccf9d0 (patch) | |
tree | c10a25b68edae582a6f0dadfbd5aea6a5c72f079 /builtin/submodule--helper.c | |
parent | t9107: switch inverted single/double quotes in test (diff) | |
download | tgif-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