 6741aa6c39
			
		
	
	6741aa6c39
	
	
	
		
			
			Make it easier to edit just the commit message for a commit using 'git rebase -i' by introducing the "reword" command. Signed-off-by: Björn Gustavsson <bgustavsson@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
		
			
				
	
	
		
			488 lines
		
	
	
		
			13 KiB
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
			
		
		
	
	
			488 lines
		
	
	
		
			13 KiB
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
| #!/bin/sh
 | |
| #
 | |
| # Copyright (c) 2007 Johannes E. Schindelin
 | |
| #
 | |
| 
 | |
| test_description='git rebase interactive
 | |
| 
 | |
| This test runs git rebase "interactively", by faking an edit, and verifies
 | |
| that the result still makes sense.
 | |
| '
 | |
| . ./test-lib.sh
 | |
| 
 | |
| . "$TEST_DIRECTORY"/lib-rebase.sh
 | |
| 
 | |
| set_fake_editor
 | |
| 
 | |
| # set up two branches like this:
 | |
| #
 | |
| # A - B - C - D - E
 | |
| #   \
 | |
| #     F - G - H
 | |
| #       \
 | |
| #         I
 | |
| #
 | |
| # where B, D and G touch the same file.
 | |
| 
 | |
| test_expect_success 'setup' '
 | |
| 	: > file1 &&
 | |
| 	git add file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m A &&
 | |
| 	git tag A &&
 | |
| 	echo 1 > file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m B file1 &&
 | |
| 	: > file2 &&
 | |
| 	git add file2 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m C &&
 | |
| 	echo 2 > file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m D file1 &&
 | |
| 	: > file3 &&
 | |
| 	git add file3 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m E &&
 | |
| 	git checkout -b branch1 A &&
 | |
| 	: > file4 &&
 | |
| 	git add file4 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m F &&
 | |
| 	git tag F &&
 | |
| 	echo 3 > file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m G file1 &&
 | |
| 	: > file5 &&
 | |
| 	git add file5 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m H &&
 | |
| 	git checkout -b branch2 F &&
 | |
| 	: > file6 &&
 | |
| 	git add file6 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m I &&
 | |
| 	git tag I
 | |
| '
 | |
| 
 | |
| test_expect_success 'no changes are a nop' '
 | |
| 	git rebase -i F &&
 | |
| 	test "$(git symbolic-ref -q HEAD)" = "refs/heads/branch2" &&
 | |
| 	test $(git rev-parse I) = $(git rev-parse HEAD)
 | |
| '
 | |
| 
 | |
| test_expect_success 'test the [branch] option' '
 | |
| 	git checkout -b dead-end &&
 | |
| 	git rm file6 &&
 | |
| 	git commit -m "stop here" &&
 | |
| 	git rebase -i F branch2 &&
 | |
| 	test "$(git symbolic-ref -q HEAD)" = "refs/heads/branch2" &&
 | |
| 	test $(git rev-parse I) = $(git rev-parse branch2) &&
 | |
| 	test $(git rev-parse I) = $(git rev-parse HEAD)
 | |
| '
 | |
| 
 | |
| test_expect_success 'test --onto <branch>' '
 | |
| 	git checkout -b test-onto branch2 &&
 | |
| 	git rebase -i --onto branch1 F &&
 | |
| 	test "$(git symbolic-ref -q HEAD)" = "refs/heads/test-onto" &&
 | |
| 	test $(git rev-parse HEAD^) = $(git rev-parse branch1) &&
 | |
| 	test $(git rev-parse I) = $(git rev-parse branch2)
 | |
| '
 | |
| 
 | |
| test_expect_success 'rebase on top of a non-conflicting commit' '
 | |
| 	git checkout branch1 &&
 | |
| 	git tag original-branch1 &&
 | |
| 	git rebase -i branch2 &&
 | |
| 	test file6 = $(git diff --name-only original-branch1) &&
 | |
| 	test "$(git symbolic-ref -q HEAD)" = "refs/heads/branch1" &&
 | |
| 	test $(git rev-parse I) = $(git rev-parse branch2) &&
 | |
| 	test $(git rev-parse I) = $(git rev-parse HEAD~2)
 | |
| '
 | |
| 
 | |
| test_expect_success 'reflog for the branch shows state before rebase' '
 | |
| 	test $(git rev-parse branch1@{1}) = $(git rev-parse original-branch1)
 | |
| '
 | |
| 
 | |
| test_expect_success 'exchange two commits' '
 | |
| 	FAKE_LINES="2 1" git rebase -i HEAD~2 &&
 | |
| 	test H = $(git cat-file commit HEAD^ | sed -ne \$p) &&
 | |
| 	test G = $(git cat-file commit HEAD | sed -ne \$p)
 | |
| '
 | |
| 
 | |
| cat > expect << EOF
 | |
| diff --git a/file1 b/file1
 | |
| index e69de29..00750ed 100644
 | |
| --- a/file1
 | |
| +++ b/file1
 | |
| @@ -0,0 +1 @@
 | |
| +3
 | |
| EOF
 | |
| 
 | |
| cat > expect2 << EOF
 | |
| <<<<<<< HEAD
 | |
| 2
 | |
| =======
 | |
| 3
 | |
| >>>>>>> b7ca976... G
 | |
| EOF
 | |
| 
 | |
| test_expect_success 'stop on conflicting pick' '
 | |
| 	git tag new-branch1 &&
 | |
| 	test_must_fail git rebase -i master &&
 | |
| 	test "$(git rev-parse HEAD~3)" = "$(git rev-parse master)" &&
 | |
| 	test_cmp expect .git/rebase-merge/patch &&
 | |
| 	test_cmp expect2 file1 &&
 | |
| 	test "$(git diff --name-status |
 | |
| 		sed -n -e "/^U/s/^U[^a-z]*//p")" = file1 &&
 | |
| 	test 4 = $(grep -v "^#" < .git/rebase-merge/done | wc -l) &&
 | |
| 	test 0 = $(grep -c "^[^#]" < .git/rebase-merge/git-rebase-todo)
 | |
| '
 | |
| 
 | |
| test_expect_success 'abort' '
 | |
| 	git rebase --abort &&
 | |
| 	test $(git rev-parse new-branch1) = $(git rev-parse HEAD) &&
 | |
| 	test "$(git symbolic-ref -q HEAD)" = "refs/heads/branch1" &&
 | |
| 	! test -d .git/rebase-merge
 | |
| '
 | |
| 
 | |
| test_expect_success 'retain authorship' '
 | |
| 	echo A > file7 &&
 | |
| 	git add file7 &&
 | |
| 	test_tick &&
 | |
| 	GIT_AUTHOR_NAME="Twerp Snog" git commit -m "different author" &&
 | |
| 	git tag twerp &&
 | |
| 	git rebase -i --onto master HEAD^ &&
 | |
| 	git show HEAD | grep "^Author: Twerp Snog"
 | |
| '
 | |
| 
 | |
| test_expect_success 'squash' '
 | |
| 	git reset --hard twerp &&
 | |
| 	echo B > file7 &&
 | |
| 	test_tick &&
 | |
| 	GIT_AUTHOR_NAME="Nitfol" git commit -m "nitfol" file7 &&
 | |
| 	echo "******************************" &&
 | |
| 	FAKE_LINES="1 squash 2" git rebase -i --onto master HEAD~2 &&
 | |
| 	test B = $(cat file7) &&
 | |
| 	test $(git rev-parse HEAD^) = $(git rev-parse master)
 | |
| '
 | |
| 
 | |
| test_expect_success 'retain authorship when squashing' '
 | |
| 	git show HEAD | grep "^Author: Twerp Snog"
 | |
| '
 | |
| 
 | |
| test_expect_success '-p handles "no changes" gracefully' '
 | |
| 	HEAD=$(git rev-parse HEAD) &&
 | |
| 	git rebase -i -p HEAD^ &&
 | |
| 	git update-index --refresh &&
 | |
| 	git diff-files --quiet &&
 | |
| 	git diff-index --quiet --cached HEAD -- &&
 | |
| 	test $HEAD = $(git rev-parse HEAD)
 | |
| '
 | |
| 
 | |
| test_expect_success 'preserve merges with -p' '
 | |
| 	git checkout -b to-be-preserved master^ &&
 | |
| 	: > unrelated-file &&
 | |
| 	git add unrelated-file &&
 | |
| 	test_tick &&
 | |
| 	git commit -m "unrelated" &&
 | |
| 	git checkout -b another-branch master &&
 | |
| 	echo B > file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m J file1 &&
 | |
| 	test_tick &&
 | |
| 	git merge to-be-preserved &&
 | |
| 	echo C > file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m K file1 &&
 | |
| 	echo D > file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m L1 file1 &&
 | |
| 	git checkout HEAD^ &&
 | |
| 	echo 1 > unrelated-file &&
 | |
| 	test_tick &&
 | |
| 	git commit -m L2 unrelated-file &&
 | |
| 	test_tick &&
 | |
| 	git merge another-branch &&
 | |
| 	echo E > file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m M file1 &&
 | |
| 	git checkout -b to-be-rebased &&
 | |
| 	test_tick &&
 | |
| 	git rebase -i -p --onto branch1 master &&
 | |
| 	git update-index --refresh &&
 | |
| 	git diff-files --quiet &&
 | |
| 	git diff-index --quiet --cached HEAD -- &&
 | |
| 	test $(git rev-parse HEAD~6) = $(git rev-parse branch1) &&
 | |
| 	test $(git rev-parse HEAD~4^2) = $(git rev-parse to-be-preserved) &&
 | |
| 	test $(git rev-parse HEAD^^2^) = $(git rev-parse HEAD^^^) &&
 | |
| 	test $(git show HEAD~5:file1) = B &&
 | |
| 	test $(git show HEAD~3:file1) = C &&
 | |
| 	test $(git show HEAD:file1) = E &&
 | |
| 	test $(git show HEAD:unrelated-file) = 1
 | |
| '
 | |
| 
 | |
| test_expect_success 'edit ancestor with -p' '
 | |
| 	FAKE_LINES="1 edit 2 3 4" git rebase -i -p HEAD~3 &&
 | |
| 	echo 2 > unrelated-file &&
 | |
| 	test_tick &&
 | |
| 	git commit -m L2-modified --amend unrelated-file &&
 | |
| 	git rebase --continue &&
 | |
| 	git update-index --refresh &&
 | |
| 	git diff-files --quiet &&
 | |
| 	git diff-index --quiet --cached HEAD -- &&
 | |
| 	test $(git show HEAD:unrelated-file) = 2
 | |
| '
 | |
| 
 | |
| test_expect_success '--continue tries to commit' '
 | |
| 	test_tick &&
 | |
| 	test_must_fail git rebase -i --onto new-branch1 HEAD^ &&
 | |
| 	echo resolved > file1 &&
 | |
| 	git add file1 &&
 | |
| 	FAKE_COMMIT_MESSAGE="chouette!" git rebase --continue &&
 | |
| 	test $(git rev-parse HEAD^) = $(git rev-parse new-branch1) &&
 | |
| 	git show HEAD | grep chouette
 | |
| '
 | |
| 
 | |
| test_expect_success 'verbose flag is heeded, even after --continue' '
 | |
| 	git reset --hard HEAD@{1} &&
 | |
| 	test_tick &&
 | |
| 	test_must_fail git rebase -v -i --onto new-branch1 HEAD^ &&
 | |
| 	echo resolved > file1 &&
 | |
| 	git add file1 &&
 | |
| 	git rebase --continue > output &&
 | |
| 	grep "^ file1 |    2 +-$" output
 | |
| '
 | |
| 
 | |
| test_expect_success 'multi-squash only fires up editor once' '
 | |
| 	base=$(git rev-parse HEAD~4) &&
 | |
| 	FAKE_COMMIT_AMEND="ONCE" FAKE_LINES="1 squash 2 squash 3 squash 4" \
 | |
| 		git rebase -i $base &&
 | |
| 	test $base = $(git rev-parse HEAD^) &&
 | |
| 	test 1 = $(git show | grep ONCE | wc -l)
 | |
| '
 | |
| 
 | |
| test_expect_success 'squash works as expected' '
 | |
| 	for n in one two three four
 | |
| 	do
 | |
| 		echo $n >> file$n &&
 | |
| 		git add file$n &&
 | |
| 		git commit -m $n
 | |
| 	done &&
 | |
| 	one=$(git rev-parse HEAD~3) &&
 | |
| 	FAKE_LINES="1 squash 3 2" git rebase -i HEAD~3 &&
 | |
| 	test $one = $(git rev-parse HEAD~2)
 | |
| '
 | |
| 
 | |
| test_expect_success 'interrupted squash works as expected' '
 | |
| 	for n in one two three four
 | |
| 	do
 | |
| 		echo $n >> conflict &&
 | |
| 		git add conflict &&
 | |
| 		git commit -m $n
 | |
| 	done &&
 | |
| 	one=$(git rev-parse HEAD~3) &&
 | |
| 	(
 | |
| 		FAKE_LINES="1 squash 3 2" &&
 | |
| 		export FAKE_LINES &&
 | |
| 		test_must_fail git rebase -i HEAD~3
 | |
| 	) &&
 | |
| 	(echo one; echo two; echo four) > conflict &&
 | |
| 	git add conflict &&
 | |
| 	test_must_fail git rebase --continue &&
 | |
| 	echo resolved > conflict &&
 | |
| 	git add conflict &&
 | |
| 	git rebase --continue &&
 | |
| 	test $one = $(git rev-parse HEAD~2)
 | |
| '
 | |
| 
 | |
| test_expect_success 'interrupted squash works as expected (case 2)' '
 | |
| 	for n in one two three four
 | |
| 	do
 | |
| 		echo $n >> conflict &&
 | |
| 		git add conflict &&
 | |
| 		git commit -m $n
 | |
| 	done &&
 | |
| 	one=$(git rev-parse HEAD~3) &&
 | |
| 	(
 | |
| 		FAKE_LINES="3 squash 1 2" &&
 | |
| 		export FAKE_LINES &&
 | |
| 		test_must_fail git rebase -i HEAD~3
 | |
| 	) &&
 | |
| 	(echo one; echo four) > conflict &&
 | |
| 	git add conflict &&
 | |
| 	test_must_fail git rebase --continue &&
 | |
| 	(echo one; echo two; echo four) > conflict &&
 | |
| 	git add conflict &&
 | |
| 	test_must_fail git rebase --continue &&
 | |
| 	echo resolved > conflict &&
 | |
| 	git add conflict &&
 | |
| 	git rebase --continue &&
 | |
| 	test $one = $(git rev-parse HEAD~2)
 | |
| '
 | |
| 
 | |
| test_expect_success 'ignore patch if in upstream' '
 | |
| 	HEAD=$(git rev-parse HEAD) &&
 | |
| 	git checkout -b has-cherry-picked HEAD^ &&
 | |
| 	echo unrelated > file7 &&
 | |
| 	git add file7 &&
 | |
| 	test_tick &&
 | |
| 	git commit -m "unrelated change" &&
 | |
| 	git cherry-pick $HEAD &&
 | |
| 	EXPECT_COUNT=1 git rebase -i $HEAD &&
 | |
| 	test $HEAD = $(git rev-parse HEAD^)
 | |
| '
 | |
| 
 | |
| test_expect_success '--continue tries to commit, even for "edit"' '
 | |
| 	parent=$(git rev-parse HEAD^) &&
 | |
| 	test_tick &&
 | |
| 	FAKE_LINES="edit 1" git rebase -i HEAD^ &&
 | |
| 	echo edited > file7 &&
 | |
| 	git add file7 &&
 | |
| 	FAKE_COMMIT_MESSAGE="chouette!" git rebase --continue &&
 | |
| 	test edited = $(git show HEAD:file7) &&
 | |
| 	git show HEAD | grep chouette &&
 | |
| 	test $parent = $(git rev-parse HEAD^)
 | |
| '
 | |
| 
 | |
| test_expect_success 'aborted --continue does not squash commits after "edit"' '
 | |
| 	old=$(git rev-parse HEAD) &&
 | |
| 	test_tick &&
 | |
| 	FAKE_LINES="edit 1" git rebase -i HEAD^ &&
 | |
| 	echo "edited again" > file7 &&
 | |
| 	git add file7 &&
 | |
| 	(
 | |
| 		FAKE_COMMIT_MESSAGE=" " &&
 | |
| 		export FAKE_COMMIT_MESSAGE &&
 | |
| 		test_must_fail git rebase --continue
 | |
| 	) &&
 | |
| 	test $old = $(git rev-parse HEAD) &&
 | |
| 	git rebase --abort
 | |
| '
 | |
| 
 | |
| test_expect_success 'auto-amend only edited commits after "edit"' '
 | |
| 	test_tick &&
 | |
| 	FAKE_LINES="edit 1" git rebase -i HEAD^ &&
 | |
| 	echo "edited again" > file7 &&
 | |
| 	git add file7 &&
 | |
| 	FAKE_COMMIT_MESSAGE="edited file7 again" git commit &&
 | |
| 	echo "and again" > file7 &&
 | |
| 	git add file7 &&
 | |
| 	test_tick &&
 | |
| 	(
 | |
| 		FAKE_COMMIT_MESSAGE="and again" &&
 | |
| 		export FAKE_COMMIT_MESSAGE &&
 | |
| 		test_must_fail git rebase --continue
 | |
| 	) &&
 | |
| 	git rebase --abort
 | |
| '
 | |
| 
 | |
| test_expect_success 'rebase a detached HEAD' '
 | |
| 	grandparent=$(git rev-parse HEAD~2) &&
 | |
| 	git checkout $(git rev-parse HEAD) &&
 | |
| 	test_tick &&
 | |
| 	FAKE_LINES="2 1" git rebase -i HEAD~2 &&
 | |
| 	test $grandparent = $(git rev-parse HEAD~2)
 | |
| '
 | |
| 
 | |
| test_expect_success 'rebase a commit violating pre-commit' '
 | |
| 
 | |
| 	mkdir -p .git/hooks &&
 | |
| 	PRE_COMMIT=.git/hooks/pre-commit &&
 | |
| 	echo "#!/bin/sh" > $PRE_COMMIT &&
 | |
| 	echo "test -z \"\$(git diff --cached --check)\"" >> $PRE_COMMIT &&
 | |
| 	chmod a+x $PRE_COMMIT &&
 | |
| 	echo "monde! " >> file1 &&
 | |
| 	test_tick &&
 | |
| 	test_must_fail git commit -m doesnt-verify file1 &&
 | |
| 	git commit -m doesnt-verify --no-verify file1 &&
 | |
| 	test_tick &&
 | |
| 	FAKE_LINES=2 git rebase -i HEAD~2
 | |
| 
 | |
| '
 | |
| 
 | |
| test_expect_success 'rebase with a file named HEAD in worktree' '
 | |
| 
 | |
| 	rm -fr .git/hooks &&
 | |
| 	git reset --hard &&
 | |
| 	git checkout -b branch3 A &&
 | |
| 
 | |
| 	(
 | |
| 		GIT_AUTHOR_NAME="Squashed Away" &&
 | |
| 		export GIT_AUTHOR_NAME &&
 | |
| 		>HEAD &&
 | |
| 		git add HEAD &&
 | |
| 		git commit -m "Add head" &&
 | |
| 		>BODY &&
 | |
| 		git add BODY &&
 | |
| 		git commit -m "Add body"
 | |
| 	) &&
 | |
| 
 | |
| 	FAKE_LINES="1 squash 2" git rebase -i to-be-rebased &&
 | |
| 	test "$(git show -s --pretty=format:%an)" = "Squashed Away"
 | |
| 
 | |
| '
 | |
| 
 | |
| test_expect_success 'do "noop" when there is nothing to cherry-pick' '
 | |
| 
 | |
| 	git checkout -b branch4 HEAD &&
 | |
| 	GIT_EDITOR=: git commit --amend \
 | |
| 		--author="Somebody else <somebody@else.com>" 
 | |
| 	test $(git rev-parse branch3) != $(git rev-parse branch4) &&
 | |
| 	git rebase -i branch3 &&
 | |
| 	test $(git rev-parse branch3) = $(git rev-parse branch4)
 | |
| 
 | |
| '
 | |
| 
 | |
| test_expect_success 'submodule rebase setup' '
 | |
| 	git checkout A &&
 | |
| 	mkdir sub &&
 | |
| 	(
 | |
| 		cd sub && git init && >elif &&
 | |
| 		git add elif && git commit -m "submodule initial"
 | |
| 	) &&
 | |
| 	echo 1 >file1 &&
 | |
| 	git add file1 sub
 | |
| 	test_tick &&
 | |
| 	git commit -m "One" &&
 | |
| 	echo 2 >file1 &&
 | |
| 	test_tick &&
 | |
| 	git commit -a -m "Two" &&
 | |
| 	(
 | |
| 		cd sub && echo 3 >elif &&
 | |
| 		git commit -a -m "submodule second"
 | |
| 	) &&
 | |
| 	test_tick &&
 | |
| 	git commit -a -m "Three changes submodule"
 | |
| '
 | |
| 
 | |
| test_expect_success 'submodule rebase -i' '
 | |
| 	FAKE_LINES="1 squash 2 3" git rebase -i A
 | |
| '
 | |
| 
 | |
| test_expect_success 'avoid unnecessary reset' '
 | |
| 	git checkout master &&
 | |
| 	test-chmtime =123456789 file3 &&
 | |
| 	git update-index --refresh &&
 | |
| 	HEAD=$(git rev-parse HEAD) &&
 | |
| 	git rebase -i HEAD~4 &&
 | |
| 	test $HEAD = $(git rev-parse HEAD) &&
 | |
| 	MTIME=$(test-chmtime -v +0 file3 | sed 's/[^0-9].*$//') &&
 | |
| 	test 123456789 = $MTIME
 | |
| '
 | |
| 
 | |
| test_expect_success 'reword' '
 | |
| 	git checkout -b reword-branch master &&
 | |
| 	FAKE_LINES="1 2 3 reword 4" FAKE_COMMIT_MESSAGE="E changed" git rebase -i A &&
 | |
| 	git show HEAD | grep "E changed" &&
 | |
| 	test $(git rev-parse master) != $(git rev-parse HEAD) &&
 | |
| 	test $(git rev-parse master^) = $(git rev-parse HEAD^) &&
 | |
| 	FAKE_LINES="1 2 reword 3 4" FAKE_COMMIT_MESSAGE="D changed" git rebase -i A &&
 | |
| 	git show HEAD^ | grep "D changed" &&
 | |
| 	FAKE_LINES="reword 1 2 3 4" FAKE_COMMIT_MESSAGE="B changed" git rebase -i A &&
 | |
| 	git show HEAD~3 | grep "B changed" &&
 | |
| 	FAKE_LINES="1 reword 2 3 4" FAKE_COMMIT_MESSAGE="C changed" git rebase -i A &&
 | |
| 	git show HEAD~2 | grep "C changed"
 | |
| '
 | |
| 
 | |
| test_done
 |