diff options
| author |  Junio C Hamano <gitster@pobox.com> | 2016-08-08 14:21:46 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-08-08 14:21:46 -0700 |
| commit | f7b01d3eb76b5c1d04b774454364e87b49e80b56 (patch) | |
| tree | 01202bfe9490f404e011ad3ade90ea0f0c1404cf /t | |
| parent | Merge branch 'sb/submodule-deinit-all' into maint (diff) | |
| parent | submodule-config: fix test binary crashing when no arguments given (diff) | |
| download | tgif-f7b01d3eb76b5c1d04b774454364e87b49e80b56.tar.xz | |
Merge branch 'rs/submodule-config-code-cleanup' into maint
Code cleanup.
* rs/submodule-config-code-cleanup:
submodule-config: fix test binary crashing when no arguments given
submodule-config: combine early return code into one goto
submodule-config: passing name reference for .gitmodule blobs
submodule-config: use explicit empty string instead of strbuf in config_from()
Diffstat (limited to 't')
| -rw-r--r-- | t/helper/test-submodule-config.c | 2 | ||||
| -rwxr-xr-x | t/t7411-submodule-config.sh | 11 |
2 files changed, 12 insertions, 1 deletions
diff --git a/t/helper/test-submodule-config.c b/t/helper/test-submodule-config.c index dab8c27768..a4e4098a0f 100644 --- a/t/helper/test-submodule-config.c +++ b/t/helper/test-submodule-config.c @@ -23,7 +23,7 @@ int main(int argc, char **argv) arg++; my_argc--; - while (starts_with(arg[0], "--")) { + while (arg[0] && starts_with(arg[0], "--")) { if (!strcmp(arg[0], "--url")) output_url = 1; if (!strcmp(arg[0], "--name")) diff --git a/t/t7411-submodule-config.sh b/t/t7411-submodule-config.sh index fc97c3314e..400e2b1439 100755 --- a/t/t7411-submodule-config.sh +++ b/t/t7411-submodule-config.sh @@ -82,6 +82,17 @@ test_expect_success 'error in one submodule config lets continue' ' ) ' +test_expect_success 'error message contains blob reference' ' + (cd super && + sha1=$(git rev-parse HEAD) && + test-submodule-config \ + HEAD b \ + HEAD submodule \ + 2>actual_err && + grep "submodule-blob $sha1:.gitmodules" actual_err >/dev/null + ) +' + cat >super/expect_url <<EOF Submodule url: 'git@somewhere.else.net:a.git' for path 'b' Submodule url: 'git@somewhere.else.net:submodule.git' for path 'submodule' |