Jeff King <p...@peff.net> writes:

> On Wed, Apr 03, 2013 at 12:10:38PM -0400, Jeff King wrote:
>
>> Hmm. This is testing just the ref advertisement. It would be nice to see
>> a complete transaction tested with namespaces turned on. Something like
>> this (squashed into your patch) seems to work for me:
>
> Actually, I guess the point of your patch was to fix the
> dumb-via-http-backend transport. So this would be more complete:

Yeah, sounds sensible to me.

> diff --git a/t/t5551-http-fetch.sh b/t/t5551-http-fetch.sh
> index 47eb769..b5032bd 100755
> --- a/t/t5551-http-fetch.sh
> +++ b/t/t5551-http-fetch.sh
> @@ -162,6 +162,28 @@ test_expect_success 'invalid Content-Type rejected' '
>       grep "not valid:" actual
>  '
>  
> +test_expect_success 'create namespaced refs' '
> +     test_commit namespaced &&
> +     git push public HEAD:refs/namespaces/ns/refs/heads/master
> +'
> +
> +test_expect_success 'smart clone respects namespace' '
> +     git clone --bare "$HTTPD_URL/smart_namespace/repo.git" ns-smart.git &&
> +     echo namespaced >expect &&
> +     git --git-dir=ns-smart.git log -1 --format=%s >actual &&
> +     test_cmp expect actual
> +'
> +
> +test_expect_success 'dumb clone via http-backend respects namespace' '
> +     git --git-dir="$HTTPD_DOCUMENT_ROOT_PATH/repo.git" \
> +             config http.getanyfile true &&
> +     GIT_SMART_HTTP=0 git clone --bare \
> +             "$HTTPD_URL/smart_namespace/repo.git" ns-dumb.git &&
> +     echo namespaced >expect &&
> +     git --git-dir=ns-dumb.git log -1 --format=%s >actual &&
> +     test_cmp expect actual
> +'
> +
>  test -n "$GIT_TEST_LONG" && test_set_prereq EXPENSIVE
>  
>  test_expect_success EXPENSIVE 'create 50,000 tags in the repo' '
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to