Merge branch 'tb/banned-vsprintf-namefix'
[gitweb.git] / t / t5703-upload-pack-ref-in-want.sh
index 0951d1bbdc26d998cdca7ed491eca557d53b2ddf..3a2c143c6d677755a9eb4c60ea9e272698b793d1 100755 (executable)
@@ -157,106 +157,6 @@ test_expect_success 'want-ref with ref we already have commit for' '
        check_output
 '
 
-. "$TEST_DIRECTORY"/lib-httpd.sh
-start_httpd
-
-REPO="$HTTPD_DOCUMENT_ROOT_PATH/repo"
-LOCAL_PRISTINE="$(pwd)/local_pristine"
-
-test_expect_success 'setup repos for change-while-negotiating test' '
-       (
-               git init "$REPO" &&
-               cd "$REPO" &&
-               >.git/git-daemon-export-ok &&
-               test_commit m1 &&
-               git tag -d m1 &&
-
-               # Local repo with many commits (so that negotiation will take
-               # more than 1 request/response pair)
-               git clone "http://127.0.0.1:$LIB_HTTPD_PORT/smart/repo" "$LOCAL_PRISTINE" &&
-               cd "$LOCAL_PRISTINE" &&
-               git checkout -b side &&
-               for i in $(test_seq 1 33); do test_commit s$i; done &&
-
-               # Add novel commits to upstream
-               git checkout master &&
-               cd "$REPO" &&
-               test_commit m2 &&
-               test_commit m3 &&
-               git tag -d m2 m3
-       ) &&
-       git -C "$LOCAL_PRISTINE" remote set-url origin "http://127.0.0.1:$LIB_HTTPD_PORT/one_time_sed/repo" &&
-       git -C "$LOCAL_PRISTINE" config protocol.version 2
-'
-
-inconsistency () {
-       # Simulate that the server initially reports $2 as the ref
-       # corresponding to $1, and after that, $1 as the ref corresponding to
-       # $1. This corresponds to the real-life situation where the server's
-       # repository appears to change during negotiation, for example, when
-       # different servers in a load-balancing arrangement serve (stateless)
-       # RPCs during a single negotiation.
-       printf "s/%s/%s/" \
-              $(git -C "$REPO" rev-parse $1 | tr -d "\n") \
-              $(git -C "$REPO" rev-parse $2 | tr -d "\n") \
-              >"$HTTPD_ROOT_PATH/one-time-sed"
-}
-
-test_expect_success 'server is initially ahead - no ref in want' '
-       git -C "$REPO" config uploadpack.allowRefInWant false &&
-       rm -rf local &&
-       cp -r "$LOCAL_PRISTINE" local &&
-       inconsistency master 1234567890123456789012345678901234567890 &&
-       test_must_fail git -C local fetch 2>err &&
-       test_i18ngrep "fatal: remote error: upload-pack: not our ref" err
-'
-
-test_expect_success 'server is initially ahead - ref in want' '
-       git -C "$REPO" config uploadpack.allowRefInWant true &&
-       rm -rf local &&
-       cp -r "$LOCAL_PRISTINE" local &&
-       inconsistency master 1234567890123456789012345678901234567890 &&
-       git -C local fetch &&
-
-       git -C "$REPO" rev-parse --verify master >expected &&
-       git -C local rev-parse --verify refs/remotes/origin/master >actual &&
-       test_cmp expected actual
-'
-
-test_expect_success 'server is initially behind - no ref in want' '
-       git -C "$REPO" config uploadpack.allowRefInWant false &&
-       rm -rf local &&
-       cp -r "$LOCAL_PRISTINE" local &&
-       inconsistency master "master^" &&
-       git -C local fetch &&
-
-       git -C "$REPO" rev-parse --verify "master^" >expected &&
-       git -C local rev-parse --verify refs/remotes/origin/master >actual &&
-       test_cmp expected actual
-'
-
-test_expect_success 'server is initially behind - ref in want' '
-       git -C "$REPO" config uploadpack.allowRefInWant true &&
-       rm -rf local &&
-       cp -r "$LOCAL_PRISTINE" local &&
-       inconsistency master "master^" &&
-       git -C local fetch &&
-
-       git -C "$REPO" rev-parse --verify "master" >expected &&
-       git -C local rev-parse --verify refs/remotes/origin/master >actual &&
-       test_cmp expected actual
-'
-
-test_expect_success 'server loses a ref - ref in want' '
-       git -C "$REPO" config uploadpack.allowRefInWant true &&
-       rm -rf local &&
-       cp -r "$LOCAL_PRISTINE" local &&
-       echo "s/master/raster/" >"$HTTPD_ROOT_PATH/one-time-sed" &&
-       test_must_fail git -C local fetch 2>err &&
-
-       test_i18ngrep "fatal: remote error: unknown ref refs/heads/raster" err
-'
-
 REPO="$(pwd)/repo"
 LOCAL_PRISTINE="$(pwd)/local_pristine"
 
@@ -287,7 +187,7 @@ test_expect_success 'setup repos for fetching with ref-in-want tests' '
                git clone "file://$REPO" "$LOCAL_PRISTINE" &&
                cd "$LOCAL_PRISTINE" &&
                git checkout -b side &&
-               for i in $(test_seq 1 33); do test_commit s$i; done &&
+               test_commit_bulk --id=s 33 &&
 
                # Add novel commits to upstream
                git checkout master &&
@@ -372,4 +272,108 @@ test_expect_success 'fetching with wildcard that matches multiple refs' '
        grep "want-ref refs/heads/o/bar" log
 '
 
+. "$TEST_DIRECTORY"/lib-httpd.sh
+start_httpd
+
+REPO="$HTTPD_DOCUMENT_ROOT_PATH/repo"
+LOCAL_PRISTINE="$(pwd)/local_pristine"
+
+test_expect_success 'setup repos for change-while-negotiating test' '
+       (
+               git init "$REPO" &&
+               cd "$REPO" &&
+               >.git/git-daemon-export-ok &&
+               test_commit m1 &&
+               git tag -d m1 &&
+
+               # Local repo with many commits (so that negotiation will take
+               # more than 1 request/response pair)
+               rm -rf "$LOCAL_PRISTINE" &&
+               git clone "http://127.0.0.1:$LIB_HTTPD_PORT/smart/repo" "$LOCAL_PRISTINE" &&
+               cd "$LOCAL_PRISTINE" &&
+               git checkout -b side &&
+               test_commit_bulk --id=s 33 &&
+
+               # Add novel commits to upstream
+               git checkout master &&
+               cd "$REPO" &&
+               test_commit m2 &&
+               test_commit m3 &&
+               git tag -d m2 m3
+       ) &&
+       git -C "$LOCAL_PRISTINE" remote set-url origin "http://127.0.0.1:$LIB_HTTPD_PORT/one_time_sed/repo" &&
+       git -C "$LOCAL_PRISTINE" config protocol.version 2
+'
+
+inconsistency () {
+       # Simulate that the server initially reports $2 as the ref
+       # corresponding to $1, and after that, $1 as the ref corresponding to
+       # $1. This corresponds to the real-life situation where the server's
+       # repository appears to change during negotiation, for example, when
+       # different servers in a load-balancing arrangement serve (stateless)
+       # RPCs during a single negotiation.
+       printf "s/%s/%s/" \
+              $(git -C "$REPO" rev-parse $1 | tr -d "\n") \
+              $(git -C "$REPO" rev-parse $2 | tr -d "\n") \
+              >"$HTTPD_ROOT_PATH/one-time-sed"
+}
+
+test_expect_success 'server is initially ahead - no ref in want' '
+       git -C "$REPO" config uploadpack.allowRefInWant false &&
+       rm -rf local &&
+       cp -r "$LOCAL_PRISTINE" local &&
+       inconsistency master 1234567890123456789012345678901234567890 &&
+       test_must_fail git -C local fetch 2>err &&
+       test_i18ngrep "fatal: remote error: upload-pack: not our ref" err
+'
+
+test_expect_success 'server is initially ahead - ref in want' '
+       git -C "$REPO" config uploadpack.allowRefInWant true &&
+       rm -rf local &&
+       cp -r "$LOCAL_PRISTINE" local &&
+       inconsistency master 1234567890123456789012345678901234567890 &&
+       git -C local fetch &&
+
+       git -C "$REPO" rev-parse --verify master >expected &&
+       git -C local rev-parse --verify refs/remotes/origin/master >actual &&
+       test_cmp expected actual
+'
+
+test_expect_success 'server is initially behind - no ref in want' '
+       git -C "$REPO" config uploadpack.allowRefInWant false &&
+       rm -rf local &&
+       cp -r "$LOCAL_PRISTINE" local &&
+       inconsistency master "master^" &&
+       git -C local fetch &&
+
+       git -C "$REPO" rev-parse --verify "master^" >expected &&
+       git -C local rev-parse --verify refs/remotes/origin/master >actual &&
+       test_cmp expected actual
+'
+
+test_expect_success 'server is initially behind - ref in want' '
+       git -C "$REPO" config uploadpack.allowRefInWant true &&
+       rm -rf local &&
+       cp -r "$LOCAL_PRISTINE" local &&
+       inconsistency master "master^" &&
+       git -C local fetch &&
+
+       git -C "$REPO" rev-parse --verify "master" >expected &&
+       git -C local rev-parse --verify refs/remotes/origin/master >actual &&
+       test_cmp expected actual
+'
+
+test_expect_success 'server loses a ref - ref in want' '
+       git -C "$REPO" config uploadpack.allowRefInWant true &&
+       rm -rf local &&
+       cp -r "$LOCAL_PRISTINE" local &&
+       echo "s/master/raster/" >"$HTTPD_ROOT_PATH/one-time-sed" &&
+       test_must_fail git -C local fetch 2>err &&
+
+       test_i18ngrep "fatal: remote error: unknown ref refs/heads/raster" err
+'
+
+# DO NOT add non-httpd-specific tests here, because the last part of this
+# test script is only executed when httpd is available and enabled.
+
 test_done