t7502-commit: add tests for summary output
Signed-off-by: Tay Ray Chuan <rctay89@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>maint
parent
c197702156
commit
fc6fa0d0f3
|
@ -4,8 +4,51 @@ test_description='git commit porcelain-ish'
|
||||||
|
|
||||||
. ./test-lib.sh
|
. ./test-lib.sh
|
||||||
|
|
||||||
|
# Arguments: [<prefix] [<commit message>]
|
||||||
|
check_summary_oneline() {
|
||||||
|
test_tick &&
|
||||||
|
git commit -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_cmp 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 &&
|
||||||
|
check_summary_oneline "" "a change"
|
||||||
|
'
|
||||||
|
|
||||||
|
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' '
|
test_expect_success 'the basics' '
|
||||||
|
|
||||||
|
output_tests_cleanup &&
|
||||||
|
|
||||||
echo doing partial >"commit is" &&
|
echo doing partial >"commit is" &&
|
||||||
mkdir not &&
|
mkdir not &&
|
||||||
echo very much encouraged but we should >not/forbid &&
|
echo very much encouraged but we should >not/forbid &&
|
||||||
|
|
Loading…
Reference in New Issue