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

Re: [PATCH v2] test-lib-functions.sh: fix test_grep fail message wording

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 18, 2023, 18:47 UTC
Message-ID
<xmqqh6kfe4am.fsf@gitster.g>
In-Reply-To
<xmqqjzpbh3kq.fsf@gitster.g>
Junio C Hamano <gitster@pobox.com> writes:
Show 5 quoted lines
> I'll see if that is the only glitch in the patch (in which case I'll
> manually adjust the authorship and apply) or respond on list
> (otherwise).
>
> Thanks for pinging and ponging.

Here is the version I queued. Thanks, both.

--- >8 ---
From: Shreyansh Paliwal <shreyanshpaliwalcmsmn@gmail.com>
Date: Sun, 3 Dec 2023 22:47:59 +0530
Subject: [PATCH] test-lib-functions.sh: fix test_grep fail message wording

In the recent commit 2e87fca189 (test framework: further deprecate test_i18ngrep, 2023-10-31), the test_i18ngrep function was deprecated, and all the callers were updated to call the test_grep function instead. But test_grep inherited an error message that still refers to test_i18ngrep by mistake. Correct it so that a broken call to the test_grep will identify itself as such.

Signed-off-by: Shreyansh Paliwal <shreyanshpaliwalcmsmn@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 t/test-lib-functions.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh
index c50bc18861..502f892fad 100644
--- a/t/test-lib-functions.sh
+++ b/t/test-lib-functions.sh
@@ -1222,7 +1222,7 @@ test_grep () {
 	if test $# -lt 2 ||
 	   { test "x!" = "x$1" && test $# -lt 3 ; }
 	then
-		BUG "too few parameters to test_i18ngrep"
+		BUG "too few parameters to test_grep"
 	fi
 
 	if test "x!" = "x$1"
-- 
2.43.0-76-g1a87c842ec
Previous: Junio C HamanoNext: Eric Sunshine
Message 9 of 10 in “test-lib-functions.sh : change test_i18ngrep to test_grep”
  1. test-lib-functions.sh : change test_i18ngrep to test_grepShreyansh Paliwal, Dec 2, 2023
  2. Kousik SanagavarapuDec 3, 2023
  3. Junio C HamanoDec 3, 2023
  4. test-lib-functions.sh: fix test_grep fail message wordingShreyansh Paliwal, Dec 3, 2023
  5. Kousik SanagavarapuDec 4, 2023
  6. Shreyansh PaliwalDec 17, 2023
  7. Eric SunshineDec 18, 2023
  8. Junio C HamanoDec 18, 2023
  9. Junio C HamanoDec 18, 2023
  10. Eric SunshineDec 18, 2023

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.