DORMANTno replies

[StGit PATCH] Remove broken branch creation subtest

From: Karl Hasselström <hidden>
Date: 2016-06-15 22:44:28
Subsystem: the rest · Maintainer: Linus Torvalds

This subtest has started to cause subsequent subtests to fail with
recent versions of git. And I don't think we can blame this one on
git. What the subtest does is:

  1. Remove all files or directories called "foo" under .git/. This is
     supposed to delete the "foo" branch and associated StGit files,
     but what about packed refs? This isn't actually malfunctioning
     yet as far as I can tell, but it's a ticking bomb.

  2. Create an empty file .git/refs/heads/foo. This is supposed to be
     a "broken branch", and indeed it is -- for example, git show-ref
     barfs on such a repository even if asked to only show a branch
     other than foo!

  3. Makes sure that stg branch won't successfully create a "foo"
     branch. I'm pretty sure this fails because git thinks the repo is
     broken, not because stg handles it gracefully. This is what the
     test is supposed to be testing, but if we wanted that, we'd need
     a more detailed test.

  4. Doesn't clean up the broken ref, which causes some subsequent
     subtests to fail.

What probably happened is that git got ever so slightly fussier about
broken refs, so that (4) became a problem.

Signed-off-by: Karl Hasselström <redacted>

---

This should go to the stable branch. (master is affected too, but a
merge will fix that.)

 t/t1000-branch-create.sh |    7 -------
 1 files changed, 0 insertions(+), 7 deletions(-)

diff --git a/t/t1000-branch-create.sh b/t/t1000-branch-create.sh
index d6cf34a..298eb1a 100755
--- a/t/t1000-branch-create.sh
+++ b/t/t1000-branch-create.sh
@@ -54,13 +54,6 @@ test_expect_success \
 '
 
 test_expect_success \
-    'Create an invalid refs/heads/ entry' '
-    find .git -name foo | xargs rm -rf &&
-    touch .git/refs/heads/foo &&
-    ! stg branch -c foo
-'
-
-test_expect_success \
     'Setup two commits including removal of generated files' '
     git init &&
     touch a.c a.o &&
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help