Re: [PATCH 1/2] daemon: add tests

On Mon, Oct 17, 2011 at 10:05:28PM +0200, Clemens Buchacher wrote:

On Sun, Oct 16, 2011 at 10:01:03PM -0400, Jeff King wrote:

quoted

Thanks, it's nice to have some tests. Overall, some of the tests feel a
little silly, because the results should be exactly the same as fetching
or pushing a local repository (so the "set-head" thing, for example,
really has little to do with git-daemon).

Hmm, yes. Actually, I thought I had found a bug with the failure of
"set-head -a". But now I see that in t5505 this treated like a
feature.

It's not a feature, exactly. It's just documenting that we fail in the
face of ambiguous HEADs. Arguably, the test should be switched to use
text_expect_failure to document that we would prefer it the other way,
but it doesn't work now.

Would it be difficult to support this over the git protocol? Maybe
I will have a look.

It needs a protocol extension to communicate symbolic ref destinations.
The topic has come up a few times, and I think Junio even had patches at
one point.

-Peff