Taylor Blau <me@xxxxxxxxxxxx> writes: > To prepare for reading the reverse index data out of the MIDX itself, > teach the `test_rev_exists` function to take an expected "source" for > the reverse index data. Thanks - up to here looks good. Thanks especially for patch 6, which was easy to verify using "--color-moved --color-moved-ws=allow-indentation-change". > diff --git a/t/lib-bitmap.sh b/t/lib-bitmap.sh > index 48a8730a13..77b5f46a03 100644 > --- a/t/lib-bitmap.sh > +++ b/t/lib-bitmap.sh > @@ -275,17 +275,23 @@ midx_pack_source () { > > test_rev_exists () { > commit="$1" > + kind="$2" > > test_expect_success 'reverse index exists' ' To make it easier to understand test failures, we should probably include "kind" in the name of the test case. In a separate commit after this one, we should do it for the other blocks.