diff options
author | SZEDER Gábor <szeder@ira.uka.de> | 2015-05-10 14:50:17 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2015-05-12 15:12:19 -0700 |
commit | e8f9e42829ddb966194f0d38443f321113032bb0 (patch) | |
tree | a83cc3878cd43a71ecb3d9516b3c2ae95d3af1ad /t/t9902-completion.sh | |
parent | c518059b263bb506b96a0ae90c4d40408c760cb0 (diff) | |
download | git-e8f9e42829ddb966194f0d38443f321113032bb0.tar.gz git-e8f9e42829ddb966194f0d38443f321113032bb0.tar.xz |
completion: add a helper function to get config variables
Currently there are a few completion functions that perform similar 'git
config' queries and filtering to get config variable names: the completion
of pretty aliases, aliases, and remote groups for 'git remote update'.
Unify those 'git config' queries in a helper function to eliminate code
duplication.
Though the helper functions to get pretty aliases and alieses are reduced
to mere one-liner wrappers around the newly added function, keep these
helpers still, because users' completion functions out there might depend
on them. And they keep their callers a tad easier to read, too.
Add tests for the pretty alias and alias helper to show that they work
as before; not for the remote groups query, though, because that's not
extracted into a helper function and it's not worth the effort to do so
for a sole callsite.
Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t9902-completion.sh')
-rwxr-xr-x | t/t9902-completion.sh | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh index 4a14a5892..07f2478c9 100755 --- a/t/t9902-completion.sh +++ b/t/t9902-completion.sh @@ -370,6 +370,28 @@ test_expect_success '__git_remotes - list remotes from $GIT_DIR/remotes and from test_cmp expect actual ' +test_expect_success '__git_pretty_aliases' ' + cat >expect <<-EOF && + author + hash + EOF + test_config pretty.author "%an %ae" && + test_config pretty.hash %H && + __git_pretty_aliases >actual && + test_cmp expect actual +' + +test_expect_success '__git_aliases' ' + cat >expect <<-EOF && + ci + co + EOF + test_config alias.ci commit && + test_config alias.co checkout && + __git_aliases >actual && + test_cmp expect actual +' + test_expect_success 'basic' ' run_completion "git " && # built-in |