From: Tor Arne Vestbø Date: Wed, 14 Jan 2009 16:28:11 GMT Subject: [PATCH next v2] git-notes: add test case for multi-line notes Message-ID: <496E129B.3020502@trolltech.com> In-Reply-To: The tests adds a third commit with a multi-line note. The output of git log -2 is then checked to see if the note lines are wrapped correctly, and that there's a line separator between the two commits. Signed-off-by: Tor Arne Vestbø --- Thanks for the feedback Johannes! Here's an updated patch. I removed the blank line instead of adding another, as that's the current style of that file. t/t3301-notes.sh | 35 ++++++++++++++++++++++++++++++++--- 1 files changed, 32 insertions(+), 3 deletions(-) diff --git a/t/t3301-notes.sh b/t/t3301-notes.sh index ba42c45..e260d79 100755 --- a/t/t3301-notes.sh +++ b/t/t3301-notes.sh @@ -8,8 +8,9 @@ test_description='Test commit notes' . ./test-lib.sh cat > fake_editor.sh << \EOF -echo "$MSG" > "$1" -echo "$MSG" >& 2 +MSG=${MSG//%/} +printf "$MSG" > "$1" +printf "$MSG" >& 2 EOF chmod a+x fake_editor.sh VISUAL=./fake_editor.sh @@ -59,7 +60,35 @@ EOF test_expect_success 'show notes' ' ! (git cat-file commit HEAD | grep b1) && git log -1 > output && - git diff expect output + test_cmp expect output +' +test_expect_success 'create multi-line notes (setup)' ' + : > a3 && + git add a3 && + test_tick && + git commit -m 3rd && + MSG="b3\nc3c3c3c3\nd3d3d3" git notes edit +' + +cat > expect-multiline << EOF +commit 1584215f1d29c65e99c6c6848626553fdd07fd75 +Author: A U Thor +Date: Thu Apr 7 15:15:13 2005 -0700 + + 3rd + +Notes: + b3 + c3c3c3c3 + d3d3d3 +EOF + +printf "\n" >> expect-multiline +cat expect >> expect-multiline + +test_expect_success 'show multi-line notes' ' + git log -2 > output && + test_cmp expect-multiline output ' test_done -- 1.6.0.2.GIT