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: 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html