Stefan Beller <sbel...@google.com> writes:

> diff --git a/t/t7400-submodule-basic.sh b/t/t7400-submodule-basic.sh
> index 814ee63..0adc4e4 100755
> --- a/t/t7400-submodule-basic.sh
> +++ b/t/t7400-submodule-basic.sh
> @@ -1056,6 +1056,7 @@ test_expect_success 'submodule with UTF-8 name' '
>  '
>  
>  test_expect_success 'submodule add clone shallow submodule' '
> +     test_when_finished "rm -rf super" &&
>       mkdir super &&
>       pwd=$(pwd) &&
>       (
> @@ -1094,5 +1095,48 @@ test_expect_success 'submodule helper list is not 
> confused by common prefixes' '
>       test_cmp expect actual
>  '
>  
> +test_expect_success 'submodule add records a label' '
> +     test_when_finished "rm -rf super" &&
> +     mkdir super &&
> +     pwd=$(pwd) &&
> +     (
> +             cd super &&
> +             git init &&
> +             git submodule add --label labelA file://"$pwd"/example2 
> submodule &&
> +             git config -f .gitmodules submodule."submodule".label 
> >../actual &&
> +             echo labelA >../expect
> +     ) &&
> +     test_cmp expect actual
> +'
> +
> +cat >expect <<-EOF
> +labelA
> +labelB
> +EOF
> +
> +test_expect_success 'submodule add records multiple labels' '

The existing tests in this file may be littered with this bad
construct, but please do not add more example of running things
outside of test_expect_{success,failure} block unless there is a
good reason to do so.
--
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