Re: [PATCH] blame: use different author name for fake commit generated by --contents
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Apr 21, 2023, 23:34 UTC
- Message-ID
- <xmqqbkjgizcv.fsf@gitster.g>
- In-Reply-To
- <20230421223013.467142-1-jacob.e.keller@intel.com>
Jacob Keller <jacob.e.keller@intel.com> writes:
> +test_expect_success 'blame working copy' ' > + test_when_finished "git restore file" && > + echo "1A quick brown fox jumps over" >file && > + echo "another lazy dog" >> file &&
Lose the SP between ">>" redirection operator and its operand "file".
So, we have "1A quick brown fox jumps over the" and "lazy dog" in :file and HEAD:file, and both of these lines are different in the working tree files as shown above.
> + check_count A 1 "Not Committed Yet" 1
So why do we expect one is attributed to A while the other is attributed to the working tree file? Shouldn't we be expecting both to be attributed to "Not Committed Yet"?
WIth this updated like the attached, 8001, 8002, and 8012 seem to all pass (and without, they all fail).
t/annotate-tests.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git c/t/annotate-tests.sh w/t/annotate-tests.sh index 859693949b..4238ce45f8 100644 --- c/t/annotate-tests.sh +++ w/t/annotate-tests.sh @@ -74,8 +74,8 @@ test_expect_success 'blame 1 author' ' test_expect_success 'blame working copy' ' test_when_finished "git restore file" && - echo "1A quick brown fox jumps over" >file && - echo "another lazy dog" >> file && + echo "11A quick brown fox jumps over the" >file && + echo "lazy dog" >>file && check_count A 1 "Not Committed Yet" 1 '