path: root/t/
diff options
authorJeff King <>2015-03-25 05:29:52 (GMT)
committerJunio C Hamano <>2015-03-25 17:25:27 (GMT)
commite6821d09e4655af94f9c8af07333ae710094996a (patch)
treef8c82901ba5cff198cd624354a06b907d88ebcf0 /t/
parent76e057dba29af1f322a19cb75b2921010dbe459c (diff)
t: fix some trivial cases of ignored exit codes in loops
These are all cases where we do a setup step of the form: for i in $foo; do set_up $i || break done && more_setup would not notice a failure in set_up (because break always returns a 0 exit code). These are just setup steps that we do not expect to fail, but it does not hurt to be defensive. Most can be fixed by converting the "break" to a "return 1" (since we eval our tests inside a function for just this purpose). A few of the loops are inside subshells, so we can use just "exit 1" to break out of the subshell. And a few can actually be made shorter by just unrolling the loop. Signed-off-by: Jeff King <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 't/')
1 files changed, 1 insertions, 1 deletions
diff --git a/t/ b/t/
index 0a5d5e6..6adf478 100755
--- a/t/
+++ b/t/
@@ -19,7 +19,7 @@ test_expect_success 'setup' '
> file$i &&
git add file$i &&
test_tick &&
- git commit -m branch$i || break
+ git commit -m branch$i || return 1