t1100: move creation of expected output into setup test
The "expected" file is created at the top-level of the script, outside of any test. Code that runs outside of a test is not protected by the test harness: a failure there is not reported as a test failure and is easy to miss. Move the here-doc that creates "expected" into the existing setup test ("test preparation: write empty tree"), using a "<<-" here-doc so its body can be indented along with the rest of the test. Signed-off-by: Shlok Kulshreshtha <diy2903@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Shlok Kulshreshtha committed
Jul 14, 2026 at 17:50 UTC
9719c290ee5926487caacc1ff059c043323ed183
1 file changed
+7
-8
t/t1100-commit-tree-options.sh
+7
-8
@@ -14,15 +14,14 @@ Also make sure that command line parser understands the normal
14
15
. ./test-lib.sh
16
17
-cat >expected <<EOF
18
-tree $EMPTY_TREE
19
-author Author Name <author@email> 1117148400 +0000
20
-committer Committer Name <committer@email> 1117150200 +0000
21
-
22
-comment text
23
-EOF
24
-
17
test_expect_success 'test preparation: write empty tree' '
18
+ cat >expected <<-EOF &&
19
+ tree $EMPTY_TREE
20
+ author Author Name <author@email> 1117148400 +0000
21
+ committer Committer Name <committer@email> 1117150200 +0000
22
+
23
+ comment text
24
+ EOF
25
git write-tree >treeid
26
'
27