df458ca9391e9a6fada477a630cb077902b0c1ae
   1#!/bin/sh
   2#
   3# Copyright (c) 2007 Johannes E. Schindelin
   4#
   5
   6test_description='Test commit notes'
   7
   8. ./test-lib.sh
   9
  10cat > fake_editor.sh << \EOF
  11echo "$MSG" > "$1"
  12echo "$MSG" >& 2
  13EOF
  14chmod a+x fake_editor.sh
  15GIT_EDITOR=./fake_editor.sh
  16export GIT_EDITOR
  17
  18test_expect_success 'cannot annotate non-existing HEAD' '
  19        (MSG=3 && export MSG && test_must_fail git notes add)
  20'
  21
  22test_expect_success setup '
  23        : > a1 &&
  24        git add a1 &&
  25        test_tick &&
  26        git commit -m 1st &&
  27        : > a2 &&
  28        git add a2 &&
  29        test_tick &&
  30        git commit -m 2nd
  31'
  32
  33test_expect_success 'need valid notes ref' '
  34        (MSG=1 GIT_NOTES_REF=/ && export MSG GIT_NOTES_REF &&
  35         test_must_fail git notes add) &&
  36        (MSG=2 GIT_NOTES_REF=/ && export MSG GIT_NOTES_REF &&
  37         test_must_fail git notes show)
  38'
  39
  40test_expect_success 'refusing to add notes in refs/heads/' '
  41        (MSG=1 GIT_NOTES_REF=refs/heads/bogus &&
  42         export MSG GIT_NOTES_REF &&
  43         test_must_fail git notes add)
  44'
  45
  46test_expect_success 'refusing to edit notes in refs/remotes/' '
  47        (MSG=1 GIT_NOTES_REF=refs/remotes/bogus &&
  48         export MSG GIT_NOTES_REF &&
  49         test_must_fail git notes edit)
  50'
  51
  52# 1 indicates caught gracefully by die, 128 means git-show barked
  53test_expect_success 'handle empty notes gracefully' '
  54        git notes show ; test 1 = $?
  55'
  56
  57test_expect_success 'create notes' '
  58        git config core.notesRef refs/notes/commits &&
  59        MSG=b4 git notes add &&
  60        test ! -f .git/NOTES_EDITMSG &&
  61        test 1 = $(git ls-tree refs/notes/commits | wc -l) &&
  62        test b4 = $(git notes show) &&
  63        git show HEAD^ &&
  64        test_must_fail git notes show HEAD^
  65'
  66
  67test_expect_success 'edit existing notes' '
  68        MSG=b3 git notes edit &&
  69        test ! -f .git/NOTES_EDITMSG &&
  70        test 1 = $(git ls-tree refs/notes/commits | wc -l) &&
  71        test b3 = $(git notes show) &&
  72        git show HEAD^ &&
  73        test_must_fail git notes show HEAD^
  74'
  75
  76test_expect_success 'cannot add note where one exists' '
  77        ! MSG=b2 git notes add &&
  78        test ! -f .git/NOTES_EDITMSG &&
  79        test 1 = $(git ls-tree refs/notes/commits | wc -l) &&
  80        test b3 = $(git notes show) &&
  81        git show HEAD^ &&
  82        test_must_fail git notes show HEAD^
  83'
  84
  85test_expect_success 'can overwrite existing note with "git notes add -f"' '
  86        MSG=b1 git notes add -f &&
  87        test ! -f .git/NOTES_EDITMSG &&
  88        test 1 = $(git ls-tree refs/notes/commits | wc -l) &&
  89        test b1 = $(git notes show) &&
  90        git show HEAD^ &&
  91        test_must_fail git notes show HEAD^
  92'
  93
  94cat > expect << EOF
  95commit 268048bfb8a1fb38e703baceb8ab235421bf80c5
  96Author: A U Thor <author@example.com>
  97Date:   Thu Apr 7 15:14:13 2005 -0700
  98
  99    2nd
 100
 101Notes:
 102    b1
 103EOF
 104
 105test_expect_success 'show notes' '
 106        ! (git cat-file commit HEAD | grep b1) &&
 107        git log -1 > output &&
 108        test_cmp expect output
 109'
 110
 111test_expect_success 'create multi-line notes (setup)' '
 112        : > a3 &&
 113        git add a3 &&
 114        test_tick &&
 115        git commit -m 3rd &&
 116        MSG="b3
 117c3c3c3c3
 118d3d3d3" git notes add
 119'
 120
 121cat > expect-multiline << EOF
 122commit 1584215f1d29c65e99c6c6848626553fdd07fd75
 123Author: A U Thor <author@example.com>
 124Date:   Thu Apr 7 15:15:13 2005 -0700
 125
 126    3rd
 127
 128Notes:
 129    b3
 130    c3c3c3c3
 131    d3d3d3
 132EOF
 133
 134printf "\n" >> expect-multiline
 135cat expect >> expect-multiline
 136
 137test_expect_success 'show multi-line notes' '
 138        git log -2 > output &&
 139        test_cmp expect-multiline output
 140'
 141test_expect_success 'create -F notes (setup)' '
 142        : > a4 &&
 143        git add a4 &&
 144        test_tick &&
 145        git commit -m 4th &&
 146        echo "xyzzy" > note5 &&
 147        git notes add -F note5
 148'
 149
 150cat > expect-F << EOF
 151commit 15023535574ded8b1a89052b32673f84cf9582b8
 152Author: A U Thor <author@example.com>
 153Date:   Thu Apr 7 15:16:13 2005 -0700
 154
 155    4th
 156
 157Notes:
 158    xyzzy
 159EOF
 160
 161printf "\n" >> expect-F
 162cat expect-multiline >> expect-F
 163
 164test_expect_success 'show -F notes' '
 165        git log -3 > output &&
 166        test_cmp expect-F output
 167'
 168
 169cat >expect << EOF
 170commit 15023535574ded8b1a89052b32673f84cf9582b8
 171tree e070e3af51011e47b183c33adf9736736a525709
 172parent 1584215f1d29c65e99c6c6848626553fdd07fd75
 173author A U Thor <author@example.com> 1112912173 -0700
 174committer C O Mitter <committer@example.com> 1112912173 -0700
 175
 176    4th
 177EOF
 178test_expect_success 'git log --pretty=raw does not show notes' '
 179        git log -1 --pretty=raw >output &&
 180        test_cmp expect output
 181'
 182
 183cat >>expect <<EOF
 184
 185Notes:
 186    xyzzy
 187EOF
 188test_expect_success 'git log --show-notes' '
 189        git log -1 --pretty=raw --show-notes >output &&
 190        test_cmp expect output
 191'
 192
 193test_expect_success 'git log --no-notes' '
 194        git log -1 --no-notes >output &&
 195        ! grep xyzzy output
 196'
 197
 198test_expect_success 'git format-patch does not show notes' '
 199        git format-patch -1 --stdout >output &&
 200        ! grep xyzzy output
 201'
 202
 203test_expect_success 'git format-patch --show-notes does show notes' '
 204        git format-patch --show-notes -1 --stdout >output &&
 205        grep xyzzy output
 206'
 207
 208for pretty in \
 209        "" --pretty --pretty=raw --pretty=short --pretty=medium \
 210        --pretty=full --pretty=fuller --pretty=format:%s --oneline
 211do
 212        case "$pretty" in
 213        "") p= not= negate="" ;;
 214        ?*) p="$pretty" not=" not" negate="!" ;;
 215        esac
 216        test_expect_success "git show $pretty does$not show notes" '
 217                git show $p >output &&
 218                eval "$negate grep xyzzy output"
 219        '
 220done
 221
 222test_expect_success 'create -m notes (setup)' '
 223        : > a5 &&
 224        git add a5 &&
 225        test_tick &&
 226        git commit -m 5th &&
 227        git notes add -m spam -m "foo
 228bar
 229baz"
 230'
 231
 232whitespace="    "
 233cat > expect-m << EOF
 234commit bd1753200303d0a0344be813e504253b3d98e74d
 235Author: A U Thor <author@example.com>
 236Date:   Thu Apr 7 15:17:13 2005 -0700
 237
 238    5th
 239
 240Notes:
 241    spam
 242$whitespace
 243    foo
 244    bar
 245    baz
 246EOF
 247
 248printf "\n" >> expect-m
 249cat expect-F >> expect-m
 250
 251test_expect_success 'show -m notes' '
 252        git log -4 > output &&
 253        test_cmp expect-m output
 254'
 255
 256test_expect_success 'remove note with add -f -F /dev/null (setup)' '
 257        git notes add -f -F /dev/null
 258'
 259
 260cat > expect-rm-F << EOF
 261commit bd1753200303d0a0344be813e504253b3d98e74d
 262Author: A U Thor <author@example.com>
 263Date:   Thu Apr 7 15:17:13 2005 -0700
 264
 265    5th
 266EOF
 267
 268printf "\n" >> expect-rm-F
 269cat expect-F >> expect-rm-F
 270
 271test_expect_success 'verify note removal with -F /dev/null' '
 272        git log -4 > output &&
 273        test_cmp expect-rm-F output &&
 274        ! git notes show
 275'
 276
 277test_expect_success 'do not create empty note with -m "" (setup)' '
 278        git notes add -m ""
 279'
 280
 281test_expect_success 'verify non-creation of note with -m ""' '
 282        git log -4 > output &&
 283        test_cmp expect-rm-F output &&
 284        ! git notes show
 285'
 286
 287test_expect_success 'remove note with "git notes remove" (setup)' '
 288        git notes remove HEAD^
 289'
 290
 291cat > expect-rm-remove << EOF
 292commit bd1753200303d0a0344be813e504253b3d98e74d
 293Author: A U Thor <author@example.com>
 294Date:   Thu Apr 7 15:17:13 2005 -0700
 295
 296    5th
 297
 298commit 15023535574ded8b1a89052b32673f84cf9582b8
 299Author: A U Thor <author@example.com>
 300Date:   Thu Apr 7 15:16:13 2005 -0700
 301
 302    4th
 303EOF
 304
 305printf "\n" >> expect-rm-remove
 306cat expect-multiline >> expect-rm-remove
 307
 308test_expect_success 'verify note removal with "git notes remove"' '
 309        git log -4 > output &&
 310        test_cmp expect-rm-remove output &&
 311        ! git notes show HEAD^
 312'
 313
 314cat > expect << EOF
 315c18dc024e14f08d18d14eea0d747ff692d66d6a3 1584215f1d29c65e99c6c6848626553fdd07fd75
 316c9c6af7f78bc47490dbf3e822cf2f3c24d4b9061 268048bfb8a1fb38e703baceb8ab235421bf80c5
 317EOF
 318
 319test_expect_success 'list notes with "git notes list"' '
 320        git notes list > output &&
 321        test_cmp expect output
 322'
 323
 324test_expect_success 'list notes with "git notes"' '
 325        git notes > output &&
 326        test_cmp expect output
 327'
 328
 329cat > expect << EOF
 330c18dc024e14f08d18d14eea0d747ff692d66d6a3
 331EOF
 332
 333test_expect_success 'list specific note with "git notes list <object>"' '
 334        git notes list HEAD^^ > output &&
 335        test_cmp expect output
 336'
 337
 338cat > expect << EOF
 339EOF
 340
 341test_expect_success 'listing non-existing notes fails' '
 342        test_must_fail git notes list HEAD > output &&
 343        test_cmp expect output
 344'
 345
 346test_expect_success 'create other note on a different notes ref (setup)' '
 347        : > a6 &&
 348        git add a6 &&
 349        test_tick &&
 350        git commit -m 6th &&
 351        GIT_NOTES_REF="refs/notes/other" git notes add -m "other note"
 352'
 353
 354cat > expect-other << EOF
 355commit 387a89921c73d7ed72cd94d179c1c7048ca47756
 356Author: A U Thor <author@example.com>
 357Date:   Thu Apr 7 15:18:13 2005 -0700
 358
 359    6th
 360
 361Notes:
 362    other note
 363EOF
 364
 365cat > expect-not-other << EOF
 366commit 387a89921c73d7ed72cd94d179c1c7048ca47756
 367Author: A U Thor <author@example.com>
 368Date:   Thu Apr 7 15:18:13 2005 -0700
 369
 370    6th
 371EOF
 372
 373test_expect_success 'Do not show note on other ref by default' '
 374        git log -1 > output &&
 375        test_cmp expect-not-other output
 376'
 377
 378test_expect_success 'Do show note when ref is given in GIT_NOTES_REF' '
 379        GIT_NOTES_REF="refs/notes/other" git log -1 > output &&
 380        test_cmp expect-other output
 381'
 382
 383test_expect_success 'Do show note when ref is given in core.notesRef config' '
 384        git config core.notesRef "refs/notes/other" &&
 385        git log -1 > output &&
 386        test_cmp expect-other output
 387'
 388
 389test_expect_success 'Do not show note when core.notesRef is overridden' '
 390        GIT_NOTES_REF="refs/notes/wrong" git log -1 > output &&
 391        test_cmp expect-not-other output
 392'
 393
 394test_expect_success 'Allow notes on non-commits (trees, blobs, tags)' '
 395        echo "Note on a tree" > expect
 396        git notes add -m "Note on a tree" HEAD: &&
 397        git notes show HEAD: > actual &&
 398        test_cmp expect actual &&
 399        echo "Note on a blob" > expect
 400        filename=$(git ls-tree --name-only HEAD | head -n1) &&
 401        git notes add -m "Note on a blob" HEAD:$filename &&
 402        git notes show HEAD:$filename > actual &&
 403        test_cmp expect actual &&
 404        echo "Note on a tag" > expect
 405        git tag -a -m "This is an annotated tag" foobar HEAD^ &&
 406        git notes add -m "Note on a tag" foobar &&
 407        git notes show foobar > actual &&
 408        test_cmp expect actual
 409'
 410
 411test_done