pickaxe tests: refactor to use test_commit --append --printf

Refactor the existing tests added in e0e7cb8080c (log -G: ignore
binary files, 2018-12-14) to use the --append option I added in
3373518cc8b (test-lib functions: add an --append option to
test_commit, 2021-01-12) and the --printf option added as part of an
in-flight topic of mine this commit depends on.

While I'm at it change some of the setup of the test to use a more
sensible pattern, e.g. setting up a temporary repo instead of creating
an orphan branch.

Since the -G and -S options will behave the same way with truncated
and removed content also change the "git rm" to emptying data.bin,
that's just catering to how test_commit works. The resulting test is
shorter.

See also f5d79bf7dd6 (tests: refactor a few tests to use "test_commit
--append", 2021-01-12) for prior similar refactoring.

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/t/t4209-log-pickaxe.sh b/t/t4209-log-pickaxe.sh
index 5d06f5f45eace6010c1a003e03a814b5ba03f324..ad45d8cfd0a426ca63fad1561c76ca9c66f9b481 100755 (executable)
--- a/t/t4209-log-pickaxe.sh
+++ b/t/t4209-log-pickaxe.sh
@@ -107,37 +107,35 @@ test_expect_success 'log -S --no-textconv (missing textconv tool)' '
 '
 
 test_expect_success 'setup log -[GS] binary & --text' '
-       git checkout --orphan GS-binary-and-text &&
-       git read-tree --empty &&
-       printf "a\na\0a\n" >data.bin &&
-       git add data.bin &&
-       git commit -m "create binary file" data.bin &&
-       printf "a\na\0a\n" >>data.bin &&
-       git commit -m "modify binary file" data.bin &&
-       git rm data.bin &&
-       git commit -m "delete binary file" data.bin &&
-       git log >full-log
+       test_create_repo GS-bin-txt &&
+       test_commit -C GS-bin-txt --printf A data.bin "a\na\0a\n" &&
+       test_commit -C GS-bin-txt --append --printf B data.bin "a\na\0a\n" &&
+       test_commit -C GS-bin-txt C data.bin "" &&
+       git -C GS-bin-txt log >full-log
 '
 
 test_expect_success 'log -G ignores binary files' '
-       git log -Ga >log &&
+       git -C GS-bin-txt log -Ga >log &&
        test_must_be_empty log
 '
 
 test_expect_success 'log -G looks into binary files with -a' '
-       git log -a -Ga >log &&
+       git -C GS-bin-txt log -a -Ga >log &&
        test_cmp log full-log
 '
 
 test_expect_success 'log -G looks into binary files with textconv filter' '
-       test_when_finished "rm .gitattributes" &&
-       echo "* diff=bin" >.gitattributes &&
-       git -c diff.bin.textconv=cat log -Ga >log &&
+       test_when_finished "rm GS-bin-txt/.gitattributes" &&
+       (
+               cd GS-bin-txt &&
+               echo "* diff=bin" >.gitattributes &&
+               git -c diff.bin.textconv=cat log -Ga >../log
+       ) &&
        test_cmp log full-log
 '
 
 test_expect_success 'log -S looks into binary files' '
-       git log -Sa >log &&
+       git -C GS-bin-txt log -Sa >log &&
        test_cmp log full-log
 '