Re: [PATCH 15/23] t5310: add branch-based checks

From: Derrick Stolee <stolee@gmail.com>
Date: Nov 11, 2020, 20:58 UTC
Message-ID: <abf9273f-4795-5a48-c28b-15e68d40b910@gmail.com>
In-Reply-To: <9ab4b94b3573346b31e710486799ab3d95bade8e.1605123653.git.me@ttaylorr.com>

On 11/11/2020 2:43 PM, Taylor Blau wrote:

Show 5 quoted lines

> From: Derrick Stolee <dstolee@microsoft.com>
> 
> The current rev-list tests that check the bitmap data only work on HEAD
> instead of multiple branches. Expand the test cases to handle both
> 'master' and 'other' branches.

Adding Johannes to CC since this likely will start colliding with his default branch rename efforts.

Show 8 quoted lines

> +rev_list_tests () {
> +	state=$1
> +
> +	for branch in "master" "other"
> +	do
> +		rev_list_tests_head
> +	done
> +}

Specifically, this is a _new_ instance of "master", but all the other instances of "master" are likely being converted to "main" in parallel. It would certainly be easier to convert this test _after_ these changes are applied, but that's unlikely to happen with the current schedule of things.

Thanks, -Stolee