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

[PATCH 1/2] t/t5319-multi-pack-index.sh: drop early 'test_done'

From
Taylor Blau <me@ttaylorr.com>
Date
Jan 12, 2026, 23:45 UTC
Message-ID
<9c5faa5932cdd9e570406bc85ba27f94195a4d3d.1768261435.git.me@ttaylorr.com>
In-Reply-To
<cover.1768261435.git.me@ttaylorr.com>

In 6ce9d558ced (midx-write: skip rewriting MIDX with `--stdin-packs` unless needed, 2025-12-10), an extra 'test_done' was added, causing the test script to finish before having run all of its tests.

Dropping this extraneous 'test_done' exposes a bug from commit 6ce9d558ced that causes a subsequent test to fail. Mark that test with a 'test_expect_failure' for now, and the subsequent commit will explain and fix the bug.

Signed-off-by: Taylor Blau <me@ttaylorr.com>
---
 t/t5319-multi-pack-index.sh | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/t/t5319-multi-pack-index.sh b/t/t5319-multi-pack-index.sh
index 794f8b5ab4e..b6622849db7 100755
--- a/t/t5319-multi-pack-index.sh
+++ b/t/t5319-multi-pack-index.sh
@@ -415,8 +415,6 @@ test_expect_success 'up-to-date multi-pack-index is retained' '
 	)
 '
 
-test_done
-
 test_expect_success 'verify multi-pack-index success' '
 	git multi-pack-index verify --object-dir=$objdir
 '
@@ -565,7 +563,7 @@ test_expect_success 'git fsck suppresses MIDX output with --no-progress' '
 	! grep "Verifying object offsets" err
 '
 
-test_expect_success 'corrupt MIDX is not reused' '
+test_expect_failure 'corrupt MIDX is not reused' '
 	corrupt_midx_and_verify $MIDX_BYTE_OFFSET "\377" $objdir \
 		"incorrect object offset" &&
 	git multi-pack-index write 2>err &&
-- 
2.52.0.437.gcc6f76a88cd
Previous: Taylor BlauNext: Patrick Steinhardt
Message 2 of 5 in “midx-write.c: do not optimize out writes with corrupt MIDXs”
  1. 0/2 midx-write.c: do not optimize out writes with corrupt MIDXsTaylor Blau, Jan 12, 2026
  2. 1/2 t/t5319-multi-pack-index.sh: drop early 'test_done'Taylor Blau, Jan 12, 2026
  3. Patrick SteinhardtJan 13, 2026
  4. 2/2 midx-write.c: assume checksum-invalid MIDXs require an updateTaylor Blau, Jan 12, 2026
  5. Patrick SteinhardtJan 13, 2026

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.