diff options
| author |  Jerry Zhang <jerry@skydio.com> | 2022-02-01 20:20:15 -0800 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2022-02-02 11:24:15 -0800 |
| commit | 56fa5ac39a8601d965127abebbc533f29eaef751 (patch) | |
| tree | 9733071d9705d500c5c55f1f0d20969965797d72 /t | |
| parent | Git 2.35.1 (diff) | |
| download | tgif-56fa5ac39a8601d965127abebbc533f29eaef751.tar.xz | |
patch-id: fix antipatterns in tests
Clean up the tests for patch-id by moving file preparation
tasks inside the test body and redirecting files directly into
stdin instead of using 'cat'.
Signed-off-by: Jerry Zhang <jerry@skydio.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
| -rwxr-xr-x | t/t4204-patch-id.sh | 64 |
1 files changed, 31 insertions, 33 deletions
diff --git a/t/t4204-patch-id.sh b/t/t4204-patch-id.sh index 80f4a65b28..2bc940a07e 100755 --- a/t/t4204-patch-id.sh +++ b/t/t4204-patch-id.sh @@ -166,40 +166,38 @@ test_expect_success 'patch-id respects config from subdir' ' ) ' -cat >nonl <<\EOF -diff --git i/a w/a -index e69de29..2e65efe 100644 ---- i/a -+++ w/a -@@ -0,0 +1 @@ -+a -\ No newline at end of file -diff --git i/b w/b -index e69de29..6178079 100644 ---- i/b -+++ w/b -@@ -0,0 +1 @@ -+b -EOF - -cat >withnl <<\EOF -diff --git i/a w/a -index e69de29..7898192 100644 ---- i/a -+++ w/a -@@ -0,0 +1 @@ -+a -diff --git i/b w/b -index e69de29..6178079 100644 ---- i/b -+++ w/b -@@ -0,0 +1 @@ -+b -EOF - test_expect_success 'patch-id handles no-nl-at-eof markers' ' - cat nonl | calc_patch_id nonl && - cat withnl | calc_patch_id withnl && + cat >nonl <<-\EOF && + diff --git i/a w/a + index e69de29..2e65efe 100644 + --- i/a + +++ w/a + @@ -0,0 +1 @@ + +a + \ No newline at end of file + diff --git i/b w/b + index e69de29..6178079 100644 + --- i/b + +++ w/b + @@ -0,0 +1 @@ + +b + EOF + cat >withnl <<-\EOF && + diff --git i/a w/a + index e69de29..7898192 100644 + --- i/a + +++ w/a + @@ -0,0 +1 @@ + +a + diff --git i/b w/b + index e69de29..6178079 100644 + --- i/b + +++ w/b + @@ -0,0 +1 @@ + +b + EOF + calc_patch_id nonl <nonl && + calc_patch_id withnl <withnl && test_cmp patch-id_nonl patch-id_withnl ' test_done |