mirror of
https://github.com/git/git
synced 2024-10-28 19:25:47 +00:00
b5e823594c
After resolving a conflicting patch, a user may wish to sign off the patch to declare that the patch has been modified. As such, the user will expect that running "git am --signoff --continue" will append the signoff to the commit message. However, the --signoff option is only taken into account during the mail-parsing stage. If the --signoff option is set, then the signoff will be appended to the commit message. Since the mail-parsing stage comes before the patch application stage, the --signoff option, if provided on the command-line when resuming, will have no effect at all. We cannot move the append_signoff() call to the patch application stage as the applypatch-msg hook and interactive mode, which run before patch application, may expect the signoff to be there. Fix this by taking note if the user explictly set the --signoff option on the command-line, and append the signoff to the commit message when resuming if so. Signed-off-by: Paul Tan <pyokagan@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
102 lines
2.6 KiB
Bash
Executable file
102 lines
2.6 KiB
Bash
Executable file
#!/bin/sh
|
|
|
|
test_description='git-am command-line options override saved options'
|
|
|
|
. ./test-lib.sh
|
|
. "$TEST_DIRECTORY"/lib-terminal.sh
|
|
|
|
format_patch () {
|
|
git format-patch --stdout -1 "$1" >"$1".eml
|
|
}
|
|
|
|
test_expect_success 'setup' '
|
|
test_commit initial file &&
|
|
test_commit first file &&
|
|
|
|
git checkout initial &&
|
|
git mv file file2 &&
|
|
test_tick &&
|
|
git commit -m renamed-file &&
|
|
git tag renamed-file &&
|
|
|
|
git checkout -b side initial &&
|
|
test_commit side1 file &&
|
|
test_commit side2 file &&
|
|
|
|
format_patch side1 &&
|
|
format_patch side2
|
|
'
|
|
|
|
test_expect_success TTY '--3way overrides --no-3way' '
|
|
rm -fr .git/rebase-apply &&
|
|
git reset --hard &&
|
|
git checkout renamed-file &&
|
|
|
|
# Applying side1 will fail as the file has been renamed.
|
|
test_must_fail git am --no-3way side[12].eml &&
|
|
test_path_is_dir .git/rebase-apply &&
|
|
test_cmp_rev renamed-file HEAD &&
|
|
test -z "$(git ls-files -u)" &&
|
|
|
|
# Applying side1 with am --3way will succeed due to the threeway-merge.
|
|
# Applying side2 will fail as --3way does not apply to it.
|
|
test_must_fail test_terminal git am --3way </dev/zero &&
|
|
test_path_is_dir .git/rebase-apply &&
|
|
test side1 = "$(cat file2)"
|
|
'
|
|
|
|
test_expect_success '--no-quiet overrides --quiet' '
|
|
rm -fr .git/rebase-apply &&
|
|
git reset --hard &&
|
|
git checkout first &&
|
|
|
|
# Applying side1 will be quiet.
|
|
test_must_fail git am --quiet side[123].eml >out &&
|
|
test_path_is_dir .git/rebase-apply &&
|
|
! test_i18ngrep "^Applying: " out &&
|
|
echo side1 >file &&
|
|
git add file &&
|
|
|
|
# Applying side1 will not be quiet.
|
|
# Applying side2 will be quiet.
|
|
git am --no-quiet --continue >out &&
|
|
echo "Applying: side1" >expected &&
|
|
test_i18ncmp expected out
|
|
'
|
|
|
|
test_expect_success '--signoff overrides --no-signoff' '
|
|
rm -fr .git/rebase-apply &&
|
|
git reset --hard &&
|
|
git checkout first &&
|
|
|
|
test_must_fail git am --no-signoff side[12].eml &&
|
|
test_path_is_dir .git/rebase-apply &&
|
|
echo side1 >file &&
|
|
git add file &&
|
|
git am --signoff --continue &&
|
|
|
|
# Applied side1 will be signed off
|
|
echo "Signed-off-by: $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL>" >expected &&
|
|
git cat-file commit HEAD^ | grep "Signed-off-by:" >actual &&
|
|
test_cmp expected actual &&
|
|
|
|
# Applied side2 will not be signed off
|
|
test $(git cat-file commit HEAD | grep -c "Signed-off-by:") -eq 0
|
|
'
|
|
|
|
test_expect_success TTY '--reject overrides --no-reject' '
|
|
rm -fr .git/rebase-apply &&
|
|
git reset --hard &&
|
|
git checkout first &&
|
|
rm -f file.rej &&
|
|
|
|
test_must_fail git am --no-reject side1.eml &&
|
|
test_path_is_dir .git/rebase-apply &&
|
|
test_path_is_missing file.rej &&
|
|
|
|
test_must_fail test_terminal git am --reject </dev/zero &&
|
|
test_path_is_dir .git/rebase-apply &&
|
|
test_path_is_file file.rej
|
|
'
|
|
|
|
test_done
|