git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v3 3/4] test: completion: add tests for __git_complete

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Dec 30, 2020, 23:29 UTC
Message-ID
<20201230232950.582152-4-felipe.contreras@gmail.com>
In-Reply-To
<20201230232950.582152-1-felipe.contreras@gmail.com>

Even though the function was marked as not public, it's already used in the wild.

We should at least test basic functionality.
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 t/t9902-completion.sh | 8 ++++++++
 1 file changed, 8 insertions(+)
diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh
index a1c4f1f6d4..c0b4380eae 100755
--- a/t/t9902-completion.sh
+++ b/t/t9902-completion.sh
@@ -2380,4 +2380,12 @@ test_expect_success 'sourcing the completion script clears cached --options' '
 	verbose test -z "$__gitcomp_builtin_notes_edit"
 '
 
+test_expect_success '__git_complete' '
+	unset -f __git_wrap__git_main &&
+	__git_complete foo __git_main &&
+	__git_have_func __git_wrap__git_main &&
+	__git_complete gf _git_fetch &&
+	__git_have_func __git_wrap_git_fetch
+'
+
 test_done
-- 
2.30.0
Previous: Felipe ContrerasNext: Felipe Contreras
Message 4 of 5 in “completion: make __git_complete public”
  1. 0/4 completion: make __git_complete publicFelipe Contreras, Dec 30, 2020
  2. 2/4 completion: bash: improve function detectionFelipe Contreras, Dec 30, 2020
  3. 1/4 completion: bash: add __git_have_func helperFelipe Contreras, Dec 30, 2020
  4. 3/4 test: completion: add tests for __git_completeFelipe Contreras, Dec 30, 2020
  5. 4/4 completion: add proper public __git_completeFelipe Contreras, Dec 30, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.