Re: [PATCH v3 2/2] t7501: add tests for --amend --signoff

From: phillip.wood123@gmail.com <phillip.wood123@gmail.com>
Date: Jan 11, 2024, 16:30 UTC
Message-ID: <02cf7ef3-6a16-4fe1-a032-ea7c727c9b12@gmail.com>
In-Reply-To: <20240110163622.51182-6-shyamthakkar001@gmail.com>

On 10/01/2024 16:35, Ghanshyam Thakkar wrote:

Show 10 quoted lines

> Add tests for amending the commit to add Signed-off-by trailer. And
> also to check if it does not add another trailer if one already exists.
> 
> Currently, there are tests for --signoff separately in t7501, however,
> they are not tested with --amend.
> 
> Therefore, these tests belong with other similar tests of --amend in
> t7501-commit-basic-functionality.
> 
> Signed-off-by: Ghanshyam Thakkar <shyamthakkar001@gmail.com>

This version looks good, thanks for re-rolling. I have left one comment below but it is not worth re-rolling just for that.

Show 11 quoted lines

> +test_expect_success 'amend commit to add signoff' '
> +
> +	test_commit "msg" file content &&
> +	git commit --amend --signoff &&
> +	test_commit_message HEAD <<-EOF
> +	msg
> +
> +	Signed-off-by: $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL>
> +	EOF
> +
> +'

If you do happen re-roll I think we could happily lose the empty line before the closing "'" in this test and the next.

Best Wishes

Phillip