Re: [PATCH v3 7/9] t/lib-bitmap.sh: parameterize tests over reverse index source

From: Jonathan Tan <jonathantanmy@google.com>
Date: Jan 24, 2022, 19:15 UTC
Message-ID: <20220124191503.2447008-1-jonathantanmy@google.com>
In-Reply-To: <7ce3dc60f93d5154943a63f1c09345bd47fbc7c9.1641320129.git.me@ttaylorr.com>

Taylor Blau <me@ttaylorr.com> 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".

Show 11 quoted lines

> 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.