From: Chandra Pratap <chandrapratap3519@gmail.com>
Date: Wed, 29 Jul 2026 09:58:01 GMT
Subject: Re: [PATCH GSoC 4/5] serve: advertise type capability
Message-ID: <CA+J6zkQKHVhDLSXYz=hFtfnACHviSVZEDignagbCVLhEWBZiJg@mail.gmail.com>
In-Reply-To: <20260725-objecttype-support-v1-4-2d4ca3bbabf1@gmail.com>

[snip]
> -# This tests depends on %(objecttype) not being supported yet, once supported
> -# it needs to be updated.
> -test_expect_success 'unsupported placeholder on remote returns empty string' '
> +test_expect_success 'objecttype is supported by remote-object-info' '
>         (
>                 set_transport_variables "$daemon_parent" &&
>                 cd "$daemon_parent/daemon_client_empty" &&
>
> -               echo "" >expect &&
> +               echo $hello_type >expect &&
>                 git cat-file --batch-command="%(objecttype)" >actual <<-EOF &&
>                 remote-object-info "$GIT_DAEMON_URL/parent" $hello_oid
>                 EOF

Instead of this, what about creating a single test that verifies
'type' is supported,
and modifying this test to verify that the other options are not?

That would actually preserve this test's behaviour and make it easier
to extend in the future. Something like:

+ test_expect_success 'type is supported by remote-object-info'
+ test_expect_success 'unsupported placeholder on remote returns empty string'