This only makes sense for the python remote helpers framework. The tests
don't exercise any feature of transport helper. Remove them.

Signed-off-by: Felipe Contreras <felipe.contre...@gmail.com>
Signed-off-by: Junio C Hamano <gits...@pobox.com>
---
 git-remote-testgit        |  3 ---
 t/t5801-remote-helpers.sh | 50 ++++++++++++++++++++---------------------------
 2 files changed, 21 insertions(+), 32 deletions(-)

diff --git a/git-remote-testgit b/git-remote-testgit
index 5042f5a..5117ab5 100755
--- a/git-remote-testgit
+++ b/git-remote-testgit
@@ -4,9 +4,6 @@
 alias=$1
 url=$2
 
-# huh?
-url="${url#file://}"
-
 dir="$GIT_DIR/testgit/$alias"
 prefix="refs/testgit/$alias"
 refspec="refs/heads/*:${prefix}/heads/*"
diff --git a/t/t5801-remote-helpers.sh b/t/t5801-remote-helpers.sh
index f52ab14..2f7fc10 100755
--- a/t/t5801-remote-helpers.sh
+++ b/t/t5801-remote-helpers.sh
@@ -33,11 +33,6 @@ test_expect_success 'cloning from local repo' '
        test_cmp public/file localclone/file
 '
 
-test_expect_success 'cloning from remote repo' '
-       git clone "testgit::file://${PWD}/server" clone &&
-       test_cmp public/file clone/file
-'
-
 test_expect_success 'create new commit on remote' '
        (cd public &&
         echo content >>file &&
@@ -50,11 +45,6 @@ test_expect_success 'pulling from local repo' '
        test_cmp public/file localclone/file
 '
 
-test_expect_success 'pulling from remote remote' '
-       (cd clone && git pull) &&
-       test_cmp public/file clone/file
-'
-
 test_expect_success 'pushing to local repo' '
        (cd localclone &&
        echo content >>file &&
@@ -76,19 +66,6 @@ test_expect_success 'pushing to local repo' '
        compare_refs localclone2 HEAD server2 HEAD
 '
 
-test_expect_success 'synch with changes from localclone' '
-       (cd clone &&
-        git pull)
-'
-
-test_expect_success 'pushing remote local repo' '
-       (cd clone &&
-       echo content >>file &&
-       git commit -a -m four &&
-       git push) &&
-       compare_refs clone HEAD server HEAD
-'
-
 test_expect_success 'fetch new branch' '
        (cd public &&
         git checkout -b new &&
@@ -102,6 +79,20 @@ test_expect_success 'fetch new branch' '
        compare_refs public HEAD localclone FETCH_HEAD
 '
 
+#
+# This is only needed because of a bug not detected by this script. It will be
+# fixed shortly, but for now lets not cause regressions.
+#
+test_expect_success 'bump commit in public' '
+       (cd public &&
+       git checkout master &&
+       git pull &&
+       echo content >>file &&
+       git commit -a -m four &&
+       git push) &&
+       compare_refs public HEAD server HEAD
+'
+
 test_expect_success 'fetch multiple branches' '
        (cd localclone &&
         git fetch
@@ -111,29 +102,30 @@ test_expect_success 'fetch multiple branches' '
 '
 
 test_expect_success 'push when remote has extra refs' '
-       (cd clone &&
+       (cd localclone &&
+        git reset --hard origin/master &&
         echo content >>file &&
         git commit -a -m six &&
         git push
        ) &&
-       compare_refs clone master server master
+       compare_refs localclone master server master
 '
 
 test_expect_success 'push new branch by name' '
-       (cd clone &&
+       (cd localclone &&
         git checkout -b new-name  &&
         echo content >>file &&
         git commit -a -m seven &&
         git push origin new-name
        ) &&
-       compare_refs clone HEAD server refs/heads/new-name
+       compare_refs localclone HEAD server refs/heads/new-name
 '
 
 test_expect_failure 'push new branch with old:new refspec' '
-       (cd clone &&
+       (cd localclone &&
         git push origin new-name:new-refspec
        ) &&
-       compare_refs clone HEAD server refs/heads/new-refspec
+       compare_refs localclone HEAD server refs/heads/new-refspec
 '
 
 test_done
-- 
1.8.0.1

--
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