t7502: narrow checks for author/committer name in template
[git/jnareb-git.git] / t / t7502-commit.sh
blobefecb060f32b6aca1204bee63ddc023ffa0f9f86
1 #!/bin/sh
3 test_description='git commit porcelain-ish'
5 . ./test-lib.sh
7 # Arguments: [<prefix] [<commit message>] [<commit options>]
8 check_summary_oneline() {
9 test_tick &&
10 git commit ${3+"$3"} -m "$2" | head -1 > act &&
12 # branch name
13 SUMMARY_PREFIX="$(git name-rev --name-only HEAD)" &&
15 # append the "special" prefix, like "root-commit", "detached HEAD"
16 if test -n "$1"
17 then
18 SUMMARY_PREFIX="$SUMMARY_PREFIX ($1)"
21 # abbrev SHA-1
22 SUMMARY_POSTFIX="$(git log -1 --pretty='format:%h')"
23 echo "[$SUMMARY_PREFIX $SUMMARY_POSTFIX] $2" >exp &&
25 test_i18ncmp exp act
28 test_expect_success 'output summary format' '
30 echo new >file1 &&
31 git add file1 &&
32 check_summary_oneline "root-commit" "initial" &&
34 echo change >>file1 &&
35 git add file1
38 test_expect_success 'output summary format: root-commit' '
39 check_summary_oneline "" "a change"
42 test_expect_success 'output summary format for commit with an empty diff' '
44 check_summary_oneline "" "empty" "--allow-empty"
47 test_expect_success 'output summary format for merges' '
49 git checkout -b recursive-base &&
50 test_commit base file1 &&
52 git checkout -b recursive-a recursive-base &&
53 test_commit commit-a file1 &&
55 git checkout -b recursive-b recursive-base &&
56 test_commit commit-b file1 &&
58 # conflict
59 git checkout recursive-a &&
60 test_must_fail git merge recursive-b &&
61 # resolve the conflict
62 echo commit-a > file1 &&
63 git add file1 &&
64 check_summary_oneline "" "Merge"
67 output_tests_cleanup() {
68 # this is needed for "do not fire editor in the presence of conflicts"
69 git checkout master &&
71 # this is needed for the "partial removal" test to pass
72 git rm file1 &&
73 git commit -m "cleanup"
76 test_expect_success 'the basics' '
78 output_tests_cleanup &&
80 echo doing partial >"commit is" &&
81 mkdir not &&
82 echo very much encouraged but we should >not/forbid &&
83 git add "commit is" not &&
84 echo update added "commit is" file >"commit is" &&
85 echo also update another >not/forbid &&
86 test_tick &&
87 git commit -a -m "initial with -a" &&
89 git cat-file blob HEAD:"commit is" >current.1 &&
90 git cat-file blob HEAD:not/forbid >current.2 &&
92 cmp current.1 "commit is" &&
93 cmp current.2 not/forbid
97 test_expect_success 'partial' '
99 echo another >"commit is" &&
100 echo another >not/forbid &&
101 test_tick &&
102 git commit -m "partial commit to handle a file" "commit is" &&
104 changed=$(git diff-tree --name-only HEAD^ HEAD) &&
105 test "$changed" = "commit is"
109 test_expect_success 'partial modification in a subdirectory' '
111 test_tick &&
112 git commit -m "partial commit to subdirectory" not &&
114 changed=$(git diff-tree -r --name-only HEAD^ HEAD) &&
115 test "$changed" = "not/forbid"
119 test_expect_success 'partial removal' '
121 git rm not/forbid &&
122 git commit -m "partial commit to remove not/forbid" not &&
124 changed=$(git diff-tree -r --name-only HEAD^ HEAD) &&
125 test "$changed" = "not/forbid" &&
126 remain=$(git ls-tree -r --name-only HEAD) &&
127 test "$remain" = "commit is"
131 test_expect_success 'sign off' '
133 >positive &&
134 git add positive &&
135 git commit -s -m "thank you" &&
136 actual=$(git cat-file commit HEAD | sed -ne "s/Signed-off-by: //p") &&
137 expected=$(git var GIT_COMMITTER_IDENT | sed -e "s/>.*/>/") &&
138 test "z$actual" = "z$expected"
142 test_expect_success 'multiple -m' '
144 >negative &&
145 git add negative &&
146 git commit -m "one" -m "two" -m "three" &&
147 actual=$(git cat-file commit HEAD | sed -e "1,/^\$/d") &&
148 expected=$(echo one; echo; echo two; echo; echo three) &&
149 test "z$actual" = "z$expected"
153 test_expect_success 'verbose' '
155 echo minus >negative &&
156 git add negative &&
157 git status -v | sed -ne "/^diff --git /p" >actual &&
158 echo "diff --git a/negative b/negative" >expect &&
159 test_cmp expect actual
163 test_expect_success 'verbose respects diff config' '
165 git config color.diff always &&
166 git status -v >actual &&
167 grep "\[1mdiff --git" actual &&
168 git config --unset color.diff
171 test_expect_success 'cleanup commit messages (verbatim,-t)' '
173 echo >>negative &&
174 { echo;echo "# text";echo; } >expect &&
175 git commit --cleanup=verbatim -t expect -a &&
176 git cat-file -p HEAD |sed -e "1,/^\$/d" |head -n 3 >actual &&
177 test_cmp expect actual
181 test_expect_success 'cleanup commit messages (verbatim,-F)' '
183 echo >>negative &&
184 git commit --cleanup=verbatim -F expect -a &&
185 git cat-file -p HEAD |sed -e "1,/^\$/d">actual &&
186 test_cmp expect actual
190 test_expect_success 'cleanup commit messages (verbatim,-m)' '
192 echo >>negative &&
193 git commit --cleanup=verbatim -m "$(cat expect)" -a &&
194 git cat-file -p HEAD |sed -e "1,/^\$/d">actual &&
195 test_cmp expect actual
199 test_expect_success 'cleanup commit messages (whitespace,-F)' '
201 echo >>negative &&
202 { echo;echo "# text";echo; } >text &&
203 echo "# text" >expect &&
204 git commit --cleanup=whitespace -F text -a &&
205 git cat-file -p HEAD |sed -e "1,/^\$/d">actual &&
206 test_cmp expect actual
210 test_expect_success 'cleanup commit messages (strip,-F)' '
212 echo >>negative &&
213 { echo;echo "# text";echo sample;echo; } >text &&
214 echo sample >expect &&
215 git commit --cleanup=strip -F text -a &&
216 git cat-file -p HEAD |sed -e "1,/^\$/d">actual &&
217 test_cmp expect actual
221 test_expect_success 'cleanup commit messages (strip,-F,-e)' '
223 echo >>negative &&
224 { echo;echo sample;echo; } >text &&
225 git commit -e -F text -a &&
226 head -n 4 .git/COMMIT_EDITMSG >actual
229 echo "sample
231 # Please enter the commit message for your changes. Lines starting
232 # with '#' will be ignored, and an empty message aborts the commit." >expect
234 test_expect_success 'cleanup commit messages (strip,-F,-e): output' '
235 test_i18ncmp expect actual
238 test_expect_success 'message shows author when it is not equal to committer' '
239 echo >>negative &&
240 test_might_fail git commit -e -m "sample" &&
241 test_i18ngrep \
242 "^# Author: *A U Thor <author@example.com>\$" \
243 .git/COMMIT_EDITMSG
246 test_expect_success 'message shows committer when it is automatic' '
248 echo >>negative &&
250 sane_unset GIT_COMMITTER_EMAIL &&
251 sane_unset GIT_COMMITTER_NAME &&
252 # must fail because there is no change
253 test_must_fail git commit -e -m "sample"
254 ) &&
255 # the ident is calculated from the system, so we cannot
256 # check the actual value, only that it is there
257 test_i18ngrep "^# Committer: " .git/COMMIT_EDITMSG
260 write_script .git/FAKE_EDITOR <<EOF
261 echo editor started > "$(pwd)/.git/result"
262 exit 0
265 test_expect_success 'do not fire editor in the presence of conflicts' '
267 git clean -f &&
268 echo f >g &&
269 git add g &&
270 git commit -m "add g" &&
271 git branch second &&
272 echo master >g &&
273 echo g >h &&
274 git add g h &&
275 git commit -m "modify g and add h" &&
276 git checkout second &&
277 echo second >g &&
278 git add g &&
279 git commit -m second &&
280 # Must fail due to conflict
281 test_must_fail git cherry-pick -n master &&
282 echo "editor not started" >.git/result &&
284 GIT_EDITOR="\"$(pwd)/.git/FAKE_EDITOR\"" &&
285 export GIT_EDITOR &&
286 test_must_fail git commit
287 ) &&
288 test "$(cat .git/result)" = "editor not started"
291 write_script .git/FAKE_EDITOR <<EOF
292 # kill -TERM command added below.
295 test_expect_success EXECKEEPSPID 'a SIGTERM should break locks' '
296 echo >>negative &&
297 ! "$SHELL_PATH" -c '\''
298 echo kill -TERM $$ >> .git/FAKE_EDITOR
299 GIT_EDITOR=.git/FAKE_EDITOR
300 export GIT_EDITOR
301 exec git commit -a'\'' &&
302 test ! -f .git/index.lock
305 rm -f .git/MERGE_MSG .git/COMMIT_EDITMSG
306 git reset -q --hard
308 test_expect_success 'Hand committing of a redundant merge removes dups' '
310 git rev-parse second master >expect &&
311 test_must_fail git merge second master &&
312 git checkout master g &&
313 EDITOR=: git commit -a &&
314 git cat-file commit HEAD | sed -n -e "s/^parent //p" -e "/^$/q" >actual &&
315 test_cmp expect actual
319 test_expect_success 'A single-liner subject with a token plus colon is not a footer' '
321 git reset --hard &&
322 git commit -s -m "hello: kitty" --allow-empty &&
323 git cat-file commit HEAD | sed -e "1,/^$/d" >actual &&
324 test_line_count = 3 actual
328 write_script .git/FAKE_EDITOR <<\EOF
329 mv "$1" "$1.orig"
331 echo message
332 cat "$1.orig"
333 ) >"$1"
336 echo '## Custom template' >template
338 clear_config () {
340 git config --unset-all "$1"
341 case $? in
342 0|5) exit 0 ;;
343 *) exit 1 ;;
344 esac
348 try_commit () {
349 git reset --hard &&
350 echo >>negative &&
351 GIT_EDITOR=.git/FAKE_EDITOR git commit -a $* $use_template &&
352 case "$use_template" in
354 test_i18ngrep ! "^## Custom template" .git/COMMIT_EDITMSG ;;
356 test_i18ngrep "^## Custom template" .git/COMMIT_EDITMSG ;;
357 esac
360 try_commit_status_combo () {
362 test_expect_success 'commit' '
363 clear_config commit.status &&
364 try_commit "" &&
365 test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
368 test_expect_success 'commit' '
369 clear_config commit.status &&
370 try_commit "" &&
371 test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
374 test_expect_success 'commit --status' '
375 clear_config commit.status &&
376 try_commit --status &&
377 test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
380 test_expect_success 'commit --no-status' '
381 clear_config commit.status &&
382 try_commit --no-status &&
383 test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
386 test_expect_success 'commit with commit.status = yes' '
387 clear_config commit.status &&
388 git config commit.status yes &&
389 try_commit "" &&
390 test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
393 test_expect_success 'commit with commit.status = no' '
394 clear_config commit.status &&
395 git config commit.status no &&
396 try_commit "" &&
397 test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
400 test_expect_success 'commit --status with commit.status = yes' '
401 clear_config commit.status &&
402 git config commit.status yes &&
403 try_commit --status &&
404 test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
407 test_expect_success 'commit --no-status with commit.status = yes' '
408 clear_config commit.status &&
409 git config commit.status yes &&
410 try_commit --no-status &&
411 test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
414 test_expect_success 'commit --status with commit.status = no' '
415 clear_config commit.status &&
416 git config commit.status no &&
417 try_commit --status &&
418 test_i18ngrep "^# Changes to be committed:" .git/COMMIT_EDITMSG
421 test_expect_success 'commit --no-status with commit.status = no' '
422 clear_config commit.status &&
423 git config commit.status no &&
424 try_commit --no-status &&
425 test_i18ngrep ! "^# Changes to be committed:" .git/COMMIT_EDITMSG
430 try_commit_status_combo
432 use_template="-t template"
434 try_commit_status_combo
436 test_done