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

> +test_expect_success 'setup a tree' '
> +     mkdir sub &&
> +     for p in fileA fileB fileC fileAB fileAC fileBC fileNoLabel 
> fileUnsetLabel fileSetLabel fileValue fileWrongLabel; do
> +             : >$p &&
> +             git add $p &&
> +             : >sub/$p
> +             git add sub/$p
> +     done &&
> +     git commit -m $p &&

What does this $p refer to?

> +     git ls-files >actual &&
> +     cat <<EOF >expect &&
> +fileA
> +fileAB
> +fileAC
> +fileB
> +fileBC
> +fileC
> +fileNoLabel
> +fileSetLabel
> +fileUnsetLabel
> +fileValue
> +fileWrongLabel
> +sub/fileA
> +sub/fileAB
> +sub/fileAC
> +sub/fileB
> +sub/fileBC
> +sub/fileC
> +sub/fileNoLabel
> +sub/fileSetLabel
> +sub/fileUnsetLabel
> +sub/fileValue
> +sub/fileWrongLabel
> +EOF
> +     test_cmp expect actual
> +'

If I were doing this, I'd prepare the list of paths (i.e. expect)
first and then create these paths using that list, i.e.

test_expect_success 'setup a tree' '
        cat <<-\EOF >expect &&
        fileA
        fileAB
        ...
        sub/fileWrongLabel
        EOF
        mkdir sub &&
        while read path
        do
                : >$path &&
                git add $path || return 1
        done <expect &&
        git commit -m initial &&
        git ls-files >actual &&
        test_cmp expect actual
'

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