Ben Boeckel <maths...@gmail.com> writes:

> +get_url_test () {
> +     cat >expect &&
> +     test_expect_success "get-url $*" "
> +             git remote get-url $* >actual &&
> +             test_cmp expect actual
> +     "
> +}

This makes any use of get_url_test inside test_expect_success wrong,
I suspect.  Try running the tests under prove.  I think it will
complain that the test numbers do not match or something.

Minimally, this would probably fix the breakage.

 t/t5505-remote.sh | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/t/t5505-remote.sh b/t/t5505-remote.sh
index f03ba4c..dfaf9d9 100755
--- a/t/t5505-remote.sh
+++ b/t/t5505-remote.sh
@@ -921,10 +921,8 @@ test_expect_success 'new remote' '
 
 get_url_test () {
        cat >expect &&
-       test_expect_success "get-url $*" "
-               git remote get-url $* >actual &&
-               test_cmp expect actual
-       "
+       git remote get-url "$@" >actual &&
+       test_cmp expect actual
 }
 
 test_expect_success 'get-url on new remote' '
--
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