git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[StGit PATCH] Remove broken branch creation subtest

From
Karl Hasselström <kha@treskal.com>
Date
Apr 12, 2008, 12:44 UTC
Message-ID
<20080412124212.27748.21332.stgit@yoghurt>

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 <kha@treskal.com>
---

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 &&
Next: Junio C Hamano
Message 1 of 4 in “Remove broken branch creation subtest”
  1. Remove broken branch creation subtestKarl Hasselström, Apr 12, 2008
  2. Junio C HamanoApr 12, 2008
  3. Karl HasselströmApr 13, 2008
  4. Fix problems in t1000-branch-createKarl Hasselström, Apr 13, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.