[PATCH 1/5] t/t5332-multi-pack-reuse.sh: verify pack generation with --strict
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Aug 27, 2024, 21:13 UTC
- Message-ID
- <160673231c55f51c22171a8c2fbb105f7a501083.1724793201.git.me@ttaylorr.com>
- In-Reply-To
- <cover.1724793201.git.me@ttaylorr.com>
In our tests for multi-pack reuse, we have two helper functions:
- test_pack_objects_reused_all(), and - test_pack_objects_reused()
which invoke pack-objects (either with `--all`, or the supplied tips via stdin, respectively) and ensure that (a) the number of reused objects, and (b) the number of packs which those objects were reused from both match the expected values.
Both functions discard the output of pack-objects and assert only on the contents of the trace2 stream.
However, if we store the pack and attempt to index it with `--strict`, we find that a number of our tests are broken, indicating a bug within multi-pack reuse.
That bug will be addressed in a subsequent commit. But let's first harden these tests by trying to index the resulting pack, marking the tests which fail appropriately.
Signed-off-by: Taylor Blau <me@ttaylorr.com> --- t/t5332-multi-pack-reuse.sh | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-)
diff --git a/t/t5332-multi-pack-reuse.sh b/t/t5332-multi-pack-reuse.sh index 941e73d354..ba888a83d5 100755 --- a/t/t5332-multi-pack-reuse.sh +++ b/t/t5332-multi-pack-reuse.sh @@ -31,20 +31,24 @@ test_pack_objects_reused_all () { : >trace2.txt && GIT_TRACE2_EVENT="$PWD/trace2.txt" \ git pack-objects --stdout --revs --all --delta-base-offset \ - >/dev/null && + >got.pack && test_pack_reused "$1" <trace2.txt && - test_packs_reused "$2" <trace2.txt + test_packs_reused "$2" <trace2.txt && + + git index-pack --strict -o got.idx got.pack } # test_pack_objects_reused <pack-reused> <packs-reused> test_pack_objects_reused () { : >trace2.txt && GIT_TRACE2_EVENT="$PWD/trace2.txt" \ - git pack-objects --stdout --revs >/dev/null && + git pack-objects --stdout --revs >got.pack && test_pack_reused "$1" <trace2.txt && - test_packs_reused "$2" <trace2.txt + test_packs_reused "$2" <trace2.txt && + + git index-pack --strict -o got.idx got.pack } test_expect_success 'preferred pack is reused for single-pack reuse' ' @@ -65,7 +69,7 @@ test_expect_success 'multi-pack reuse is disabled by default' ' test_pack_objects_reused_all 3 1 ' -test_expect_success 'feature.experimental implies multi-pack reuse' ' +test_expect_failure 'feature.experimental implies multi-pack reuse' ' test_config feature.experimental true && test_pack_objects_reused_all 6 2 @@ -82,7 +86,7 @@ test_expect_success 'enable multi-pack reuse' ' git config pack.allowPackReuse multi ' -test_expect_success 'reuse all objects from subset of bitmapped packs' ' +test_expect_failure 'reuse all objects from subset of bitmapped packs' ' test_commit C && git repack -d && @@ -96,7 +100,7 @@ test_expect_success 'reuse all objects from subset of bitmapped packs' ' test_pack_objects_reused 6 2 <in ' -test_expect_success 'reuse all objects from all packs' ' +test_expect_failure 'reuse all objects from all packs' ' test_pack_objects_reused_all 9 3 ' @@ -190,7 +194,7 @@ test_expect_success 'omit delta with uninteresting base (same pack)' ' test_pack_objects_reused 3 1 <in ' -test_expect_success 'omit delta from uninteresting base (cross pack)' ' +test_expect_failure 'omit delta from uninteresting base (cross pack)' ' cat >in <<-EOF && $(git rev-parse $base) ^$(git rev-parse $delta)
-- 2.46.0.426.g82754d92509