diff options
| author |  Junio C Hamano <gitster@pobox.com> | 2018-08-02 15:30:46 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-08-02 15:30:46 -0700 |
| commit | cfec6133cfcd97a23ca29c7d0ad8d2961796dd52 (patch) | |
| tree | 7bdee58508f7e17c33add21d8f436687206d7446 /t/chainlint/for-loop.test | |
| parent | Merge branch 'jt/tags-to-promised-blobs-fix' (diff) | |
| parent | t/chainlint.sed: drop extra spaces from regex character class (diff) | |
| download | tgif-cfec6133cfcd97a23ca29c7d0ad8d2961796dd52.tar.xz | |
Merge branch 'es/chain-lint-in-subshell'
Look for broken "&&" chains that are hidden in subshell, many of
which have been found and corrected.
* es/chain-lint-in-subshell:
t/chainlint.sed: drop extra spaces from regex character class
t/chainlint: add chainlint "specialized" test cases
t/chainlint: add chainlint "complex" test cases
t/chainlint: add chainlint "cuddled" test cases
t/chainlint: add chainlint "loop" and "conditional" test cases
t/chainlint: add chainlint "nested subshell" test cases
t/chainlint: add chainlint "one-liner" test cases
t/chainlint: add chainlint "whitespace" test cases
t/chainlint: add chainlint "basic" test cases
t/Makefile: add machinery to check correctness of chainlint.sed
t/test-lib: teach --chain-lint to detect broken &&-chains in subshells
Diffstat (limited to 't/chainlint/for-loop.test')
| -rw-r--r-- | t/chainlint/for-loop.test | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/t/chainlint/for-loop.test b/t/chainlint/for-loop.test new file mode 100644 index 0000000000..7db76262bc --- /dev/null +++ b/t/chainlint/for-loop.test @@ -0,0 +1,19 @@ +( +# LINT: 'for', 'do', 'done' do not need "&&" + for i in a b c + do +# LINT: missing "&&" on 'echo' + echo $i +# LINT: last statement of while does not need "&&" + cat <<-\EOF + bar + EOF +# LINT: missing "&&" on 'done' + done + +# LINT: 'do' on same line as 'for' + for i in a b c; do + echo $i && + cat $i + done +) |