patches/km/any-notes-ref-tests.txt

   1 Subject: [PATCH] t/t3308-notes-merge.sh: succeed with relaxed notes refs
   2
   3 With the recent change to allow notes refs to be located in
   4 the refs hierarchy in locations other than refs/notes/ the
   5 'git notes merge refs/heads/master' test started succeeding.
   6
   7 Previously refs/heads/master would have been expanded to
   8 a non-existing, ref refs/notes/refs/heads/master, and the
   9 merge would have failed (as expected).
  10
  11 Now, however, since refs/heads/master exists and the new,
  12 more relaxed notes refs rules leave it unchanged, the merge
  13 succeeds.  This has a follow-on effect which makes the
  14 next two tests fail as well.
  15
  16 The refs/heads/master ref could just be replaced with
  17 another ref name that does not exist such as refs/heads/xmaster,
  18 but there are already several tests using non-existant refs
  19 so instead just remove the refs/heads/master line.
  20
  21 Suggested-by: Johan Herland <johan@herland.net>
  22 Signed-off-by: Kyle J. McKay <mackyle@gmail.com>
  23
  24 ---
  25  t/t3308-notes-merge.sh | 1 -
  26  1 file changed, 1 deletion(-)
  27
  28 diff --git a/t/t3308-notes-merge.sh b/t/t3308-notes-merge.sh
  29 index 24d82b49..f0feb64b 100755
  30 --- a/t/t3308-notes-merge.sh
  31 +++ b/t/t3308-notes-merge.sh
  32 @@ -90,7 +90,6 @@ test_expect_success 'fail to merge various non-note-trees' '
  33         test_must_fail git notes merge refs/notes/ &&
  34         test_must_fail git notes merge refs/notes/dir &&
  35         test_must_fail git notes merge refs/notes/dir/ &&
  36 -       test_must_fail git notes merge refs/heads/master &&
  37         test_must_fail git notes merge x: &&
  38         test_must_fail git notes merge x:foo &&
  39         test_must_fail git notes merge foo^{bar
  40 ---