# [PATCH] t4002: fix test #63 (missing whitespace) 3 messages from 2023-07-25 to 2023-07-25. Participants: Andreas Herrmann, Eric Sunshine, Junio C Hamano. Thread: https://gitlist.dev/t/60025 ## Andreas Herrmann, 2023-07-25 06:34 Subject: [PATCH] t4002: fix test #63 (missing whitespace) Message-ID: <20230725063516.27242-1-aherrmann@suse.de> URL: https://gitlist.dev/e/20230725063516.27242-1-aherrmann%40suse.de ``` Add missing whitespace between 'test_expect_success' and test case description for last test. Without this, we get: t4002-diff-basic.sh: line 412: test_expect_successdiff can read from stdin: command not found Signed-off-by: Andreas Herrmann --- t/t4002-diff-basic.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/t/t4002-diff-basic.sh b/t/t4002-diff-basic.sh index d524d4057d..e6b6739426 100755 --- a/t/t4002-diff-basic.sh +++ b/t/t4002-diff-basic.sh @@ -401,9 +401,10 @@ test_expect_success 'diff-tree B A == diff-tree -R A B' ' test_expect_success 'diff-tree -r B A == diff-tree -r -R A B' ' git diff-tree -r $tree_B $tree_A >.test-a && git diff-tree -r -R $tree_A $tree_B >.test-b && - cmp -s .test-a .test-b' + cmp -s .test-a .test-b +' -test_expect_success'diff can read from stdin' ' +test_expect_success 'diff can read from stdin' ' test_must_fail git diff --no-index -- MN - < NN | grep -v "^index" | sed "s#/-#/NN#" >.test-a && test_must_fail git diff --no-index -- MN NN | -- 2.41.0 ``` ## Eric Sunshine, 2023-07-25 07:10 Subject: Re: [PATCH] t4002: fix test #63 (missing whitespace) Message-ID: URL: https://gitlist.dev/e/CAPig%2BcS8%3DiJD4fwvSacvWTK0WUH_DyjkQuMVP00aV%2Be4Q4TL3g%40mail.gmail.com In-Reply-To: <20230725063516.27242-1-aherrmann@suse.de> ``` On Tue, Jul 25, 2023 at 2:48 AM Andreas Herrmann wrote: > Add missing whitespace between 'test_expect_success' and test case > description for last test. Without this, we get: > > t4002-diff-basic.sh: line 412: test_expect_successdiff can read from stdin: command not found > > Signed-off-by: Andreas Herrmann Thanks for finding this problem and submitting a fix. You're the second person to do so[1]. [1]: https://lore.kernel.org/git/pull.1545.git.git.1689341410476.gitgitgadget@gmail.com/ > diff --git a/t/t4002-diff-basic.sh b/t/t4002-diff-basic.sh > @@ -401,9 +401,10 @@ test_expect_success 'diff-tree B A == diff-tree -R A B' ' > test_expect_success 'diff-tree -r B A == diff-tree -r -R A B' ' > git diff-tree -r $tree_B $tree_A >.test-a && > git diff-tree -r -R $tree_A $tree_B >.test-b && > - cmp -s .test-a .test-b' > + cmp -s .test-a .test-b > +' This minor style fixup could be resubmitted as a standalone patch, though it might be too minor to worry about. > -test_expect_success'diff can read from stdin' ' > +test_expect_success 'diff can read from stdin' ' > test_must_fail git diff --no-index -- MN - < NN | > grep -v "^index" | sed "s#/-#/NN#" >.test-a && > test_must_fail git diff --no-index -- MN NN | ``` ## Junio C Hamano, 2023-07-25 16:49 Subject: Re: [PATCH] t4002: fix test #63 (missing whitespace) Message-ID: URL: https://gitlist.dev/e/xmqqwmyolywy.fsf%40gitster.g In-Reply-To: ``` Eric Sunshine writes: > Thanks for finding this problem and submitting a fix. You're the > second person to do so[1]. > > [1]: https://lore.kernel.org/git/pull.1545.git.git.1689341410476.gitgitgadget@gmail.com/ > >> diff --git a/t/t4002-diff-basic.sh b/t/t4002-diff-basic.sh >> @@ -401,9 +401,10 @@ test_expect_success 'diff-tree B A == diff-tree -R A B' ' >> test_expect_success 'diff-tree -r B A == diff-tree -r -R A B' ' >> git diff-tree -r $tree_B $tree_A >.test-a && >> git diff-tree -r -R $tree_A $tree_B >.test-b && >> - cmp -s .test-a .test-b' >> + cmp -s .test-a .test-b >> +' > > This minor style fixup could be resubmitted as a standalone patch, > though it might be too minor to worry about. Thanks for a quick review and encouraging words. I do not mind a patch with only the above hunk but of course the title and the log message would have to be updated ;-). ```