]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t6000-t9999: detect and signal failure within loop
authorEric Sunshine <sunshine@sunshineco.com>
Thu, 9 Dec 2021 05:11:15 +0000 (00:11 -0500)
committerJunio C Hamano <gitster@pobox.com>
Mon, 13 Dec 2021 18:29:48 +0000 (10:29 -0800)
Failures within `for` and `while` loops can go unnoticed if not detected
and signaled manually since the loop itself does not abort when a
contained command fails, nor will a failure necessarily be detected when
the loop finishes since the loop returns the exit code of the last
command it ran on the final iteration, which may not be the command
which failed. Therefore, detect and signal failures manually within
loops using the idiom `|| return 1` (or `|| exit 1` within subshells).

Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Reviewed-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
32 files changed:
contrib/mw-to-git/t/t9365-continuing-queries.sh
t/annotate-tests.sh
t/t6005-rev-list-count.sh
t/t6009-rev-list-parent.sh
t/t6101-rev-parse-parents.sh
t/t6112-rev-list-filters-objects.sh
t/t6120-describe.sh
t/t6132-pathspec-exclude.sh
t/t6200-fmt-merge-msg.sh
t/t6300-for-each-ref.sh
t/t6412-merge-large-rename.sh
t/t6430-merge-recursive.sh
t/t6600-test-reach.sh
t/t7004-tag.sh
t/t7505-prepare-commit-msg-hook.sh
t/t7600-merge.sh
t/t7602-merge-octopus-many.sh
t/t7603-merge-reduce-heads.sh
t/t7700-repack.sh
t/t8014-blame-ignore-fuzzy.sh
t/t9104-git-svn-follow-parent.sh
t/t9130-git-svn-authors-file.sh
t/t9134-git-svn-ignore-paths.sh
t/t9138-git-svn-authors-prog.sh
t/t9146-git-svn-empty-dirs.sh
t/t9147-git-svn-include-paths.sh
t/t9152-svn-empty-dirs-after-gc.sh
t/t9304-fast-import-marks.sh
t/t9400-git-cvsserver-server.sh
t/t9800-git-p4-basic.sh
t/t9818-git-p4-block.sh
t/t9902-completion.sh

index 016454749f8dfa8539c7af1ca7fe4338414ab661..d3e73126595df72b5d422b6afe708f16f15e9894 100755 (executable)
@@ -12,7 +12,7 @@ test_expect_success 'creating page w/ >500 revisions' '
        for i in $(test_seq 501)
        do
                echo "creating revision $i" &&
-               wiki_editpage foo "revision $i<br/>" true
+               wiki_editpage foo "revision $i<br/>" true || return 1
        done
 '
 
index d3b299e75cb1831d2c1044d8df979bc81f22e177..09e86f9ba0804f70d6561b5bda03d69535f11dcf 100644 (file)
@@ -161,7 +161,7 @@ test_expect_success 'blame huge graft' '
                        GIT_AUTHOR_NAME=$i$j GIT_AUTHOR_EMAIL=$i$j@test.git \
                        git commit -a -m "$i$j" &&
                        commit=$(git rev-parse --verify HEAD) &&
-                       graft="$graft$commit "
+                       graft="$graft$commit " || return 1
                done
        done &&
        printf "%s " $graft >.git/info/grafts &&
index 2a2083e6bc262ab38d278dc1aa85184b50ea1492..bdc0ebbfa02412c101dd84da0fce8efebea40893 100755 (executable)
@@ -8,7 +8,7 @@ test_expect_success 'setup' '
     for n in 1 2 3 4 5 ; do
        echo $n > a &&
        git add a &&
-       git commit -m "$n"
+       git commit -m "$n" || return 1
     done
 '
 
index dc8160aa45dacfa38135a1307077e349363c5497..5a67bbc760fdc8c2b53dc7721a144e033bf8e934 100755 (executable)
@@ -124,7 +124,7 @@ test_expect_success 'dodecapus' '
                git checkout -b root$i five &&
                test_commit $i &&
                roots="$roots root$i" ||
-               return
+               return 1
        done &&
        git checkout main &&
        test_tick &&
@@ -143,7 +143,7 @@ test_expect_success 'ancestors with the same commit time' '
        test_tick_keep=$test_tick &&
        for i in 1 2 3 4 5 6 7 8; do
                test_tick=$test_tick_keep &&
-               test_commit t$i
+               test_commit t$i || return 1
        done &&
        git rev-list t1^! --not t$i >result &&
        test_must_be_empty result
index 78b585178051615a3712b58d1f87192aa678fb7f..c571fa517978818703e29c72e36faf52643a8ee5 100755 (executable)
@@ -32,7 +32,7 @@ test_expect_success 'setup' '
                test_tick &&
                git commit --allow-empty -m "$i" &&
                commit=$(git rev-parse --verify HEAD) &&
-               printf "$commit " >>.git/info/grafts
+               printf "$commit " >>.git/info/grafts || return 1
        done
 '
 
index 9848425192365e0c3cf0748400b62045ae2549fc..8d9d6604f052f6645e8760d3890f1ed29384fd49 100755 (executable)
@@ -18,7 +18,7 @@ test_expect_success 'setup r1' '
        do
                echo "This is file: $n" > r1/file.$n &&
                git -C r1 add file.$n &&
-               git -C r1 commit -m "$n"
+               git -C r1 commit -m "$n" || return 1
        done
 '
 
@@ -75,7 +75,7 @@ test_expect_success 'setup r2' '
        do
                printf "%"$n"s" X > r2/large.$n &&
                git -C r2 add large.$n &&
-               git -C r2 commit -m "$n"
+               git -C r2 commit -m "$n" || return 1
        done
 '
 
@@ -248,7 +248,7 @@ test_expect_success 'setup r3' '
                echo "This is file: $n" > r3/$n &&
                git -C r3 add $n &&
                echo "This is file: dir1/$n" > r3/dir1/$n &&
-               git -C r3 add dir1/$n
+               git -C r3 add dir1/$n || return 1
        done &&
        git -C r3 commit -m "sparse" &&
        echo dir1/ >pattern1 &&
@@ -672,7 +672,7 @@ test_expect_success 'rev-list W/ --missing=print' '
 
        for id in `cat expected | sed "s|..|&/|"`
        do
-               rm r1/.git/objects/$id
+               rm r1/.git/objects/$id || return 1
        done &&
 
        git -C r1 rev-list --quiet --missing=print --objects HEAD >revs &&
index 9b2cc066f774775800d7245ac500c12c92e6d4a3..d8af2bb9d2b876c3277d1211468ce4717b83697e 100755 (executable)
@@ -262,7 +262,7 @@ test_expect_success 'name-rev --all' '
        >expect.unsorted &&
        for rev in $(git rev-list --all)
        do
-               git name-rev $rev >>expect.unsorted
+               git name-rev $rev >>expect.unsorted || return 1
        done &&
        sort <expect.unsorted >expect &&
        git name-rev --all >actual.unsorted &&
@@ -275,7 +275,7 @@ test_expect_success 'name-rev --stdin' '
        for rev in $(git rev-list --all)
        do
                name=$(git name-rev --name-only $rev) &&
-               echo "$rev ($name)" >>expect.unsorted
+               echo "$rev ($name)" >>expect.unsorted || return 1
        done &&
        sort <expect.unsorted >expect &&
        git rev-list --all | git name-rev --stdin >actual.unsorted &&
@@ -395,7 +395,7 @@ EOF" &&
                then
                        echo "from refs/heads/main^0"
                fi &&
-               i=$(($i + 1))
+               i=$(($i + 1)) || return 1
        done | git fast-import &&
        git checkout main &&
        git tag far-far-away HEAD^ &&
index 30328b87f07657c899801304bf6e0331f569f1bb..8ff1d76f794783346c1b3597c55924e8acb4811d 100755 (executable)
@@ -11,7 +11,7 @@ test_expect_success 'setup' '
                fi &&
                : >$p &&
                git add $p &&
-               git commit -m $p
+               git commit -m $p || return 1
        done &&
        git log --oneline --format=%s >actual &&
        cat <<EOF >expect &&
index 06c5fb56157f28359b47c04818e2a51f9098ee36..5cf7b7673e497af5f84e6c5a02bc04a313ebc0cf 100755 (executable)
@@ -519,7 +519,7 @@ test_expect_success 'merge-msg lots of commits' '
                while test $i -gt 9
                do
                        echo "  $i" &&
-                       i=$(($i-1))
+                       i=$(($i-1)) || return 1
                done &&
                echo "  ..."
        } >expected &&
index aae57908f88c7c15b1ad33a95de3dfc3e5df24bb..abc8e1495afee8c732cec9011097bde17fe1c10b 100755 (executable)
@@ -1338,7 +1338,7 @@ test_expect_success ':remotename and :remoteref' '
                        echo "${pair#*=}" >expect &&
                        git for-each-ref --format="${pair%=*}" \
                                refs/heads/main >actual &&
-                       test_cmp expect actual
+                       test_cmp expect actual || exit 1
                done &&
                git branch push-simple &&
                git config branch.push-simple.pushRemote from &&
index ed40801380737d45e2c808066983d4b293b3a44f..ca018d11f547978bf66a41804c16bf24a0b614f8 100755 (executable)
@@ -37,18 +37,18 @@ test_rename() {
        test_might_fail git branch -D test$n &&
        git reset --hard initial &&
        for i in $(count $n); do
-               make_text $i initial initial >$i
+               make_text $i initial initial >$i || return 1
        done &&
        git add . &&
        git commit -m add=$n &&
        for i in $(count $n); do
-               make_text $i changed initial >$i
+               make_text $i changed initial >$i || return 1
        done &&
        git commit -a -m change=$n &&
        git checkout -b test$n HEAD^ &&
        for i in $(count $n); do
                git rm $i &&
-               make_text $i initial changed >$i.moved
+               make_text $i initial changed >$i.moved || return 1
        done &&
        git add . &&
        git commit -m change+rename=$n &&
@@ -79,7 +79,7 @@ test_expect_success 'setup large simple rename' '
 
        git reset --hard initial &&
        for i in $(count 200); do
-               make_text foo bar baz >$i
+               make_text foo bar baz >$i || return 1
        done &&
        git add . &&
        git commit -m create-files &&
index a0efe7cb6dbe77215f0b2bd220f6974ad0c17ec0..07067bb347955b146b9654b353e9db2b8b2cd323 100755 (executable)
@@ -706,7 +706,7 @@ test_expect_success 'merge-recursive remembers the names of all base trees' '
        # more trees than static slots used by oid_to_hex()
        for commit in $c0 $c2 $c4 $c5 $c6 $c7
        do
-               git rev-parse "$commit^{tree}"
+               git rev-parse "$commit^{tree}" || return 1
        done >trees &&
 
        # ignore the return code; it only fails because the input is weird...
index 3d7a62ddab61729d37c183fe327a9d8997f96c1c..338a9c46a24b02caf471da4afb1cff14a75b6f2f 100755 (executable)
@@ -32,7 +32,7 @@ test_expect_success 'setup' '
        do
                test_commit "1-$i" &&
                git branch -f commit-1-$i &&
-               git tag -a -m "1-$i" tag-1-$i commit-1-$i
+               git tag -a -m "1-$i" tag-1-$i commit-1-$i || return 1
        done &&
        for j in $(test_seq 1 9)
        do
@@ -46,7 +46,7 @@ test_expect_success 'setup' '
                do
                        git merge commit-$j-$i -m "$x-$i" &&
                        git branch -f commit-$x-$i &&
-                       git tag -a -m "$x-$i" tag-$x-$i commit-$x-$i
+                       git tag -a -m "$x-$i" tag-$x-$i commit-$x-$i || return 1
                done
        done &&
        git commit-graph write --reachable &&
index 660cde5a63981679c6eb58377e71bd3b4a5007d2..25a26563dd2c45a65dd4bffad2b8e5292a90cf2d 100755 (executable)
@@ -1981,7 +1981,7 @@ EOF" &&
                then
                        echo "from refs/heads/main^0"
                fi &&
-               i=$(($i + 1))
+               i=$(($i + 1)) || return 1
        done | git fast-import &&
        git checkout main &&
        git tag far-far-away HEAD^ &&
index 2a07c70867966ccfa901d91667358a7d86f34e5e..e39c809ca42f04dbda578689e627b4aceb67d044 100755 (executable)
@@ -16,7 +16,7 @@ test_expect_success 'set up commits for rebasing' '
        test_commit rebase-b b bb &&
        for i in $(test_seq 1 13)
        do
-               test_commit rebase-$i c $i
+               test_commit rebase-$i c $i || return 1
        done &&
        git checkout main &&
 
index c773e30b3fa17bc129ca78afe3aef9194d77cb1d..f0f6fda150bc29695dc8b56cb3c204de251e6e1d 100755 (executable)
@@ -967,7 +967,7 @@ test_expect_success 'set up mod-256 conflict scenario' '
        # 256 near-identical stanzas...
        for i in $(test_seq 1 256); do
                for j in 1 2 3 4 5; do
-                       echo $i-$j
+                       echo $i-$j || return 1
                done
        done >file &&
        git add file &&
index ddf64dc5f75e7cc3524dffc2cedcb6a8b49230c0..ff085b086cc38f36a180e22ab02bbea12a29cc0c 100755 (executable)
@@ -30,7 +30,7 @@ test_expect_success 'merge c1 with c2, c3, c4, ... c29' '
        while test $i -le 30
        do
                refs="$refs c$i" &&
-               i=$(expr $i + 1)
+               i=$(expr $i + 1) || return 1
        done &&
        git merge $refs &&
        test "$(git rev-parse c1)" != "$(git rev-parse HEAD)" &&
index 27cd94ad6f7770f93c84bad8a0a966e6d026ea78..4887ca705b330e8cbf6f25595cbcadfdfd67f9f2 100755 (executable)
@@ -95,7 +95,7 @@ test_expect_success 'setup' '
                echo $i > $i.c &&
                git add $i.c &&
                git commit -m $i &&
-               git tag $i
+               git tag $i || return 1
        done &&
        git reset --hard A &&
        for i in F G H I
@@ -103,7 +103,7 @@ test_expect_success 'setup' '
                echo $i > $i.c &&
                git add $i.c &&
                git commit -m $i &&
-               git tag $i
+               git tag $i || return 1
        done
 '
 
index 0260ad6f0e06ec3cbd4d6515a4f893e219b42b0a..4693f8dc2bb9d5182649af7b1cadba26750a1c03 100755 (executable)
@@ -117,7 +117,7 @@ test_expect_success 'packed obs in alternate ODB kept pack are repacked' '
                        rm alt_objects/pack/$base_name.keep
                else
                        touch alt_objects/pack/$base_name.keep
-               fi
+               fi || return 1
        done &&
        git repack -a -d &&
        test_no_missing_in_packs
index e68e6115a66d3722413a1982aa86cc2a5d5932eb..0bd034130189db8561d824449de646f2bfcef61f 100755 (executable)
@@ -310,7 +310,7 @@ test_expect_success setup '
                        echo "$line" >>"$i" &&
                        git add "$i" &&
                        test_tick &&
-                       GIT_AUTHOR_NAME="$line_count" git commit -m "$line_count"
+                       GIT_AUTHOR_NAME="$line_count" git commit -m "$line_count" || return 1
                done <"a$i"
        done &&
 
@@ -318,7 +318,7 @@ test_expect_success setup '
        do
                # Overwrite the files with the final content.
                cp b$i $i &&
-               git add $i
+               git add $i || return 1
        done &&
        test_tick &&
 
index 67eed2fefcec974f7f166b3221150392323211c5..c7d8e0bf00f6344d1216f117f57587cda3116fc2 100755 (executable)
@@ -117,7 +117,7 @@ test_expect_success 'follow-parent avoids deleting relevant info' '
        mkdir -p import/trunk/subversion/bindings/swig/perl/t &&
        for i in a b c ; do \
          echo $i > import/trunk/subversion/bindings/swig/perl/$i.pm &&
-         echo _$i > import/trunk/subversion/bindings/swig/perl/t/$i.t; \
+         echo _$i > import/trunk/subversion/bindings/swig/perl/t/$i.t || return 1
        done &&
          echo "bad delete test" > \
           import/trunk/subversion/bindings/swig/perl/t/larger-parent &&
@@ -134,7 +134,7 @@ test_expect_success 'follow-parent avoids deleting relevant info' '
                svn mv t native/t &&
                for i in a b c
                do
-                       svn mv $i.pm native/$i.pm
+                       svn mv $i.pm native/$i.pm || return 1
                done &&
                echo z >>native/t/c.t &&
                poke native/t/c.t &&
index b4081fefbace5d6bb3083cdaf1825056b95dce56..90325db909e43c09b13cdf0c731d5e4260566313 100755 (executable)
@@ -15,7 +15,7 @@ EOF
 test_expect_success 'setup svnrepo' '
        for i in aa bb cc dd
        do
-               svn_cmd mkdir -m $i --username $i "$svnrepo"/$i
+               svn_cmd mkdir -m $i --username $i "$svnrepo"/$i || return 1
        done
        '
 
@@ -60,7 +60,7 @@ test_expect_success 'authors-file against globs' '
        for i in bb ee cc
        do
                branch="aa/branches/$i" &&
-               svn_cmd mkdir -m "$branch" --username $i "$svnrepo/$branch"
+               svn_cmd mkdir -m "$branch" --username $i "$svnrepo/$branch" || return 1
        done
        '
 
index fff49c4100852b28899c6694a33acd3b96b188b8..4a77eb9f60da3ade87ed701ecb0dca7f7ad13d87 100755 (executable)
@@ -27,7 +27,7 @@ test_expect_success 'setup test repository' '
 test_expect_success 'clone an SVN repository with ignored www directory' '
        git svn clone --ignore-paths="^www" "$svnrepo" g &&
        echo test_qqq > expect &&
-       for i in g/*/*.txt; do cat $i >> expect2; done &&
+       for i in g/*/*.txt; do cat $i >> expect2 || return 1; done &&
        test_cmp expect expect2
 '
 
@@ -36,7 +36,7 @@ test_expect_success 'init+fetch an SVN repository with ignored www directory' '
        ( cd c && git svn fetch --ignore-paths="^www" ) &&
        rm expect2 &&
        echo test_qqq > expect &&
-       for i in c/*/*.txt; do cat $i >> expect2; done &&
+       for i in c/*/*.txt; do cat $i >> expect2 || return 1; done &&
        test_cmp expect expect2
 '
 
@@ -62,7 +62,7 @@ test_expect_success 'update git svn-cloned repo (config ignore)' '
                cd g &&
                git svn rebase &&
                printf "test_qqq\nb\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -73,7 +73,7 @@ test_expect_success 'update git svn-cloned repo (option ignore)' '
                cd c &&
                git svn rebase --ignore-paths="^www" &&
                printf "test_qqq\nb\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -94,7 +94,7 @@ test_expect_success 'update git svn-cloned repo (config ignore)' '
                cd g &&
                git svn rebase &&
                printf "test_qqq\nb\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -105,7 +105,7 @@ test_expect_success 'update git svn-cloned repo (option ignore)' '
                cd c &&
                git svn rebase --ignore-paths="^www" &&
                printf "test_qqq\nb\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -127,7 +127,7 @@ test_expect_success 'update git svn-cloned repo again (config ignore)' '
                cd g &&
                git svn rebase &&
                printf "test_qqq\nb\nygg\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -138,7 +138,7 @@ test_expect_success 'update git svn-cloned repo again (option ignore)' '
                cd c &&
                git svn rebase --ignore-paths="^www" &&
                printf "test_qqq\nb\nygg\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
index 027b416720ddf774ffd52e9bd3267b009bd70899..784ec7fc2d6e4d22feaad6dfc17120e24856b407 100755 (executable)
@@ -27,7 +27,7 @@ test_expect_success 'svn-authors setup' '
 test_expect_success 'setup svnrepo' '
        for i in aa bb cc-sub dd-sub ee-foo ff
        do
-               svn mkdir -m $i --username $i "$svnrepo"/$i
+               svn mkdir -m $i --username $i "$svnrepo"/$i || return 1
        done
 '
 
index 5f91c0d68b4582958e2ba5e20e888b9369e6a3f4..80cb55fee70e2affd719c0b6812e01f3125ce42e 100755 (executable)
@@ -8,7 +8,7 @@ test_description='git svn creates empty directories'
 test_expect_success 'initialize repo' '
        for i in a b c d d/e d/e/f "weird file name"
        do
-               svn_cmd mkdir -m "mkdir $i" "$svnrepo"/"$i"
+               svn_cmd mkdir -m "mkdir $i" "$svnrepo"/"$i" || return 1
        done
 '
 
@@ -102,7 +102,7 @@ test_expect_success 'git svn mkdirs -r works' '
 test_expect_success 'initialize trunk' '
        for i in trunk trunk/a trunk/"weird file name"
        do
-               svn_cmd mkdir -m "mkdir $i" "$svnrepo"/"$i"
+               svn_cmd mkdir -m "mkdir $i" "$svnrepo"/"$i" || return 1
        done
 '
 
index d292bf9f55cdbedefca4e3e0777dd1f701dea9be..257fc8f2f8d194bbf1a0bb151da1b80d85874e28 100755 (executable)
@@ -28,7 +28,7 @@ test_expect_success 'setup test repository' '
 test_expect_success 'clone an SVN repository with filter to include qqq directory' '
        git svn clone --include-paths="qqq" "$svnrepo" g &&
        echo test_qqq > expect &&
-       for i in g/*/*.txt; do cat $i >> expect2; done &&
+       for i in g/*/*.txt; do cat $i >> expect2 || return 1; done &&
        test_cmp expect expect2
 '
 
@@ -38,7 +38,7 @@ test_expect_success 'init+fetch an SVN repository with included qqq directory' '
        ( cd c && git svn fetch --include-paths="qqq" ) &&
        rm expect2 &&
        echo test_qqq > expect &&
-       for i in c/*/*.txt; do cat $i >> expect2; done &&
+       for i in c/*/*.txt; do cat $i >> expect2 || return 1; done &&
        test_cmp expect expect2
 '
 
@@ -64,7 +64,7 @@ test_expect_success 'update git svn-cloned repo (config include)' '
                cd g &&
                git svn rebase &&
                printf "test_qqq\nb\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -75,7 +75,7 @@ test_expect_success 'update git svn-cloned repo (option include)' '
                cd c &&
                git svn rebase --include-paths="qqq" &&
                printf "test_qqq\nb\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -96,7 +96,7 @@ test_expect_success 'update git svn-cloned repo (config include)' '
                cd g &&
                git svn rebase &&
                printf "test_qqq\nb\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -107,7 +107,7 @@ test_expect_success 'update git svn-cloned repo (option include)' '
                cd c &&
                git svn rebase --include-paths="qqq" &&
                printf "test_qqq\nb\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -129,7 +129,7 @@ test_expect_success 'update git svn-cloned repo again (config include)' '
                cd g &&
                git svn rebase &&
                printf "test_qqq\nb\nygg\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
@@ -140,7 +140,7 @@ test_expect_success 'update git svn-cloned repo again (option include)' '
                cd c &&
                git svn rebase --include-paths="qqq" &&
                printf "test_qqq\nb\nygg\n" > expect &&
-               for i in */*.txt; do cat $i >> expect2; done &&
+               for i in */*.txt; do cat $i >> expect2 || exit 1; done &&
                test_cmp expect2 expect &&
                rm expect expect2
        )
index 89f285d082965c0b8f0d8cbee226f3c8e2a75cfd..a597c42f77de667047c94a02534f11f884995ed7 100755 (executable)
@@ -8,7 +8,7 @@ test_description='git svn creates empty directories, calls git gc, makes sure th
 test_expect_success 'initialize repo' '
        for i in a b c d d/e d/e/f "weird file name"
        do
-               svn_cmd mkdir -m "mkdir $i" "$svnrepo"/"$i"
+               svn_cmd mkdir -m "mkdir $i" "$svnrepo"/"$i" || return 1
        done
 '
 
index d4359dba21c9cff29da7f96541e7c2293da759f3..bed01c99ea704244ace8c189dc428cd432eb6205 100755 (executable)
@@ -16,7 +16,7 @@ test_expect_success 'setup large marks file' '
        blob=$(git rev-parse HEAD:one.t) &&
        for i in $(test_seq 1024 16384)
        do
-               echo ":$i $blob"
+               echo ":$i $blob" || return 1
        done >>marks
 '
 
index 567d7f2bb819254ad24b7735e1d64f77c915b429..210ddf09e3094ebe5fa00599db22106813f4229e 100755 (executable)
@@ -338,7 +338,7 @@ test_expect_success 'cvs update (subdirectories)' \
   '(for dir in A A/B A/B/C A/D E; do
       mkdir $dir &&
       echo "test file in $dir" >"$dir/file_in_$(echo $dir|sed -e "s#/# #g")"  &&
-      git add $dir
+      git add $dir || exit 1
    done) &&
    git commit -q -m "deep sub directory structure" &&
    git push gitcvs.git >/dev/null &&
@@ -381,7 +381,7 @@ test_expect_success 'cvs update (merge)' \
    for i in 1 2 3 4 5 6 7
    do
      echo Line $i >>merge &&
-     echo Line $i >>expected
+     echo Line $i >>expected || return 1
    done &&
    echo Line 8 >>expected &&
    git add merge &&
index 81bc8e8da1aa3feeec001b9f479998d76e899dd8..806005a793a3ed70492d08878d99fae499464ca9 100755 (executable)
@@ -171,7 +171,7 @@ test_expect_success 'clone using non-numeric revision ranges' '
                        cd "$git" &&
                        git ls-files >lines &&
                        test_line_count = 8 lines
-               )
+               ) || return 1
        done
 '
 
index 0db7ab99184add2c500f821659b09cc833c6a225..de591d875c2bbc94fd1c9b093867d23f3ce1b00c 100755 (executable)
@@ -92,11 +92,11 @@ test_expect_success 'Add some more files' '
        for i in $(test_seq 0 10)
        do
                p4_add_file "included/x$i" &&
-               p4_add_file "excluded/x$i"
+               p4_add_file "excluded/x$i" || return 1
        done &&
        for i in $(test_seq 0 10)
        do
-               p4_add_file "excluded/y$i"
+               p4_add_file "excluded/y$i" || return 1
        done
 '
 
@@ -123,7 +123,7 @@ test_expect_success 'Create a repo with multiple depot paths' '
        do
                for i in $(test_seq 1 10)
                do
-                       p4_add_file "$p/file$p$i"
+                       p4_add_file "$p/file$p$i" || return 1
                done
        done
 '
index 518203fbe07399f794651d2bb81878ed0321d922..caa9d2dc9b68647ec65aa625030c60701246fefe 100755 (executable)
@@ -879,7 +879,7 @@ test_expect_success '__git_refs - unique remote branches for git checkout DWIMer
                refs/remotes/remote/branch-in-remote
        do
                git update-ref $remote_ref main &&
-               test_when_finished "git update-ref -d $remote_ref"
+               test_when_finished "git update-ref -d $remote_ref" || return 1
        done &&
        (
                cur= &&
@@ -1052,7 +1052,7 @@ test_expect_success '__git_refs - only matching refs - checkout DWIMery' '
                refs/remotes/remote/branch-in-remote
        do
                git update-ref $remote_ref main &&
-               test_when_finished "git update-ref -d $remote_ref"
+               test_when_finished "git update-ref -d $remote_ref" || return 1
        done &&
        (
                cur=mat &&