[thirdparty/git.git] / t / t7502-commit.sh

#!/bin/sh

test_description='git commit porcelain-ish'

. ./test-lib.sh

commit_msg_is () {
	expect=commit_msg_is.expect
	actual=commit_msg_is.actual

	printf "%s" "$(git log --pretty=format:%s%b -1)" >$actual &&
	printf "%s" "$1" >$expect &&
	test_i18ncmp $expect $actual
}

# Arguments: [<prefix] [<commit message>] [<commit options>]
check_summary_oneline() {
	test_tick &&
	git commit ${3+"$3"} -m "$2" | head -1 > act &&

	# branch name
	SUMMARY_PREFIX="$(git name-rev --name-only HEAD)" &&

	# append the "special" prefix, like "root-commit", "detached HEAD"
	if test -n "$1"
	then
		SUMMARY_PREFIX="$SUMMARY_PREFIX ($1)"
	fi

	# abbrev SHA-1
	SUMMARY_POSTFIX="$(git log -1 --pretty='format:%h')"
	echo "[$SUMMARY_PREFIX $SUMMARY_POSTFIX] $2" >exp &&

	test_i18ncmp exp act
}

test_expect_success 'output summary format' '

	echo new >file1 &&
	git add file1 &&
	check_summary_oneline "root-commit" "initial" &&

	echo change >>file1 &&
	git add file1
'

test_expect_success 'output summary format: root-commit' '
	check_summary_oneline "" "a change"
'

test_expect_success 'output summary format for commit with an empty diff' '

	check_summary_oneline "" "empty" "--allow-empty"
'

test_expect_success 'output summary format for merges' '

	git checkout -b recursive-base &&
	test_commit base file1 &&

	git checkout -b recursive-a recursive-base &&
	test_commit commit-a file1 &&

	git checkout -b recursive-b recursive-base &&
	test_commit commit-b file1 &&

	# conflict
	git checkout recursive-a &&
	test_must_fail git merge recursive-b &&
	# resolve the conflict
	echo commit-a > file1 &&
	git add file1 &&
	check_summary_oneline "" "Merge"
'

output_tests_cleanup() {
	# this is needed for "do not fire editor in the presence of conflicts"
	git checkout master &&

	# this is needed for the "partial removal" test to pass
	git rm file1 &&
	git commit -m "cleanup"
}

test_expect_success 'the basics' '

	output_tests_cleanup &&

	echo doing partial >"commit is" &&
	mkdir not &&
	echo very much encouraged but we should >not/forbid &&
	git add "commit is" not &&
	echo update added "commit is" file >"commit is" &&
	echo also update another >not/forbid &&
	test_tick &&
	git commit -a -m "initial with -a" &&

	git cat-file blob HEAD:"commit is" >current.1 &&
	git cat-file blob HEAD:not/forbid >current.2 &&

	cmp current.1 "commit is" &&
	cmp current.2 not/forbid

'

test_expect_success 'partial' '

	echo another >"commit is" &&
	echo another >not/forbid &&
	test_tick &&
	git commit -m "partial commit to handle a file" "commit is" &&

	changed=$(git diff-tree --name-only HEAD^ HEAD) &&
	test "$changed" = "commit is"

'

test_expect_success 'partial modification in a subdirectory' '

	test_tick &&
	git commit -m "partial commit to subdirectory" not &&

	changed=$(git diff-tree -r --name-only HEAD^ HEAD) &&
	test "$changed" = "not/forbid"

'

test_expect_success 'partial removal' '

	git rm not/forbid &&
	git commit -m "partial commit to remove not/forbid" not &&

	changed=$(git diff-tree -r --name-only HEAD^ HEAD) &&
	test "$changed" = "not/forbid" &&
	remain=$(git ls-tree -r --name-only HEAD) &&
	test "$remain" = "commit is"

'

test_expect_success 'sign off' '

	>positive &&
	git add positive &&
	git commit -s -m "thank you" &&
	actual=$(git cat-file commit HEAD | sed -ne "s/Signed-off-by: //p") &&
	expected=$(git var GIT_COMMITTER_IDENT | sed -e "s/>.*/>/") &&
	test "z$actual" = "z$expected"

'

test_expect_success 'multiple -m' '

	>negative &&
	git add negative &&
	git commit -m "one" -m "two" -m "three" &&
	actual=$(git cat-file commit HEAD | sed -e "1,/^\$/d") &&
	expected=$(echo one; echo; echo two; echo; echo three) &&
	test "z$actual" = "z$expected"

'

test_expect_success 'verbose' '

	echo minus >negative &&
	git add negative &&
	git status -v | sed -ne "/^diff --git /p" >actual &&
	echo "diff --git a/negative b/negative" >expect &&
	test_cmp expect actual

'

test_expect_success 'verbose respects diff config' '

	test_config color.diff always &&
	git status -v >actual &&
	grep "\[1mdiff --git" actual
'

mesg_with_comment_and_newlines='
# text

'

test_expect_success 'prepare file with comment line and trailing newlines'  '
	printf "%s" "$mesg_with_comment_and_newlines" >expect
'

test_expect_success 'cleanup commit messages (verbatim option,-t)' '

	echo >>negative &&
	git commit --cleanup=verbatim --no-status -t expect -a &&
	git cat-file -p HEAD |sed -e "1,/^\$/d" >actual &&
	test_cmp expect actual

'

test_expect_success 'cleanup commit messages (verbatim option,-F)' '

	echo >>negative &&
	git commit --cleanup=verbatim -F expect -a &&
	git cat-file -p HEAD |sed -e "1,/^\$/d">actual &&
	test_cmp expect actual

'

test_expect_success 'cleanup commit messages (verbatim option,-m)' '

	echo >>negative &&
	git commit --cleanup=verbatim -m "$mesg_with_comment_and_newlines" -a &&
	git cat-file -p HEAD |sed -e "1,/^\$/d">actual &&
	test_cmp expect actual

'

test_expect_success 'cleanup commit messages (whitespace option,-F)' '

	echo >>negative &&
	{ echo;echo "# text";echo; } >text &&
	echo "# text" >expect &&
	git commit --cleanup=whitespace -F text -a &&
	git cat-file -p HEAD |sed -e "1,/^\$/d">actual &&
	test_cmp expect actual

'

test_expect_success 'cleanup commit messages (strip option,-F)' '

	echo >>negative &&
	{ echo;echo "# text";echo sample;echo; } >text &&
	echo sample >expect &&
	git commit --cleanup=strip -F text -a &&
	git cat-file -p HEAD |sed -e "1,/^\$/d">actual &&
	test_cmp expect actual

'

test_expect_success 'cleanup commit messages (strip option,-F,-e)' '

	echo >>negative &&
	{ echo;echo sample;echo; } >text &&
	git commit -e -F text -a &&
	head -n 4 .git/COMMIT_EDITMSG >actual
'

echo "sample

# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit." >expect

test_expect_success 'cleanup commit messages (strip option,-F,-e): output' '
	test_i18ncmp expect actual
'

test_expect_success 'cleanup commit message (fail on invalid cleanup mode option)' '
	test_must_fail git commit --cleanup=non-existent
'

test_expect_success 'cleanup commit message (fail on invalid cleanup mode configuration)' '
	test_must_fail git -c commit.cleanup=non-existent commit
'

test_expect_success 'cleanup commit message (no config and no option uses default)' '
	echo content >>file &&
	git add file &&
	(
	  test_set_editor "$TEST_DIRECTORY"/t7500/add-content-and-comment &&
	  git commit --no-status
	) &&
	commit_msg_is "commit message"
'

test_expect_success 'cleanup commit message (option overrides default)' '
	echo content >>file &&
	git add file &&
	(
	  test_set_editor "$TEST_DIRECTORY"/t7500/add-content-and-comment &&
	  git commit --cleanup=whitespace --no-status
	) &&
	commit_msg_is "commit message # comment"
'

test_expect_success 'cleanup commit message (config overrides default)' '
	echo content >>file &&
	git add file &&
	(
	  test_set_editor "$TEST_DIRECTORY"/t7500/add-content-and-comment &&
	  git -c commit.cleanup=whitespace commit --no-status
	) &&
	commit_msg_is "commit message # comment"
'

test_expect_success 'cleanup commit message (option overrides config)' '
	echo content >>file &&
	git add file &&
	(
	  test_set_editor "$TEST_DIRECTORY"/t7500/add-content-and-comment &&
	  git -c commit.cleanup=whitespace commit --cleanup=default
	) &&
	commit_msg_is "commit message"
'

test_expect_success 'cleanup commit message (default, -m)' '
	echo content >>file &&
	git add file &&
	git commit -m "message #comment " &&
	commit_msg_is "message #comment"
'

test_expect_success 'cleanup commit message (whitespace option, -m)' '
	echo content >>file &&
	git add file &&
	git commit --cleanup=whitespace --no-status -m "message #comment " &&
	commit_msg_is "message #comment"
'

test_expect_success 'cleanup commit message (whitespace config, -m)' '
	echo content >>file &&
	git add file &&
	git -c commit.cleanup=whitespace commit --no-status -m "message #comment " &&
	commit_msg_is "message #comment"
'

test_expect_success 'message shows author when it is not equal to committer' '
	echo >>negative &&
	git commit -e -m "sample" -a &&
	test_i18ngrep \
	  "^# Author: *A U Thor <author@example.com>\$" \
	  .git/COMMIT_EDITMSG
'

test_expect_success AUTOIDENT 'message shows committer when it is automatic' '

	echo >>negative &&
	(
		sane_unset GIT_COMMITTER_EMAIL &&
		sane_unset GIT_COMMITTER_NAME &&
		git commit -e -m "sample" -a
	) &&
	# the ident is calculated from the system, so we cannot
	# check the actual value, only that it is there
	test_i18ngrep "^# Committer: " .git/COMMIT_EDITMSG
'

write_script .git/FAKE_EDITOR <<EOF
echo editor started > "$(pwd)/.git/result"
exit 0
EOF

test_expect_success !AUTOIDENT 'do not fire editor when committer is bogus' '
	>.git/result
	>expect &&

	echo >>negative &&
	(
		sane_unset GIT_COMMITTER_EMAIL &&
		sane_unset GIT_COMMITTER_NAME &&
		GIT_EDITOR="\"$(pwd)/.git/FAKE_EDITOR\"" &&
		export GIT_EDITOR &&
		test_must_fail git commit -e -m sample -a
	) &&
	test_cmp expect .git/result
'

test_expect_success 'do not fire editor in the presence of conflicts' '

	git clean -f &&
	echo f >g &&
	git add g &&
	git commit -m "add g" &&
	git branch second &&
	echo master >g &&
	echo g >h &&
	git add g h &&
	git commit -m "modify g and add h" &&
	git checkout second &&
	echo second >g &&
	git add g &&
	git commit -m second &&
	# Must fail due to conflict
	test_must_fail git cherry-pick -n master &&
	echo "editor not started" >.git/result &&
	(
		GIT_EDITOR="\"$(pwd)/.git/FAKE_EDITOR\"" &&
		export GIT_EDITOR &&
		test_must_fail git commit
	) &&
	test "$(cat .git/result)" = "editor not started"
'

write_script .git/FAKE_EDITOR <<EOF
# kill -TERM command added below.
EOF

test_expect_success EXECKEEPSPID 'a SIGTERM should break locks' '
	echo >>negative &&
	! "$SHELL_PATH" -c '\''
	  echo kill -TERM $$ >> .git/FAKE_EDITOR
	  GIT_EDITOR=.git/FAKE_EDITOR
	  export GIT_EDITOR
	  exec git commit -a'\'' &&
	test ! -f .git/index.lock
'

rm -f .git/MERGE_MSG .git/COMMIT_EDITMSG
git reset -q --hard

test_expect_success 'Hand committing of a redundant merge removes dups' '

	git rev-parse second master >expect &&
	test_must_fail git merge second master &&
	git checkout master g &&
	EDITOR=: git commit -a &&
	git cat-file commit HEAD | sed -n -e "s/^parent //p" -e "/^$/q" >actual &&
	test_cmp expect actual

'

test_expect_success 'A single-liner subject with a token plus colon is not a footer' '

	git reset --hard &&
	git commit -s -m "hello: kitty" --allow-empty &&
	git cat-file commit HEAD | sed -e "1,/^$/d" >actual &&
	test_line_count = 3 actual

'

test_expect_success 'commit -s places sob on third line after two empty lines' '
	git commit -s --allow-empty --allow-empty-message &&
	cat <<-EOF >expect &&


	Signed-off-by: $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL>

	EOF
	sed -e "/^#/d" -e "s/^:.*//" .git/COMMIT_EDITMSG >actual &&
	test_cmp expect actual
'

write_script .git/FAKE_EDITOR <<\EOF
mv "$1" "$1.orig"
(
	echo message
	cat "$1.orig"
) >"$1"
EOF

echo '## Custom template' >template

try_commit () {
	git reset --hard &&
	echo >>negative &&
	GIT_EDITOR=.git/FAKE_EDITOR git commit -a $* $use_template &&
	case "$use_template" in
	'')
		test_i18ngrep ! "^## Custom template" .git/COMMIT_EDITMSG ;;
	*)
		test_i18ngrep "^## Custom template" .git/COMMIT_EDITMSG ;;
	esac
}

try_commit_status_combo () {

	test_expect_success 'commit' '
		try_commit "" &&
		test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit' '
		try_commit "" &&
		test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit --status' '
		try_commit --status &&
		test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit --no-status' '
		try_commit --no-status &&
		test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit with commit.status = yes' '
		test_config commit.status yes &&
		try_commit "" &&
		test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit with commit.status = no' '
		test_config commit.status no &&
		try_commit "" &&
		test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit --status with commit.status = yes' '
		test_config commit.status yes &&
		try_commit --status &&
		test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit --no-status with commit.status = yes' '
		test_config commit.status yes &&
		try_commit --no-status &&
		test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit --status with commit.status = no' '
		test_config commit.status no &&
		try_commit --status &&
		test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

	test_expect_success 'commit --no-status with commit.status = no' '
		test_config commit.status no &&
		try_commit --no-status &&
		test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
	'

}

try_commit_status_combo

use_template="-t template"

try_commit_status_combo

test_expect_success 'commit --status with custom comment character' '
	test_config core.commentchar ";" &&
	try_commit --status &&
	test_i18ngrep "^; Changes to be committed:" .git/COMMIT_EDITMSG
'

test_done
Commit	Line	Data
b468f0ce JH	1	#!/bin/sh
	2
	3	test_description='git commit porcelain-ish'
	4
	5	. ./test-lib.sh
	6
51fb3a3d RT	7	commit_msg_is () {
	8	expect=commit_msg_is.expect
	9	actual=commit_msg_is.actual
	10
	11	printf "%s" "$(git log --pretty=format:%s%b -1)" >$actual &&
	12	printf "%s" "$1" >$expect &&
	13	test_i18ncmp $expect $actual
	14	}
	15
cee9f2b3	16	# Arguments: [<prefix] [<commit message>] [<commit options>]
fc6fa0d0 TRC	17	check_summary_oneline() {
fc6fa0d0 TRC	18	test_tick &&
cee9f2b3	19	git commit ${3+"$3"} -m "$2" \| head -1 > act &&
fc6fa0d0 TRC	20
	21	# branch name
	22	SUMMARY_PREFIX="$(git name-rev --name-only HEAD)" &&
	23
	24	# append the "special" prefix, like "root-commit", "detached HEAD"
	25	if test -n "$1"
	26	then
	27	SUMMARY_PREFIX="$SUMMARY_PREFIX ($1)"
	28	fi
	29
	30	# abbrev SHA-1
	31	SUMMARY_POSTFIX="$(git log -1 --pretty='format:%h')"
	32	echo "[$SUMMARY_PREFIX $SUMMARY_POSTFIX] $2" >exp &&
	33
f79ce8db	34	test_i18ncmp exp act
fc6fa0d0 TRC	35	}
	36
	37	test_expect_success 'output summary format' '
	38
	39	echo new >file1 &&
	40	git add file1 &&
	41	check_summary_oneline "root-commit" "initial" &&
	42
	43	echo change >>file1 &&
7f5673d7 ÆAB	44	git add file1
	45	'
	46
	47	test_expect_success 'output summary format: root-commit' '
fc6fa0d0 TRC	48	check_summary_oneline "" "a change"
	49	'
	50
a45e1a87	51	test_expect_success 'output summary format for commit with an empty diff' '
cee9f2b3 TRC	52
	53	check_summary_oneline "" "empty" "--allow-empty"
	54	'
	55
a45e1a87	56	test_expect_success 'output summary format for merges' '
cee9f2b3 TRC	57
	58	git checkout -b recursive-base &&
	59	test_commit base file1 &&
	60
	61	git checkout -b recursive-a recursive-base &&
	62	test_commit commit-a file1 &&
	63
	64	git checkout -b recursive-b recursive-base &&
	65	test_commit commit-b file1 &&
	66
	67	# conflict
	68	git checkout recursive-a &&
	69	test_must_fail git merge recursive-b &&
	70	# resolve the conflict
	71	echo commit-a > file1 &&
	72	git add file1 &&
	73	check_summary_oneline "" "Merge"
	74	'
	75
fc6fa0d0 TRC	76	output_tests_cleanup() {
	77	# this is needed for "do not fire editor in the presence of conflicts"
	78	git checkout master &&
	79
	80	# this is needed for the "partial removal" test to pass
	81	git rm file1 &&
	82	git commit -m "cleanup"
	83	}
	84
b468f0ce JH	85	test_expect_success 'the basics' '
b468f0ce JH	86
fc6fa0d0 TRC	87	output_tests_cleanup &&
fc6fa0d0 TRC	88
b468f0ce JH	89	echo doing partial >"commit is" &&
	90	mkdir not &&
	91	echo very much encouraged but we should >not/forbid &&
	92	git add "commit is" not &&
	93	echo update added "commit is" file >"commit is" &&
	94	echo also update another >not/forbid &&
	95	test_tick &&
	96	git commit -a -m "initial with -a" &&
	97
	98	git cat-file blob HEAD:"commit is" >current.1 &&
	99	git cat-file blob HEAD:not/forbid >current.2 &&
	100
	101	cmp current.1 "commit is" &&
	102	cmp current.2 not/forbid
	103
	104	'
	105
	106	test_expect_success 'partial' '
	107
	108	echo another >"commit is" &&
	109	echo another >not/forbid &&
	110	test_tick &&
	111	git commit -m "partial commit to handle a file" "commit is" &&
	112
	113	changed=$(git diff-tree --name-only HEAD^ HEAD) &&
	114	test "$changed" = "commit is"
	115
	116	'
	117
e8f30160	118	test_expect_success 'partial modification in a subdirectory' '
b468f0ce JH	119
	120	test_tick &&
	121	git commit -m "partial commit to subdirectory" not &&
	122
	123	changed=$(git diff-tree -r --name-only HEAD^ HEAD) &&
	124	test "$changed" = "not/forbid"
	125
	126	'
	127
	128	test_expect_success 'partial removal' '
	129
	130	git rm not/forbid &&
	131	git commit -m "partial commit to remove not/forbid" not &&
	132
	133	changed=$(git diff-tree -r --name-only HEAD^ HEAD) &&
	134	test "$changed" = "not/forbid" &&
	135	remain=$(git ls-tree -r --name-only HEAD) &&
	136	test "$remain" = "commit is"
	137
	138	'
	139
	140	test_expect_success 'sign off' '
	141
	142	>positive &&
	143	git add positive &&
	144	git commit -s -m "thank you" &&
	145	actual=$(git cat-file commit HEAD \| sed -ne "s/Signed-off-by: //p") &&
	146	expected=$(git var GIT_COMMITTER_IDENT \| sed -e "s/>.*/>/") &&
	147	test "z$actual" = "z$expected"
	148
	149	'
	150
	151	test_expect_success 'multiple -m' '
	152
	153	>negative &&
	154	git add negative &&
	155	git commit -m "one" -m "two" -m "three" &&
	156	actual=$(git cat-file commit HEAD \| sed -e "1,/^\$/d") &&
	157	expected=$(echo one; echo; echo two; echo; echo three) &&
	158	test "z$actual" = "z$expected"
	159
	160	'
	161
	162	test_expect_success 'verbose' '
	163
	164	echo minus >negative &&
	165	git add negative &&
	166	git status -v \| sed -ne "/^diff --git /p" >actual &&
	167	echo "diff --git a/negative b/negative" >expect &&
82ebb0b6	168	test_cmp expect actual
b468f0ce JH	169
	170	'
	171
4f672ad6 JK	172	test_expect_success 'verbose respects diff config' '
4f672ad6 JK	173
464be630	174	test_config color.diff always &&
4f672ad6	175	git status -v >actual &&
464be630	176	grep "\[1mdiff --git" actual
4f672ad6 JK	177	'
4f672ad6 JK	178
5b012c80 BC	179	mesg_with_comment_and_newlines='
	180	# text
	181
	182	'
	183
	184	test_expect_success 'prepare file with comment line and trailing newlines' '
	185	printf "%s" "$mesg_with_comment_and_newlines" >expect
	186	'
	187
51fb3a3d	188	test_expect_success 'cleanup commit messages (verbatim option,-t)' '
5f065737 AR	189
5f065737 AR	190	echo >>negative &&
67dabab0 BC	191	git commit --cleanup=verbatim --no-status -t expect -a &&
67dabab0 BC	192	git cat-file -p HEAD \|sed -e "1,/^\$/d" >actual &&
82ebb0b6	193	test_cmp expect actual
5f065737 AR	194
	195	'
	196
51fb3a3d	197	test_expect_success 'cleanup commit messages (verbatim option,-F)' '
5f065737 AR	198
	199	echo >>negative &&
	200	git commit --cleanup=verbatim -F expect -a &&
	201	git cat-file -p HEAD \|sed -e "1,/^\$/d">actual &&
82ebb0b6	202	test_cmp expect actual
5f065737 AR	203
	204	'
	205
a24a41ea	206	test_expect_success 'cleanup commit messages (verbatim option,-m)' '
5f065737 AR	207
5f065737 AR	208	echo >>negative &&
5b012c80	209	git commit --cleanup=verbatim -m "$mesg_with_comment_and_newlines" -a &&
5f065737	210	git cat-file -p HEAD \|sed -e "1,/^\$/d">actual &&
82ebb0b6	211	test_cmp expect actual
5f065737 AR	212
	213	'
	214
51fb3a3d	215	test_expect_success 'cleanup commit messages (whitespace option,-F)' '
5f065737 AR	216
	217	echo >>negative &&
	218	{ echo;echo "# text";echo; } >text &&
	219	echo "# text" >expect &&
	220	git commit --cleanup=whitespace -F text -a &&
	221	git cat-file -p HEAD \|sed -e "1,/^\$/d">actual &&
82ebb0b6	222	test_cmp expect actual
5f065737 AR	223
	224	'
	225
51fb3a3d	226	test_expect_success 'cleanup commit messages (strip option,-F)' '
5f065737 AR	227
	228	echo >>negative &&
	229	{ echo;echo "# text";echo sample;echo; } >text &&
	230	echo sample >expect &&
	231	git commit --cleanup=strip -F text -a &&
	232	git cat-file -p HEAD \|sed -e "1,/^\$/d">actual &&
82ebb0b6	233	test_cmp expect actual
5f065737 AR	234
	235	'
	236
51fb3a3d	237	test_expect_success 'cleanup commit messages (strip option,-F,-e)' '
5f065737 AR	238
	239	echo >>negative &&
	240	{ echo;echo sample;echo; } >text &&
	241	git commit -e -F text -a &&
0b430a17 ÆAB	242	head -n 4 .git/COMMIT_EDITMSG >actual
	243	'
	244
	245	echo "sample
5f065737	246
0b430a17 ÆAB	247	# Please enter the commit message for your changes. Lines starting
	248	# with '#' will be ignored, and an empty message aborts the commit." >expect
	249
51fb3a3d	250	test_expect_success 'cleanup commit messages (strip option,-F,-e): output' '
f79ce8db	251	test_i18ncmp expect actual
5f065737 AR	252	'
5f065737 AR	253
51fb3a3d RT	254	test_expect_success 'cleanup commit message (fail on invalid cleanup mode option)' '
	255	test_must_fail git commit --cleanup=non-existent
	256	'
	257
	258	test_expect_success 'cleanup commit message (fail on invalid cleanup mode configuration)' '
	259	test_must_fail git -c commit.cleanup=non-existent commit
	260	'
	261
	262	test_expect_success 'cleanup commit message (no config and no option uses default)' '
	263	echo content >>file &&
	264	git add file &&
24e099f4 BC	265	(
	266	test_set_editor "$TEST_DIRECTORY"/t7500/add-content-and-comment &&
	267	git commit --no-status
	268	) &&
51fb3a3d RT	269	commit_msg_is "commit message"
	270	'
	271
	272	test_expect_success 'cleanup commit message (option overrides default)' '
	273	echo content >>file &&
	274	git add file &&
24e099f4 BC	275	(
	276	test_set_editor "$TEST_DIRECTORY"/t7500/add-content-and-comment &&
	277	git commit --cleanup=whitespace --no-status
	278	) &&
51fb3a3d RT	279	commit_msg_is "commit message # comment"
	280	'
	281
	282	test_expect_success 'cleanup commit message (config overrides default)' '
	283	echo content >>file &&
	284	git add file &&
24e099f4 BC	285	(
	286	test_set_editor "$TEST_DIRECTORY"/t7500/add-content-and-comment &&
	287	git -c commit.cleanup=whitespace commit --no-status
	288	) &&
51fb3a3d RT	289	commit_msg_is "commit message # comment"
	290	'
	291
	292	test_expect_success 'cleanup commit message (option overrides config)' '
	293	echo content >>file &&
	294	git add file &&
24e099f4 BC	295	(
	296	test_set_editor "$TEST_DIRECTORY"/t7500/add-content-and-comment &&
	297	git -c commit.cleanup=whitespace commit --cleanup=default
	298	) &&
51fb3a3d RT	299	commit_msg_is "commit message"
	300	'
	301
	302	test_expect_success 'cleanup commit message (default, -m)' '
	303	echo content >>file &&
	304	git add file &&
	305	git commit -m "message #comment " &&
	306	commit_msg_is "message #comment"
	307	'
	308
	309	test_expect_success 'cleanup commit message (whitespace option, -m)' '
	310	echo content >>file &&
	311	git add file &&
	312	git commit --cleanup=whitespace --no-status -m "message #comment " &&
	313	commit_msg_is "message #comment"
	314	'
	315
	316	test_expect_success 'cleanup commit message (whitespace config, -m)' '
	317	echo content >>file &&
	318	git add file &&
	319	git -c commit.cleanup=whitespace commit --no-status -m "message #comment " &&
	320	commit_msg_is "message #comment"
	321	'
	322
1f4bf345	323	test_expect_success 'message shows author when it is not equal to committer' '
e83dbe80	324	echo >>negative &&
ceacd91a	325	git commit -e -m "sample" -a &&
1f4bf345 JK	326	test_i18ngrep \
	327	"^# Author: *A U Thor <author@example.com>\$" \
	328	.git/COMMIT_EDITMSG
e83dbe80 SB	329	'
e83dbe80 SB	330
1d7dc264	331	test_expect_success AUTOIDENT 'message shows committer when it is automatic' '
bb1ae3f6 SB	332
bb1ae3f6 SB	333	echo >>negative &&
7845944c	334	(
00648ba0 EN	335	sane_unset GIT_COMMITTER_EMAIL &&
00648ba0 EN	336	sane_unset GIT_COMMITTER_NAME &&
1d7dc264	337	git commit -e -m "sample" -a
7845944c	338	) &&
1f4bf345 JK	339	# the ident is calculated from the system, so we cannot
	340	# check the actual value, only that it is there
	341	test_i18ngrep "^# Committer: " .git/COMMIT_EDITMSG
bb1ae3f6 SB	342	'
bb1ae3f6 SB	343
a9ebc43b JK	344	write_script .git/FAKE_EDITOR <<EOF
a9ebc43b JK	345	echo editor started > "$(pwd)/.git/result"
ec84bd00 PB	346	exit 0
ec84bd00 PB	347	EOF
ec84bd00	348
09feffb6	349	test_expect_success !AUTOIDENT 'do not fire editor when committer is bogus' '
8c8b3bc3 JK	350	>.git/result
	351	>expect &&
	352
	353	echo >>negative &&
	354	(
	355	sane_unset GIT_COMMITTER_EMAIL &&
	356	sane_unset GIT_COMMITTER_NAME &&
	357	GIT_EDITOR="\"$(pwd)/.git/FAKE_EDITOR\"" &&
	358	export GIT_EDITOR &&
	359	test_must_fail git commit -e -m sample -a
	360	) &&
	361	test_cmp expect .git/result
	362	'
	363
ec84bd00 PB	364	test_expect_success 'do not fire editor in the presence of conflicts' '
ec84bd00 PB	365
a3c91e08 JH	366	git clean -f &&
	367	echo f >g &&
	368	git add g &&
	369	git commit -m "add g" &&
	370	git branch second &&
	371	echo master >g &&
	372	echo g >h &&
	373	git add g h &&
	374	git commit -m "modify g and add h" &&
	375	git checkout second &&
	376	echo second >g &&
	377	git add g &&
	378	git commit -m second &&
	379	# Must fail due to conflict
	380	test_must_fail git cherry-pick -n master &&
	381	echo "editor not started" >.git/result &&
e2007832	382	(
34565f27	383	GIT_EDITOR="\"$(pwd)/.git/FAKE_EDITOR\"" &&
e2007832 BC	384	export GIT_EDITOR &&
	385	test_must_fail git commit
	386	) &&
a3c91e08	387	test "$(cat .git/result)" = "editor not started"
ec84bd00 PB	388	'
ec84bd00 PB	389
a9ebc43b	390	write_script .git/FAKE_EDITOR <<EOF
ad5fa3cc PB	391	# kill -TERM command added below.
	392	EOF
	393
fb9a2bea	394	test_expect_success EXECKEEPSPID 'a SIGTERM should break locks' '
ad5fa3cc	395	echo >>negative &&
09b78bc1	396	! "$SHELL_PATH" -c '\''
ad5fa3cc	397	echo kill -TERM $$ >> .git/FAKE_EDITOR
09b78bc1 BC	398	GIT_EDITOR=.git/FAKE_EDITOR
	399	export GIT_EDITOR
	400	exec git commit -a'\'' &&
	401	test ! -f .git/index.lock
ad5fa3cc PB	402	'
ad5fa3cc PB	403
67bfc030 JH	404	rm -f .git/MERGE_MSG .git/COMMIT_EDITMSG
	405	git reset -q --hard
	406
	407	test_expect_success 'Hand committing of a redundant merge removes dups' '
	408
	409	git rev-parse second master >expect &&
	410	test_must_fail git merge second master &&
	411	git checkout master g &&
	412	EDITOR=: git commit -a &&
	413	git cat-file commit HEAD \| sed -n -e "s/^parent //p" -e "/^$/q" >actual &&
	414	test_cmp expect actual
	415
	416	'
	417
e5138436 JH	418	test_expect_success 'A single-liner subject with a token plus colon is not a footer' '
	419
	420	git reset --hard &&
	421	git commit -s -m "hello: kitty" --allow-empty &&
	422	git cat-file commit HEAD \| sed -e "1,/^$/d" >actual &&
3fb0459b	423	test_line_count = 3 actual
e5138436 JH	424
	425	'
	426
8c613fd5 BC	427	test_expect_success 'commit -s places sob on third line after two empty lines' '
	428	git commit -s --allow-empty --allow-empty-message &&
	429	cat <<-EOF >expect &&
	430
	431
	432	Signed-off-by: $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL>
	433
	434	EOF
	435	sed -e "/^#/d" -e "s/^:.*//" .git/COMMIT_EDITMSG >actual &&
	436	test_cmp expect actual
	437	'
	438
a9ebc43b JK	439	write_script .git/FAKE_EDITOR <<\EOF
a9ebc43b JK	440	mv "$1" "$1.orig"
f9c01817 JH	441	(
f9c01817 JH	442	echo message
a9ebc43b JK	443	cat "$1.orig"
a9ebc43b JK	444	) >"$1"
f9c01817 JH	445	EOF
	446
	447	echo '## Custom template' >template
	448
f9c01817 JH	449	try_commit () {
	450	git reset --hard &&
	451	echo >>negative &&
	452	GIT_EDITOR=.git/FAKE_EDITOR git commit -a $* $use_template &&
	453	case "$use_template" in
	454	'')
f79ce8db	455	test_i18ngrep ! "^## Custom template" .git/COMMIT_EDITMSG ;;
f9c01817	456	*)
f79ce8db	457	test_i18ngrep "^## Custom template" .git/COMMIT_EDITMSG ;;
f9c01817 JH	458	esac
	459	}
	460
	461	try_commit_status_combo () {
	462
f79ce8db	463	test_expect_success 'commit' '
f9c01817	464	try_commit "" &&
f79ce8db	465	test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	466	'
f9c01817 JH	467
f79ce8db	468	test_expect_success 'commit' '
f9c01817	469	try_commit "" &&
f79ce8db	470	test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	471	'
f9c01817 JH	472
f79ce8db	473	test_expect_success 'commit --status' '
f9c01817	474	try_commit --status &&
f79ce8db	475	test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	476	'
f9c01817 JH	477
f79ce8db	478	test_expect_success 'commit --no-status' '
2dec68cf	479	try_commit --no-status &&
f79ce8db	480	test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	481	'
f9c01817 JH	482
f79ce8db	483	test_expect_success 'commit with commit.status = yes' '
e023a31d	484	test_config commit.status yes &&
f9c01817	485	try_commit "" &&
f79ce8db	486	test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	487	'
f9c01817 JH	488
f79ce8db	489	test_expect_success 'commit with commit.status = no' '
e023a31d	490	test_config commit.status no &&
f9c01817	491	try_commit "" &&
f79ce8db	492	test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	493	'
f9c01817 JH	494
f79ce8db	495	test_expect_success 'commit --status with commit.status = yes' '
e023a31d	496	test_config commit.status yes &&
f9c01817	497	try_commit --status &&
f79ce8db	498	test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	499	'
f9c01817 JH	500
f79ce8db	501	test_expect_success 'commit --no-status with commit.status = yes' '
e023a31d	502	test_config commit.status yes &&
f9c01817	503	try_commit --no-status &&
f79ce8db	504	test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	505	'
f9c01817 JH	506
f79ce8db	507	test_expect_success 'commit --status with commit.status = no' '
e023a31d	508	test_config commit.status no &&
f9c01817	509	try_commit --status &&
f79ce8db	510	test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	511	'
f9c01817 JH	512
f79ce8db	513	test_expect_success 'commit --no-status with commit.status = no' '
e023a31d	514	test_config commit.status no &&
f9c01817	515	try_commit --no-status &&
f79ce8db	516	test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
f9c01817 JH	517	'
	518
	519	}
	520
	521	try_commit_status_combo
	522
	523	use_template="-t template"
	524
	525	try_commit_status_combo
	526
eff80a9f	527	test_expect_success 'commit --status with custom comment character' '
464be630	528	test_config core.commentchar ";" &&
eff80a9f JH	529	try_commit --status &&
	530	test_i18ngrep "^; Changes to be committed:" .git/COMMIT_EDITMSG
	531	'
	532
b468f0ce	533	test_done