430b75f720
In dcb500dc16 (cherry-pick/revert: advise using --skip, 2019-07-02),
`git commit` learned to suggest to run `git cherry-pick --skip` when
trying to cherry-pick an empty patch.
However, it was overlooked that there are more conditions than just a
`git cherry-pick` when this advice is printed (which originally
suggested the neutral `git reset`): the same can happen during a rebase.
Let's suggest the correct command, even during a rebase.
While at it, we adjust more places in `builtin/commit.c` that
incorrectly assumed that the presence of a `CHERRY_PICK_HEAD` meant that
surely this must be a `cherry-pick` in progress.
Note: we take pains to handle the situation when a user runs a `git
cherry-pick` _during_ a rebase. This is quite valid (e.g. in an `exec`
line in an interactive rebase). On the other hand, it is not possible to
run a rebase during a cherry-pick, meaning: if both `rebase-merge/` and
`sequencer/` exist or CHERRY_PICK_HEAD and REBASE_HEAD point to the same
commit , we still want to advise to use `git cherry-pick --skip`.
Original-patch-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Phillip Wood <phillip.wood@dunelm.org.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
191 lines
5.3 KiB
Bash
Executable File
191 lines
5.3 KiB
Bash
Executable File
#!/bin/sh
|
|
#
|
|
# Copyright (c) 2006 Eric Wong
|
|
#
|
|
|
|
test_description='git rebase --merge --skip tests'
|
|
|
|
. ./test-lib.sh
|
|
|
|
. "$TEST_DIRECTORY"/lib-rebase.sh
|
|
|
|
# we assume the default git am -3 --skip strategy is tested independently
|
|
# and always works :)
|
|
|
|
test_expect_success setup '
|
|
echo hello > hello &&
|
|
git add hello &&
|
|
git commit -m "hello" &&
|
|
git branch skip-reference &&
|
|
|
|
echo world >> hello &&
|
|
git commit -a -m "hello world" &&
|
|
echo goodbye >> hello &&
|
|
git commit -a -m "goodbye" &&
|
|
git tag goodbye &&
|
|
|
|
git checkout --detach &&
|
|
git checkout HEAD^ . &&
|
|
test_tick &&
|
|
git commit -m reverted-goodbye &&
|
|
git tag reverted-goodbye &&
|
|
git checkout goodbye &&
|
|
test_tick &&
|
|
GIT_AUTHOR_NAME="Another Author" \
|
|
GIT_AUTHOR_EMAIL="another.author@example.com" \
|
|
git commit --amend --no-edit -m amended-goodbye &&
|
|
test_tick &&
|
|
git tag amended-goodbye &&
|
|
|
|
git checkout -f skip-reference &&
|
|
echo moo > hello &&
|
|
git commit -a -m "we should skip this" &&
|
|
echo moo > cow &&
|
|
git add cow &&
|
|
git commit -m "this should not be skipped" &&
|
|
git branch pre-rebase skip-reference &&
|
|
git branch skip-merge skip-reference
|
|
'
|
|
|
|
test_expect_success 'rebase with git am -3 (default)' '
|
|
test_must_fail git rebase master
|
|
'
|
|
|
|
test_expect_success 'rebase --skip can not be used with other options' '
|
|
test_must_fail git rebase -v --skip &&
|
|
test_must_fail git rebase --skip -v
|
|
'
|
|
|
|
test_expect_success 'rebase --skip with am -3' '
|
|
git rebase --skip
|
|
'
|
|
|
|
test_expect_success 'rebase moves back to skip-reference' '
|
|
test refs/heads/skip-reference = $(git symbolic-ref HEAD) &&
|
|
git branch post-rebase &&
|
|
git reset --hard pre-rebase &&
|
|
test_must_fail git rebase master &&
|
|
echo "hello" > hello &&
|
|
git add hello &&
|
|
git rebase --continue &&
|
|
test refs/heads/skip-reference = $(git symbolic-ref HEAD) &&
|
|
git reset --hard post-rebase
|
|
'
|
|
|
|
test_expect_success 'checkout skip-merge' 'git checkout -f skip-merge'
|
|
|
|
test_expect_success 'rebase with --merge' '
|
|
test_must_fail git rebase --merge master
|
|
'
|
|
|
|
test_expect_success 'rebase --skip with --merge' '
|
|
git rebase --skip
|
|
'
|
|
|
|
test_expect_success 'merge and reference trees equal' '
|
|
test -z "$(git diff-tree skip-merge skip-reference)"
|
|
'
|
|
|
|
test_expect_success 'moved back to branch correctly' '
|
|
test refs/heads/skip-merge = $(git symbolic-ref HEAD)
|
|
'
|
|
|
|
test_debug 'gitk --all & sleep 1'
|
|
|
|
test_expect_success 'correct advice upon picking empty commit' '
|
|
test_when_finished "git rebase --abort" &&
|
|
test_must_fail git rebase -i --onto goodbye \
|
|
amended-goodbye^ amended-goodbye 2>err &&
|
|
test_i18ngrep "previous cherry-pick is now empty" err &&
|
|
test_i18ngrep "git rebase --skip" err &&
|
|
test_must_fail git commit &&
|
|
test_i18ngrep "git rebase --skip" err
|
|
'
|
|
|
|
test_expect_success 'correct authorship when committing empty pick' '
|
|
test_when_finished "git rebase --abort" &&
|
|
test_must_fail git rebase -i --onto goodbye \
|
|
amended-goodbye^ amended-goodbye &&
|
|
git commit --allow-empty &&
|
|
git log --pretty=format:"%an <%ae>%n%ad%B" -1 amended-goodbye >expect &&
|
|
git log --pretty=format:"%an <%ae>%n%ad%B" -1 HEAD >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success 'correct advice upon rewording empty commit' '
|
|
test_when_finished "git rebase --abort" &&
|
|
(
|
|
set_fake_editor &&
|
|
test_must_fail env FAKE_LINES="reword 1" git rebase -i \
|
|
--onto goodbye amended-goodbye^ amended-goodbye 2>err
|
|
) &&
|
|
test_i18ngrep "previous cherry-pick is now empty" err &&
|
|
test_i18ngrep "git rebase --skip" err &&
|
|
test_must_fail git commit &&
|
|
test_i18ngrep "git rebase --skip" err
|
|
'
|
|
|
|
test_expect_success 'correct advice upon editing empty commit' '
|
|
test_when_finished "git rebase --abort" &&
|
|
(
|
|
set_fake_editor &&
|
|
test_must_fail env FAKE_LINES="edit 1" git rebase -i \
|
|
--onto goodbye amended-goodbye^ amended-goodbye 2>err
|
|
) &&
|
|
test_i18ngrep "previous cherry-pick is now empty" err &&
|
|
test_i18ngrep "git rebase --skip" err &&
|
|
test_must_fail git commit &&
|
|
test_i18ngrep "git rebase --skip" err
|
|
'
|
|
|
|
test_expect_success 'correct advice upon cherry-picking an empty commit during a rebase' '
|
|
test_when_finished "git rebase --abort" &&
|
|
(
|
|
set_fake_editor &&
|
|
test_must_fail env FAKE_LINES="1 exec_git_cherry-pick_amended-goodbye" \
|
|
git rebase -i goodbye^ goodbye 2>err
|
|
) &&
|
|
test_i18ngrep "previous cherry-pick is now empty" err &&
|
|
test_i18ngrep "git cherry-pick --skip" err &&
|
|
test_must_fail git commit 2>err &&
|
|
test_i18ngrep "git cherry-pick --skip" err
|
|
'
|
|
|
|
test_expect_success 'correct advice upon multi cherry-pick picking an empty commit during a rebase' '
|
|
test_when_finished "git rebase --abort" &&
|
|
(
|
|
set_fake_editor &&
|
|
test_must_fail env FAKE_LINES="1 exec_git_cherry-pick_goodbye_amended-goodbye" \
|
|
git rebase -i goodbye^^ goodbye 2>err
|
|
) &&
|
|
test_i18ngrep "previous cherry-pick is now empty" err &&
|
|
test_i18ngrep "git cherry-pick --skip" err &&
|
|
test_must_fail git commit 2>err &&
|
|
test_i18ngrep "git cherry-pick --skip" err
|
|
'
|
|
|
|
test_expect_success 'fixup that empties commit fails' '
|
|
test_when_finished "git rebase --abort" &&
|
|
(
|
|
set_fake_editor &&
|
|
test_must_fail env FAKE_LINES="1 fixup 2" git rebase -i \
|
|
goodbye^ reverted-goodbye
|
|
)
|
|
'
|
|
|
|
test_expect_success 'squash that empties commit fails' '
|
|
test_when_finished "git rebase --abort" &&
|
|
(
|
|
set_fake_editor &&
|
|
test_must_fail env FAKE_LINES="1 squash 2" git rebase -i \
|
|
goodbye^ reverted-goodbye
|
|
)
|
|
'
|
|
|
|
# Must be the last test in this file
|
|
test_expect_success '$EDITOR and friends are unchanged' '
|
|
test_editor_unchanged
|
|
'
|
|
|
|
test_done
|