Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> wrote: > Refactor a test added in 83c9433e67 (git-svn: support for git-svn > propset, 2014-12-07) to avoid using "set -e" in the test body. This > would have broken in combination with a subsequent change to make > "test_expect_success" return 1 to catch such cases. > > While I'm at it rewrite the test to conform to a modern style in our > tests, using the "test_when_finished" function for the "rm -rf", and > avoiding repeated "mkdir" in favor of "mkdir -p". Thank you for working on these (and sorry for making a mess all those years ago!). I had it in my mind that "mkdir -p" wasn't portable, but systems without it probably haven't been relevant since pre-git times. > --- a/t/t9148-git-svn-propset.sh > +++ b/t/t9148-git-svn-propset.sh > @@ -7,19 +7,22 @@ test_description='git svn propset tests' > > . ./lib-git-svn.sh > > -foo_subdir2="subdir/subdir2/foo_subdir2" > +test_expect_success 'setup propset via import' ' > + test_when_finished "rm -rf import" && > > -set -e > -mkdir import && > -(set -e ; cd import > - mkdir subdir > - mkdir subdir/subdir2 > - touch foo # for 'add props top level' > - touch subdir/foo_subdir # for 'add props relative' > - touch "$foo_subdir2" # for 'add props subdir' > - svn_cmd import -m 'import for git svn' . "$svnrepo" >/dev/null > -) > -rm -rf import > + foo_subdir2="subdir/subdir2/foo_subdir2" && > + mkdir -p import/subdir/subdir2 && > + ( > + cd import && > + # for "add props top level" > + touch foo && > + # for "add props relative" > + touch subdir/foo_subdir && > + # for "add props subdir" > + touch "$foo_subdir2" && > + svn_cmd import -m "import for git svn" . "$svnrepo" I've noticed '>' could be used instead of touch, but that's probably better as another patch :>
