Jonathan Tan <jonathantanmy@xxxxxxxxxx> writes:
> 4f37d45706 ("clone: respect remote unborn HEAD", 2021-02-05) introduces
> a new feature (if the remote has an unborn HEAD, e.g. when the remote
> repository is empty, use it as the name of the branch) that only works
> in protocol v2, but did not ensure that one of its tests always uses
> protocol v2, and thus that test would fail if
> GIT_TEST_PROTOCOL_VERSION=0 (or 1) is used. Therefore, add "-c
> protocol.version=2" to the appropriate test.
>
> (The rest of the tests from that commit have "-c protocol.version=2"
> already added.)
>
> Signed-off-by: Jonathan Tan <jonathantanmy@xxxxxxxxxx>
> ---
> Thanks, Son Luong, for noticing this. Here's a fix for the
> GIT_TEST_PROTOCOL_VERSION part. This was built on 4f37d45706 but also
> applies cleanly on master.
Makes sense. And I do not see need for any other changes, like
test_expect_failure with protocol 0 (or 1).
Will queue as a candidate for maint-2.31.
Thanks.
>
> t/t5606-clone-options.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/t/t5606-clone-options.sh b/t/t5606-clone-options.sh
> index ca6339a5fb..5e30772735 100755
> --- a/t/t5606-clone-options.sh
> +++ b/t/t5606-clone-options.sh
> @@ -106,7 +106,7 @@ test_expect_success 'chooses correct default initial branch name' '
> git -c init.defaultBranch=foo init --bare empty &&
> test_config -C empty lsrefs.unborn advertise &&
> GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME= \
> - git -c init.defaultBranch=up clone empty whats-up &&
> + git -c init.defaultBranch=up -c protocol.version=2 clone empty whats-up &&
> test refs/heads/foo = $(git -C whats-up symbolic-ref HEAD) &&
> test refs/heads/foo = $(git -C whats-up config branch.foo.merge)
> '
