diff: reject unknown change class given to --diff-filter
[gitweb.git] / t / t0000-basic.sh
index f4e8f43bae5d1929718578a2a589125beb5d30c8..cefe33d6d1976e4017569507a6f76ad89c83e4ff 100755 (executable)
@@ -18,85 +18,212 @@ swapping compression and hashing order, the person who is making the
 modification *should* take notice and update the test vectors here.
 '
 
-################################################################
-# It appears that people try to run tests without building...
-
-../git >/dev/null
-if test $? != 1
-then
-       echo >&2 'You do not seem to have built git yet.'
-       exit 1
-fi
-
 . ./test-lib.sh
 
 ################################################################
 # git init has been done in an empty repository.
 # make sure it is empty.
 
-find .git/objects -type f -print >should-be-empty
-test_expect_success \
-    '.git/objects should be empty after git init in an empty repo.' \
-    'cmp -s /dev/null should-be-empty'
+test_expect_success '.git/objects should be empty after git init in an empty repo' '
+       find .git/objects -type f -print >should-be-empty &&
+       test_line_count = 0 should-be-empty
+'
 
 # also it should have 2 subdirectories; no fan-out anymore, pack, and info.
 # 3 is counting "objects" itself
-find .git/objects -type d -print >full-of-directories
-test_expect_success \
-    '.git/objects should have 3 subdirectories.' \
-    'test $(wc -l < full-of-directories) = 3'
+test_expect_success '.git/objects should have 3 subdirectories' '
+       find .git/objects -type d -print >full-of-directories &&
+       test_line_count = 3 full-of-directories
+'
 
 ################################################################
 # Test harness
 test_expect_success 'success is reported like this' '
-    :
+       :
 '
 test_expect_failure 'pretend we have a known breakage' '
-    false
+       false
 '
 
-test_expect_success 'pretend we have fixed a known breakage (run in sub test-lib)' "
-    mkdir passing-todo &&
-    (cd passing-todo &&
-    cat >passing-todo.sh <<EOF &&
-#!$SHELL_PATH
+run_sub_test_lib_test () {
+       name="$1" descr="$2" # stdin is the body of the test code
+       mkdir "$name" &&
+       (
+               cd "$name" &&
+               cat >"$name.sh" <<-EOF &&
+               #!$SHELL_PATH
+
+               test_description='$descr (run in sub test-lib)
+
+               This is run in a sub test-lib so that we do not get incorrect
+               passing metrics
+               '
+
+               # Point to the t/test-lib.sh, which isn't in ../ as usual
+               . "\$TEST_DIRECTORY"/test-lib.sh
+               EOF
+               cat >>"$name.sh" &&
+               chmod +x "$name.sh" &&
+               export TEST_DIRECTORY &&
+               ./"$name.sh" >out 2>err
+       )
+}
+
+check_sub_test_lib_test () {
+       name="$1" # stdin is the expected output from the test
+       (
+               cd "$name" &&
+               ! test -s err &&
+               sed -e 's/^> //' -e 's/Z$//' >expect &&
+               test_cmp expect out
+       )
+}
+
+test_expect_success 'pretend we have a fully passing test suite' "
+       run_sub_test_lib_test full-pass '3 passing tests' <<-\\EOF &&
+       for i in 1 2 3
+       do
+               test_expect_success \"passing test #\$i\" 'true'
+       done
+       test_done
+       EOF
+       check_sub_test_lib_test full-pass <<-\\EOF
+       > ok 1 - passing test #1
+       > ok 2 - passing test #2
+       > ok 3 - passing test #3
+       > # passed all 3 test(s)
+       > 1..3
+       EOF
+"
 
-test_description='A passing TODO test
+test_expect_success 'pretend we have a partially passing test suite' "
+       test_must_fail run_sub_test_lib_test \
+               partial-pass '2/3 tests passing' <<-\\EOF &&
+       test_expect_success 'passing test #1' 'true'
+       test_expect_success 'failing test #2' 'false'
+       test_expect_success 'passing test #3' 'true'
+       test_done
+       EOF
+       check_sub_test_lib_test partial-pass <<-\\EOF
+       > ok 1 - passing test #1
+       > not ok 2 - failing test #2
+       #       false
+       > ok 3 - passing test #3
+       > # failed 1 among 3 test(s)
+       > 1..3
+       EOF
+"
 
-This is run in a sub test-lib so that we do not get incorrect passing
-metrics
-'
+test_expect_success 'pretend we have a known breakage' "
+       run_sub_test_lib_test failing-todo 'A failing TODO test' <<-\\EOF &&
+       test_expect_success 'passing test' 'true'
+       test_expect_failure 'pretend we have a known breakage' 'false'
+       test_done
+       EOF
+       check_sub_test_lib_test failing-todo <<-\\EOF
+       > ok 1 - passing test
+       > not ok 2 - pretend we have a known breakage # TODO known breakage
+       > # still have 1 known breakage(s)
+       > # passed all remaining 1 test(s)
+       > 1..2
+       EOF
+"
 
-# Point to the t/test-lib.sh, which isn't in ../ as usual
-TEST_DIRECTORY=\"$TEST_DIRECTORY\"
-. \"\$TEST_DIRECTORY\"/test-lib.sh
+test_expect_success 'pretend we have fixed a known breakage' "
+       run_sub_test_lib_test passing-todo 'A passing TODO test' <<-\\EOF &&
+       test_expect_failure 'pretend we have fixed a known breakage' 'true'
+       test_done
+       EOF
+       check_sub_test_lib_test passing-todo <<-\\EOF
+       > ok 1 - pretend we have fixed a known breakage # TODO known breakage vanished
+       > # 1 known breakage(s) vanished; please update test(s)
+       > 1..1
+       EOF
+"
 
-test_expect_failure 'pretend we have fixed a known breakage' '
-    :
-'
+test_expect_success 'pretend we have fixed one of two known breakages (run in sub test-lib)' "
+       run_sub_test_lib_test partially-passing-todos \
+               '2 TODO tests, one passing' <<-\\EOF &&
+       test_expect_failure 'pretend we have a known breakage' 'false'
+       test_expect_success 'pretend we have a passing test' 'true'
+       test_expect_failure 'pretend we have fixed another known breakage' 'true'
+       test_done
+       EOF
+       check_sub_test_lib_test partially-passing-todos <<-\\EOF
+       > not ok 1 - pretend we have a known breakage # TODO known breakage
+       > ok 2 - pretend we have a passing test
+       > ok 3 - pretend we have fixed another known breakage # TODO known breakage vanished
+       > # 1 known breakage(s) vanished; please update test(s)
+       > # still have 1 known breakage(s)
+       > # passed all remaining 1 test(s)
+       > 1..3
+       EOF
+"
 
-test_done
-EOF
-    chmod +x passing-todo.sh &&
-    ./passing-todo.sh >out 2>err &&
-    ! test -s err &&
-sed -e 's/^> //' >expect <<EOF &&
-> ok 1 - pretend we have fixed a known breakage # TODO known breakage
-> # fixed 1 known breakage(s)
-> # passed all 1 test(s)
-> 1..1
-EOF
-    test_cmp expect out)
+test_expect_success 'pretend we have a pass, fail, and known breakage' "
+       test_must_fail run_sub_test_lib_test \
+               mixed-results1 'mixed results #1' <<-\\EOF &&
+       test_expect_success 'passing test' 'true'
+       test_expect_success 'failing test' 'false'
+       test_expect_failure 'pretend we have a known breakage' 'false'
+       test_done
+       EOF
+       check_sub_test_lib_test mixed-results1 <<-\\EOF
+       > ok 1 - passing test
+       > not ok 2 - failing test
+       > #     false
+       > not ok 3 - pretend we have a known breakage # TODO known breakage
+       > # still have 1 known breakage(s)
+       > # failed 1 among remaining 2 test(s)
+       > 1..3
+       EOF
+"
+
+test_expect_success 'pretend we have a mix of all possible results' "
+       test_must_fail run_sub_test_lib_test \
+               mixed-results2 'mixed results #2' <<-\\EOF &&
+       test_expect_success 'passing test' 'true'
+       test_expect_success 'passing test' 'true'
+       test_expect_success 'passing test' 'true'
+       test_expect_success 'passing test' 'true'
+       test_expect_success 'failing test' 'false'
+       test_expect_success 'failing test' 'false'
+       test_expect_success 'failing test' 'false'
+       test_expect_failure 'pretend we have a known breakage' 'false'
+       test_expect_failure 'pretend we have a known breakage' 'false'
+       test_expect_failure 'pretend we have fixed a known breakage' 'true'
+       test_done
+       EOF
+       check_sub_test_lib_test mixed-results2 <<-\\EOF
+       > ok 1 - passing test
+       > ok 2 - passing test
+       > ok 3 - passing test
+       > ok 4 - passing test
+       > not ok 5 - failing test
+       > #     false
+       > not ok 6 - failing test
+       > #     false
+       > not ok 7 - failing test
+       > #     false
+       > not ok 8 - pretend we have a known breakage # TODO known breakage
+       > not ok 9 - pretend we have a known breakage # TODO known breakage
+       > ok 10 - pretend we have fixed a known breakage # TODO known breakage vanished
+       > # 1 known breakage(s) vanished; please update test(s)
+       > # still have 2 known breakage(s)
+       > # failed 3 among remaining 7 test(s)
+       > 1..10
+       EOF
 "
+
 test_set_prereq HAVEIT
 haveit=no
 test_expect_success HAVEIT 'test runs if prerequisite is satisfied' '
-    test_have_prereq HAVEIT &&
-    haveit=yes
+       test_have_prereq HAVEIT &&
+       haveit=yes
 '
 donthaveit=yes
 test_expect_success DONTHAVEIT 'unmet prerequisite causes test to be skipped' '
-    donthaveit=no
+       donthaveit=no
 '
 if test $haveit$donthaveit != yesyes
 then
@@ -107,17 +234,17 @@ fi
 test_set_prereq HAVETHIS
 haveit=no
 test_expect_success HAVETHIS,HAVEIT 'test runs if prerequisites are satisfied' '
-    test_have_prereq HAVEIT &&
-    test_have_prereq HAVETHIS &&
-    haveit=yes
+       test_have_prereq HAVEIT &&
+       test_have_prereq HAVETHIS &&
+       haveit=yes
 '
 donthaveit=yes
 test_expect_success HAVEIT,DONTHAVEIT 'unmet prerequisites causes test to be skipped' '
-    donthaveit=no
+       donthaveit=no
 '
 donthaveiteither=yes
 test_expect_success DONTHAVEIT,HAVEIT 'unmet prerequisites causes test to be skipped' '
-    donthaveiteither=no
+       donthaveiteither=no
 '
 if test $haveit$donthaveit$donthaveiteither != yesyesyes
 then
@@ -125,118 +252,125 @@ then
        exit 1
 fi
 
-clean=no
-test_expect_success 'tests clean up after themselves' '
-    test_when_finished clean=yes
+test_lazy_prereq LAZY_TRUE true
+havetrue=no
+test_expect_success LAZY_TRUE 'test runs if lazy prereq is satisfied' '
+       havetrue=yes
+'
+donthavetrue=yes
+test_expect_success !LAZY_TRUE 'missing lazy prereqs skip tests' '
+       donthavetrue=no
 '
 
-if test $clean != yes
+if test "$havetrue$donthavetrue" != yesyes
 then
-       say "bug in test framework: basic cleanup command does not work reliably"
+       say 'bug in test framework: lazy prerequisites do not work'
        exit 1
 fi
 
-test_expect_success 'tests clean up even on failures' "
-    mkdir failing-cleanup &&
-    (cd failing-cleanup &&
-    cat >failing-cleanup.sh <<EOF &&
-#!$SHELL_PATH
-
-test_description='Failing tests with cleanup commands'
+test_lazy_prereq LAZY_FALSE false
+nothavefalse=no
+test_expect_success !LAZY_FALSE 'negative lazy prereqs checked' '
+       nothavefalse=yes
+'
+havefalse=yes
+test_expect_success LAZY_FALSE 'missing negative lazy prereqs will skip' '
+       havefalse=no
+'
 
-# Point to the t/test-lib.sh, which isn't in ../ as usual
-TEST_DIRECTORY=\"$TEST_DIRECTORY\"
-. \"\$TEST_DIRECTORY\"/test-lib.sh
+if test "$nothavefalse$havefalse" != yesyes
+then
+       say 'bug in test framework: negative lazy prerequisites do not work'
+       exit 1
+fi
 
-test_expect_success 'tests clean up even after a failure' '
-    touch clean-after-failure &&
-    test_when_finished rm clean-after-failure &&
-    (exit 1)
+clean=no
+test_expect_success 'tests clean up after themselves' '
+       test_when_finished clean=yes
 '
 
-test_expect_success 'failure to clean up causes the test to fail' '
-    test_when_finished \"(exit 2)\"
-'
+if test $clean != yes
+then
+       say "bug in test framework: basic cleanup command does not work reliably"
+       exit 1
+fi
 
-test_done
-EOF
-    chmod +x failing-cleanup.sh &&
-    test_must_fail ./failing-cleanup.sh >out 2>err &&
-    ! test -s err &&
-    ! test -f \"trash directory.failing-cleanup/clean-after-failure\" &&
-sed -e 's/Z$//' -e 's/^> //' >expect <<\EOF &&
-> not ok - 1 tests clean up even after a failure
-> #    Z
-> #        touch clean-after-failure &&
-> #        test_when_finished rm clean-after-failure &&
-> #        (exit 1)
-> #    Z
-> not ok - 2 failure to clean up causes the test to fail
-> #    Z
-> #        test_when_finished \"(exit 2)\"
-> #    Z
-> # failed 2 among 2 test(s)
-> 1..2
-EOF
-    test_cmp expect out)
+test_expect_success 'tests clean up even on failures' "
+       test_must_fail run_sub_test_lib_test \
+               failing-cleanup 'Failing tests with cleanup commands' <<-\\EOF &&
+       test_expect_success 'tests clean up even after a failure' '
+               touch clean-after-failure &&
+               test_when_finished rm clean-after-failure &&
+               (exit 1)
+       '
+       test_expect_success 'failure to clean up causes the test to fail' '
+               test_when_finished \"(exit 2)\"
+       '
+       test_done
+       EOF
+       check_sub_test_lib_test failing-cleanup <<-\\EOF
+       > not ok 1 - tests clean up even after a failure
+       > #     Z
+       > #     touch clean-after-failure &&
+       > #     test_when_finished rm clean-after-failure &&
+       > #     (exit 1)
+       > #     Z
+       > not ok 2 - failure to clean up causes the test to fail
+       > #     Z
+       > #     test_when_finished \"(exit 2)\"
+       > #     Z
+       > # failed 2 among 2 test(s)
+       > 1..2
+       EOF
 "
 
 ################################################################
 # Basics of the basics
 
 # updating a new file without --add should fail.
-test_expect_success 'git update-index without --add should fail adding.' '
-    test_must_fail git update-index should-be-empty
+test_expect_success 'git update-index without --add should fail adding' '
+       test_must_fail git update-index should-be-empty
 '
 
 # and with --add it should succeed, even if it is empty (it used to fail).
-test_expect_success \
-    'git update-index with --add should succeed.' \
-    'git update-index --add should-be-empty'
+test_expect_success 'git update-index with --add should succeed' '
+       git update-index --add should-be-empty
+'
 
-test_expect_success \
-    'writing tree out with git write-tree' \
-    'tree=$(git write-tree)'
+test_expect_success 'writing tree out with git write-tree' '
+       tree=$(git write-tree)
+'
 
 # we know the shape and contents of the tree and know the object ID for it.
-test_expect_success \
-    'validate object ID of a known tree.' \
-    'test "$tree" = 7bb943559a305bdd6bdee2cef6e5df2413c3d30a'
+test_expect_success 'validate object ID of a known tree' '
+       test "$tree" = 7bb943559a305bdd6bdee2cef6e5df2413c3d30a
+    '
 
 # Removing paths.
-rm -f should-be-empty full-of-directories
-test_expect_success 'git update-index without --remove should fail removing.' '
-    test_must_fail git update-index should-be-empty
+test_expect_success 'git update-index without --remove should fail removing' '
+       rm -f should-be-empty full-of-directories &&
+       test_must_fail git update-index should-be-empty
 '
 
-test_expect_success \
-    'git update-index with --remove should be able to remove.' \
-    'git update-index --remove should-be-empty'
+test_expect_success 'git update-index with --remove should be able to remove' '
+       git update-index --remove should-be-empty
+'
 
 # Empty tree can be written with recent write-tree.
-test_expect_success \
-    'git write-tree should be able to write an empty tree.' \
-    'tree=$(git write-tree)'
+test_expect_success 'git write-tree should be able to write an empty tree' '
+       tree=$(git write-tree)
+'
 
-test_expect_success \
-    'validate object ID of a known tree.' \
-    'test "$tree" = 4b825dc642cb6eb9a060e54bf8d69288fbee4904'
+test_expect_success 'validate object ID of a known tree' '
+       test "$tree" = 4b825dc642cb6eb9a060e54bf8d69288fbee4904
+'
 
 # Various types of objects
+
 # Some filesystems do not support symblic links; on such systems
 # some expected values are different
-mkdir path2 path3 path3/subp3
-paths='path0 path2/file2 path3/file3 path3/subp3/file3'
-for p in $paths
-do
-    echo "hello $p" >$p
-done
 if test_have_prereq SYMLINKS
 then
-       for p in $paths
-       do
-               ln -s "hello $p" ${p}sym
-       done
        expectfilter=cat
        expectedtree=087704a96baf1c2d1c869a8b084481e121c88b5b
        expectedptree1=21ae8269cacbe57ae09138dcc3a2887f904d02b3
@@ -248,135 +382,154 @@ else
        expectedptree2=ce580448f0148b985a513b693fdf7d802cacb44f
 fi
 
-test_expect_success \
-    'adding various types of objects with git update-index --add.' \
-    'find path* ! -type d -print | xargs git update-index --add'
+
+test_expect_success 'adding various types of objects with git update-index --add' '
+       mkdir path2 path3 path3/subp3 &&
+       paths="path0 path2/file2 path3/file3 path3/subp3/file3" &&
+       (
+               for p in $paths
+               do
+                       echo "hello $p" >$p || exit 1
+                       if test_have_prereq SYMLINKS
+                       then
+                               ln -s "hello $p" ${p}sym || exit 1
+                       fi
+               done
+       ) &&
+       find path* ! -type d -print | xargs git update-index --add
+'
 
 # Show them and see that matches what we expect.
-test_expect_success \
-    'showing stage with git ls-files --stage' \
-    'git ls-files --stage >current'
-
-$expectfilter >expected <<\EOF
-100644 f87290f8eb2cbbea7857214459a0739927eab154 0      path0
-120000 15a98433ae33114b085f3eb3bb03b832b3180a01 0      path0sym
-100644 3feff949ed00a62d9f7af97c15cd8a30595e7ac7 0      path2/file2
-120000 d8ce161addc5173867a3c3c730924388daedbc38 0      path2/file2sym
-100644 0aa34cae68d0878578ad119c86ca2b5ed5b28376 0      path3/file3
-120000 8599103969b43aff7e430efea79ca4636466794f 0      path3/file3sym
-100644 00fb5908cb97c2564a9783c0c64087333b3b464f 0      path3/subp3/file3
-120000 6649a1ebe9e9f1c553b66f5a6e74136a07ccc57c 0      path3/subp3/file3sym
-EOF
-test_expect_success \
-    'validate git ls-files output for a known tree.' \
-    'test_cmp expected current'
-
-test_expect_success \
-    'writing tree out with git write-tree.' \
-    'tree=$(git write-tree)'
-test_expect_success \
-    'validate object ID for a known tree.' \
-    'test "$tree" = "$expectedtree"'
-
-test_expect_success \
-    'showing tree with git ls-tree' \
-    'git ls-tree $tree >current'
-cat >expected <<\EOF
-100644 blob f87290f8eb2cbbea7857214459a0739927eab154   path0
-120000 blob 15a98433ae33114b085f3eb3bb03b832b3180a01   path0sym
-040000 tree 58a09c23e2ca152193f2786e06986b7b6712bdbe   path2
-040000 tree 21ae8269cacbe57ae09138dcc3a2887f904d02b3   path3
-EOF
-test_expect_success SYMLINKS \
-    'git ls-tree output for a known tree.' \
-    'test_cmp expected current'
+test_expect_success 'showing stage with git ls-files --stage' '
+       git ls-files --stage >current
+'
+
+test_expect_success 'validate git ls-files output for a known tree' '
+       $expectfilter >expected <<-\EOF &&
+       100644 f87290f8eb2cbbea7857214459a0739927eab154 0       path0
+       120000 15a98433ae33114b085f3eb3bb03b832b3180a01 0       path0sym
+       100644 3feff949ed00a62d9f7af97c15cd8a30595e7ac7 0       path2/file2
+       120000 d8ce161addc5173867a3c3c730924388daedbc38 0       path2/file2sym
+       100644 0aa34cae68d0878578ad119c86ca2b5ed5b28376 0       path3/file3
+       120000 8599103969b43aff7e430efea79ca4636466794f 0       path3/file3sym
+       100644 00fb5908cb97c2564a9783c0c64087333b3b464f 0       path3/subp3/file3
+       120000 6649a1ebe9e9f1c553b66f5a6e74136a07ccc57c 0       path3/subp3/file3sym
+       EOF
+       test_cmp expected current
+'
+
+test_expect_success 'writing tree out with git write-tree' '
+       tree=$(git write-tree)
+'
+
+test_expect_success 'validate object ID for a known tree' '
+       test "$tree" = "$expectedtree"
+'
+
+test_expect_success 'showing tree with git ls-tree' '
+    git ls-tree $tree >current
+'
+
+test_expect_success SYMLINKS 'git ls-tree output for a known tree' '
+       cat >expected <<-\EOF &&
+       100644 blob f87290f8eb2cbbea7857214459a0739927eab154    path0
+       120000 blob 15a98433ae33114b085f3eb3bb03b832b3180a01    path0sym
+       040000 tree 58a09c23e2ca152193f2786e06986b7b6712bdbe    path2
+       040000 tree 21ae8269cacbe57ae09138dcc3a2887f904d02b3    path3
+       EOF
+       test_cmp expected current
+'
 
 # This changed in ls-tree pathspec change -- recursive does
 # not show tree nodes anymore.
-test_expect_success \
-    'showing tree with git ls-tree -r' \
-    'git ls-tree -r $tree >current'
-$expectfilter >expected <<\EOF
-100644 blob f87290f8eb2cbbea7857214459a0739927eab154   path0
-120000 blob 15a98433ae33114b085f3eb3bb03b832b3180a01   path0sym
-100644 blob 3feff949ed00a62d9f7af97c15cd8a30595e7ac7   path2/file2
-120000 blob d8ce161addc5173867a3c3c730924388daedbc38   path2/file2sym
-100644 blob 0aa34cae68d0878578ad119c86ca2b5ed5b28376   path3/file3
-120000 blob 8599103969b43aff7e430efea79ca4636466794f   path3/file3sym
-100644 blob 00fb5908cb97c2564a9783c0c64087333b3b464f   path3/subp3/file3
-120000 blob 6649a1ebe9e9f1c553b66f5a6e74136a07ccc57c   path3/subp3/file3sym
-EOF
-test_expect_success \
-    'git ls-tree -r output for a known tree.' \
-    'test_cmp expected current'
+test_expect_success 'showing tree with git ls-tree -r' '
+       git ls-tree -r $tree >current
+'
+
+test_expect_success 'git ls-tree -r output for a known tree' '
+       $expectfilter >expected <<-\EOF &&
+       100644 blob f87290f8eb2cbbea7857214459a0739927eab154    path0
+       120000 blob 15a98433ae33114b085f3eb3bb03b832b3180a01    path0sym
+       100644 blob 3feff949ed00a62d9f7af97c15cd8a30595e7ac7    path2/file2
+       120000 blob d8ce161addc5173867a3c3c730924388daedbc38    path2/file2sym
+       100644 blob 0aa34cae68d0878578ad119c86ca2b5ed5b28376    path3/file3
+       120000 blob 8599103969b43aff7e430efea79ca4636466794f    path3/file3sym
+       100644 blob 00fb5908cb97c2564a9783c0c64087333b3b464f    path3/subp3/file3
+       120000 blob 6649a1ebe9e9f1c553b66f5a6e74136a07ccc57c    path3/subp3/file3sym
+       EOF
+       test_cmp expected current
+'
 
 # But with -r -t we can have both.
-test_expect_success \
-    'showing tree with git ls-tree -r -t' \
-    'git ls-tree -r -t $tree >current'
-cat >expected <<\EOF
-100644 blob f87290f8eb2cbbea7857214459a0739927eab154   path0
-120000 blob 15a98433ae33114b085f3eb3bb03b832b3180a01   path0sym
-040000 tree 58a09c23e2ca152193f2786e06986b7b6712bdbe   path2
-100644 blob 3feff949ed00a62d9f7af97c15cd8a30595e7ac7   path2/file2
-120000 blob d8ce161addc5173867a3c3c730924388daedbc38   path2/file2sym
-040000 tree 21ae8269cacbe57ae09138dcc3a2887f904d02b3   path3
-100644 blob 0aa34cae68d0878578ad119c86ca2b5ed5b28376   path3/file3
-120000 blob 8599103969b43aff7e430efea79ca4636466794f   path3/file3sym
-040000 tree 3c5e5399f3a333eddecce7a9b9465b63f65f51e2   path3/subp3
-100644 blob 00fb5908cb97c2564a9783c0c64087333b3b464f   path3/subp3/file3
-120000 blob 6649a1ebe9e9f1c553b66f5a6e74136a07ccc57c   path3/subp3/file3sym
-EOF
-test_expect_success SYMLINKS \
-    'git ls-tree -r output for a known tree.' \
-    'test_cmp expected current'
-
-test_expect_success \
-    'writing partial tree out with git write-tree --prefix.' \
-    'ptree=$(git write-tree --prefix=path3)'
-test_expect_success \
-    'validate object ID for a known tree.' \
-    'test "$ptree" = "$expectedptree1"'
-
-test_expect_success \
-    'writing partial tree out with git write-tree --prefix.' \
-    'ptree=$(git write-tree --prefix=path3/subp3)'
-test_expect_success \
-    'validate object ID for a known tree.' \
-    'test "$ptree" = "$expectedptree2"'
-
-cat >badobjects <<EOF
-100644 blob 1000000000000000000000000000000000000000   dir/file1
-100644 blob 2000000000000000000000000000000000000000   dir/file2
-100644 blob 3000000000000000000000000000000000000000   dir/file3
-100644 blob 4000000000000000000000000000000000000000   dir/file4
-100644 blob 5000000000000000000000000000000000000000   dir/file5
-EOF
+test_expect_success 'showing tree with git ls-tree -r -t' '
+       git ls-tree -r -t $tree >current
+'
+
+test_expect_success SYMLINKS 'git ls-tree -r output for a known tree' '
+       cat >expected <<-\EOF &&
+       100644 blob f87290f8eb2cbbea7857214459a0739927eab154    path0
+       120000 blob 15a98433ae33114b085f3eb3bb03b832b3180a01    path0sym
+       040000 tree 58a09c23e2ca152193f2786e06986b7b6712bdbe    path2
+       100644 blob 3feff949ed00a62d9f7af97c15cd8a30595e7ac7    path2/file2
+       120000 blob d8ce161addc5173867a3c3c730924388daedbc38    path2/file2sym
+       040000 tree 21ae8269cacbe57ae09138dcc3a2887f904d02b3    path3
+       100644 blob 0aa34cae68d0878578ad119c86ca2b5ed5b28376    path3/file3
+       120000 blob 8599103969b43aff7e430efea79ca4636466794f    path3/file3sym
+       040000 tree 3c5e5399f3a333eddecce7a9b9465b63f65f51e2    path3/subp3
+       100644 blob 00fb5908cb97c2564a9783c0c64087333b3b464f    path3/subp3/file3
+       120000 blob 6649a1ebe9e9f1c553b66f5a6e74136a07ccc57c    path3/subp3/file3sym
+       EOF
+       test_cmp expected current
+'
 
-rm .git/index
-test_expect_success \
-    'put invalid objects into the index.' \
-    'git update-index --index-info < badobjects'
+test_expect_success 'writing partial tree out with git write-tree --prefix' '
+       ptree=$(git write-tree --prefix=path3)
+'
 
-test_expect_success 'writing this tree without --missing-ok.' '
-    test_must_fail git write-tree
+test_expect_success 'validate object ID for a known tree' '
+       test "$ptree" = "$expectedptree1"
 '
 
-test_expect_success \
-    'writing this tree with --missing-ok.' \
-    'git write-tree --missing-ok'
+test_expect_success 'writing partial tree out with git write-tree --prefix' '
+       ptree=$(git write-tree --prefix=path3/subp3)
+'
+
+test_expect_success 'validate object ID for a known tree' '
+       test "$ptree" = "$expectedptree2"
+'
+
+test_expect_success 'put invalid objects into the index' '
+       rm -f .git/index &&
+       cat >badobjects <<-\EOF &&
+       100644 blob 1000000000000000000000000000000000000000    dir/file1
+       100644 blob 2000000000000000000000000000000000000000    dir/file2
+       100644 blob 3000000000000000000000000000000000000000    dir/file3
+       100644 blob 4000000000000000000000000000000000000000    dir/file4
+       100644 blob 5000000000000000000000000000000000000000    dir/file5
+       EOF
+       git update-index --index-info <badobjects
+'
+
+test_expect_success 'writing this tree without --missing-ok' '
+       test_must_fail git write-tree
+'
+
+test_expect_success 'writing this tree with --missing-ok' '
+       git write-tree --missing-ok
+'
 
 
 ################################################################
-rm .git/index
-test_expect_success \
-    'git read-tree followed by write-tree should be idempotent.' \
-    'git read-tree $tree &&
-     test -f .git/index &&
-     newtree=$(git write-tree) &&
-     test "$newtree" = "$tree"'
-
-$expectfilter >expected <<\EOF
+test_expect_success 'git read-tree followed by write-tree should be idempotent' '
+       rm -f .git/index
+       git read-tree $tree &&
+       test -f .git/index &&
+       newtree=$(git write-tree) &&
+       test "$newtree" = "$tree"
+'
+
+test_expect_success 'validate git diff-files output for a know cache/work tree state' '
+       $expectfilter >expected <<\EOF &&
 :100644 100644 f87290f8eb2cbbea7857214459a0739927eab154 0000000000000000000000000000000000000000 M     path0
 :120000 120000 15a98433ae33114b085f3eb3bb03b832b3180a01 0000000000000000000000000000000000000000 M     path0sym
 :100644 100644 3feff949ed00a62d9f7af97c15cd8a30595e7ac7 0000000000000000000000000000000000000000 M     path2/file2
@@ -386,45 +539,47 @@ $expectfilter >expected <<\EOF
 :100644 100644 00fb5908cb97c2564a9783c0c64087333b3b464f 0000000000000000000000000000000000000000 M     path3/subp3/file3
 :120000 120000 6649a1ebe9e9f1c553b66f5a6e74136a07ccc57c 0000000000000000000000000000000000000000 M     path3/subp3/file3sym
 EOF
-test_expect_success \
-    'validate git diff-files output for a know cache/work tree state.' \
-    'git diff-files >current && test_cmp current expected >/dev/null'
+       git diff-files >current &&
+       test_cmp current expected
+'
 
-test_expect_success \
-    'git update-index --refresh should succeed.' \
-    'git update-index --refresh'
+test_expect_success 'git update-index --refresh should succeed' '
+       git update-index --refresh
+'
 
-test_expect_success \
-    'no diff after checkout and git update-index --refresh.' \
-    'git diff-files >current && cmp -s current /dev/null'
+test_expect_success 'no diff after checkout and git update-index --refresh' '
+       git diff-files >current &&
+       cmp -s current /dev/null
+'
 
 ################################################################
 P=$expectedtree
-test_expect_success \
-    'git commit-tree records the correct tree in a commit.' \
-    'commit0=$(echo NO | git commit-tree $P) &&
-     tree=$(git show --pretty=raw $commit0 |
-        sed -n -e "s/^tree //p" -e "/^author /q") &&
-     test "z$tree" = "z$P"'
-
-test_expect_success \
-    'git commit-tree records the correct parent in a commit.' \
-    'commit1=$(echo NO | git commit-tree $P -p $commit0) &&
-     parent=$(git show --pretty=raw $commit1 |
-        sed -n -e "s/^parent //p" -e "/^author /q") &&
-     test "z$commit0" = "z$parent"'
-
-test_expect_success \
-    'git commit-tree omits duplicated parent in a commit.' \
-    'commit2=$(echo NO | git commit-tree $P -p $commit0 -p $commit0) &&
-     parent=$(git show --pretty=raw $commit2 |
-        sed -n -e "s/^parent //p" -e "/^author /q" |
-        sort -u) &&
-     test "z$commit0" = "z$parent" &&
-     numparent=$(git show --pretty=raw $commit2 |
-        sed -n -e "s/^parent //p" -e "/^author /q" |
-        wc -l) &&
-     test $numparent = 1'
+
+test_expect_success 'git commit-tree records the correct tree in a commit' '
+       commit0=$(echo NO | git commit-tree $P) &&
+       tree=$(git show --pretty=raw $commit0 |
+                sed -n -e "s/^tree //p" -e "/^author /q") &&
+       test "z$tree" = "z$P"
+'
+
+test_expect_success 'git commit-tree records the correct parent in a commit' '
+       commit1=$(echo NO | git commit-tree $P -p $commit0) &&
+       parent=$(git show --pretty=raw $commit1 |
+               sed -n -e "s/^parent //p" -e "/^author /q") &&
+       test "z$commit0" = "z$parent"
+'
+
+test_expect_success 'git commit-tree omits duplicated parent in a commit' '
+       commit2=$(echo NO | git commit-tree $P -p $commit0 -p $commit0) &&
+            parent=$(git show --pretty=raw $commit2 |
+               sed -n -e "s/^parent //p" -e "/^author /q" |
+               sort -u) &&
+       test "z$commit0" = "z$parent" &&
+       numparent=$(git show --pretty=raw $commit2 |
+               sed -n -e "s/^parent //p" -e "/^author /q" |
+               wc -l) &&
+       test $numparent = 1
+'
 
 test_expect_success 'update-index D/F conflict' '
        mv path0 tmp &&
@@ -435,24 +590,6 @@ test_expect_success 'update-index D/F conflict' '
        test $numpath0 = 1
 '
 
-test_expect_success SYMLINKS 'real path works as expected' '
-       mkdir first &&
-       ln -s ../.git first/.git &&
-       mkdir second &&
-       ln -s ../first second/other &&
-       mkdir third &&
-       dir="$(cd .git; pwd -P)" &&
-       dir2=third/../second/other/.git &&
-       test "$dir" = "$(test-path-utils real_path $dir2)" &&
-       file="$dir"/index &&
-       test "$file" = "$(test-path-utils real_path $dir2/index)" &&
-       basename=blub &&
-       test "$dir/$basename" = "$(cd .git && test-path-utils real_path "$basename")" &&
-       ln -s ../first/file .git/syml &&
-       sym="$(cd first; pwd -P)"/file &&
-       test "$sym" = "$(test-path-utils real_path "$dir2/syml")"
-'
-
 test_expect_success 'very long name in the index handled sanely' '
 
        a=a && # 1