diff options
author | Ramkumar Ramachandra <artagnon@gmail.com> | 2011-12-09 16:59:13 +0530 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2011-12-09 11:30:42 -0800 |
commit | 5e1136227178f2ca48c472a4cc89ccfc92328965 (patch) | |
tree | 6bcf434de2c978a7cd0f2931e1049f344db4252b /t/t1505-rev-parse-last.sh | |
parent | t1510 (worktree): fix '&&' chaining (diff) | |
download | tgif-5e1136227178f2ca48c472a4cc89ccfc92328965.tar.xz |
t3040 (subprojects-basic): fix '&&' chaining, modernize style
Breaks in a test assertion's && chain can potentially hide failures
from earlier commands in the chain. Fix instances of this. While at
it, clean up the style to fit the prevailing style. This means:
- Put the opening quote starting each test on the same line as the
test_expect_* invocation.
- Indent the file with tabs, not spaces.
- Use test_expect_code() in preference to checking the exit status of
various statements by hand.
- Guard commands that prepare test input for individual tests in the
same test_expect_success, so that their scope is clearer and errors
at that stage can be caught.
- Use <<-\EOF in preference to <<EOF to save readers the trouble of
looking for variable interpolations.
- Include "setup" in the titles of test assertions that prepare for
later ones to make it more obvious which tests can be skipped.
Helped-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t1505-rev-parse-last.sh')
0 files changed, 0 insertions, 0 deletions